From: Brian Takita <brian.takita@...>
Date: 2010-10-23T04:14:30+09:00
Subject: Re: minitest/spec pull request

On Thu, Oct 21, 2010 at 1:04 PM, Brian Takita <brian.takita@gmail.com> wrote:
> Hello, I submitted a change request that makes the failure output in
> mini spec show the exact string of the describe block, rather than the
> camel-cased version of the string.
> http://github.com/seattlerb/minitest/pull/4
Oops, I messed up the pull request. Here is the correct one.
http://github.com/seattlerb/minitest/pull/5
>
> I was wondering what people thought.
>
> e.g.
>
> describe "A fool" do
>  describe "Running a fools errand" do
>    it "is impotent" do
>      true.must_eql false
>    end
>  end
> end
>
>  1) Failure:
> test_0001_is_impotent(AFoolSpec::RunningAFoolsErrandSpec)
> [/home/honk/workspace/honk/junk.rb:9]:
> Expected false, not true.
>
> This patch creates the following output:
>
>  1) Failure:
> test_0001_is_impotent(A fool::Running a fools errand)
> [/home/honk/workspace/honk/junk.rb:9]:
> Expected false, not true.
>
> Converting the it block to test_0001_is_impotent is a bit weird too.
> Maybe the method could just have 'test' prepended to the exact string?
>
>  1) Failure:
> test_0001 - is impotent (A fool::Running a fools errand)
> [/home/honk/workspace/honk/junk.rb:9]:
> Expected false, not true.
>
> Thank you,
> Brian
>