This project is archived and is in readonly mode.
Removing $! from assert_select_test
Hi Guys,
This patch removes $! from assert_select_test, and uses normal exception syntax, which is more readable.
-
Rohit Arondekar- Assigned user changed from José Valim to Santiago Pastorino
- Importance changed from to Low
+1
Assigning to Santiago because I think Jose is on holiday. :)
-
Thiago PradiAttached another patch to also remove it from ActiveRecord.
-
Thiago Pradi
Another one for ActiveSupport.
-
Thiago Pradi
Another for ActiveResource.
-
Santiago PastorinoCan you join all the patches in one please i will apply them ;) thanks.
-
-
Repository
- State changed from new to committed
(from [3c9bf6e1dc3066e32651c955c6e237488115354d]) Exception handling more readable
[#5601 Removing $! from assert_select_test state:committed]
Signed-off-by: Santiago Pastorino santiago@wyeworks.com
http://github.com/rails/rails/commit/3c9bf6e1dc3066e32651c955c6e237... -
Repository
(from [ab59522ea1ded4dcf8fd6c6f94bbc7385e50c298]) Exception handling more readable
[#5601 Removing $! from assert_select_test state:committed]
Signed-off-by: Santiago Pastorino santiago@wyeworks.com
http://github.com/rails/rails/commit/ab59522ea1ded4dcf8fd6c6f94bbc7...