Test suite is using assert many times when should be used assert_equal

One more added

+1

Patch applies clean in rails edge. All test pass.

Maybe should be usefull use in all the test the same expression with the '!' character, so:

assert ! content_for?(:title)

Would write like:

assert !content_for?(:title)

For all the test suite, or maybe this is useless, what do you think?

+1

Applies to rails edge, all test pass.

Does not apply to 2-3-stable though :

error: patch failed: activesupport/test/caching_test.rb:375

error: activesupport/test/caching_test.rb: patch does not apply

+1 Applies to rails edge and the tests pass. Just one thing though, a few of those (the ones from action pack) are repeating the ones I cleared in http://github.com/rails/rails/commit/fc2480a277fd4753aebf68dcf8787d...

Some of the things i did were applied on others tickets, so i deleted the repeated ones.

Looks like it's already been applied (no changes when I patch on both master and 2-3-stable).