On 2/23/07, Chad Fowler <[EMAIL PROTECTED]> wrote:
>
> What you're saying is "simple" but incorrect.
>
> I doubt anyone on the Rails core team would agree that broken tests
> aren't real defects or that fixing them isn't a high priority.


When I said "real defects" I meant defects in the codebase (I thought one
could tell from the context). Tests can have defects, of course, as well as
any other code. I've submitted a proper patch to tackle this specific issue:
http://dev.rubyonrails.org/attachment/ticket/7614/hash-comparison.diff

It modifies the test so that the hash string is extracted from the error
message, broken into parts, sorted and compared. The actual error messages
are compared without hashes in them.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to