This project is archived and is in readonly mode.
remove unused require from association test
there are several unused model require statements in
association_test.
this is simple patch for rails 3 to remove them
-
remove_require.diff
1.36 KB
-
Subbathere are several tests which have unnecessary require statements. i will submit patch with all the changes.
-
-
Neeraj Singh- State changed from new to open
- Tag set to rails 3, tests
- Milestone set to 3.x
- Importance changed from to Low
Please ensure that you apply this patch at the very end of your patches list. Otherwise a lot of patches will fail and we all will have to redo them. :-)
-
José ValimNo longer applies, rebase please?
-
Subba
i am trying to test after rebase. but the arel fork for rails is breaking many tests #5173 Id and parameter in path is not properly escaped for ActiveResource.
-
Subba
updating patch after rebase.
also attach bash script that will run changed test files individually(you need to run from activerecord directory).-
5153.diff
9.15 KB
-
run_test.sh
446 Bytes
-
-
Repository
- State changed from open to resolved
(from [b0c7dee4f2744286069c5ff4e65c3d081a4cb24a]) removing unused models from tests
[#5153 remove unused require from association test state:resolved]
Signed-off-by: José Valim jose.valim@gmail.com
http://github.com/rails/rails/commit/b0c7dee4f2744286069c5ff4e65c3d...