Commit Graph

23323 Commits

Author SHA1 Message Date
Ken Collins
c791e2d034 Allow the connection pool's #table_exists? method to give the connections #table_exists? method a chance. 2011-06-10 13:15:58 -04:00
José Valim
8eb2b519f2 Merge pull request #1631 from vijaydev/runner-help
rails runner without any options should show usage information
2011-06-10 06:45:52 -07:00
Vijay Dev
880371ef2b make 'rails runner' show usage when run without any options 2011-06-10 18:48:50 +05:30
José Valim
78acd17f31 Merge pull request #1567 from BDQ/engine_migrations
Ensure engine migrations are copied in correct sequence
2011-06-10 04:43:43 -07:00
Brian Quinn
25ad43015c Updated test to ensure correct sequence when copying engine/plugin migrations. 2011-06-10 12:12:50 +01:00
Brian Quinn
9f2e91414f Ensure engine migrations are copied in the same sequence that they are required. 2011-06-10 12:11:45 +01:00
Andrew White
0bcbcdfdad Ensure number helpers can handle HTML safe strings - closes #1597. 2011-06-10 11:32:28 +01:00
Xavier Noria
90c4885766 Merge pull request #1523 from raghunadhd/refactor_regex
rafactored the regex related code in the mapper class
2011-06-10 03:27:45 -07:00
Raghunadh
46ecb53fc2 removed the unnecessary back slashes from the regular expression 2011-06-10 13:39:24 +05:30
José Valim
2bda229c2f Merge pull request #1625 from neerajdotname/page_caching
Three minor fixes to tests
2011-06-10 00:29:50 -07:00
Neeraj Singh
06d772c0ce add missing ensure 2011-06-10 02:48:44 -04:00
Neeraj Singh
fdbe8eb08c replaced and with && as per coding guidelines 2011-06-10 02:27:57 -04:00
Neeraj Singh
49d032929b The test name is misleading. It says
test_should_cache_with_trailing_slash_on_url

A trailing slash is removed when a page is cached.

What the test meant to say was : a url with trailing slash
should be cached and the cached page should not have any
trailing slash.

This patch clarifies the name a bit.
2011-06-10 02:24:37 -04:00
Aaron Patterson
0b02284545 ensuring that json_escape returns html safe strings when passed an html safe string 2011-06-09 15:29:17 -07:00
Damien Mathieu
84f71e42b7 don't reinitialize the arel_table unless the table_name changes 2011-06-09 14:56:07 -07:00
Jacob Poulsgaard Tjoernholm
35ae29f305 Fix issue #1598 by adding a dependency to the RDoc gem. 2011-06-09 14:37:54 -07:00
Jon Leighton
4b371b602b Print out information about whether the identity map is on or off when running unit tests 2011-06-09 20:20:51 +01:00
Jon Leighton
e48829946d Update the RUNNING_UNIT_TESTS file 2011-06-09 20:20:51 +01:00
Aaron Patterson
d0a1e37d70 Merge pull request #1609 from sikachu/master-javascript
Make sure `escape_javascript` return `SafeBuffer` if the incoming argumen
2011-06-09 11:37:06 -07:00
Prem Sichanugrist
8c8652c37f Make sure escape_javascript return SafeBuffer if the incoming argument is already html_safe 2011-06-09 14:30:02 -04:00
José Valim
834bebf9cc Merge pull request #1606 from neerajdotname/remove_users_controller_from_test
remove unused UserController from test
2011-06-09 08:30:45 -07:00
José Valim
eb8c0a7b1a Merge pull request #1356 from flippingbits/fix_engine_generator
Fix engine's generator
2011-06-09 00:01:00 -07:00
José Valim
af3f0b6204 Merge pull request #1591 from smartinez87/unused
Fix "warning: assigned but unused variable" on AR tests
2011-06-08 23:36:49 -07:00
José Valim
077f65c36f Merge pull request #1588 from neerajdotname/remove_unused_class_with_symbol_returning
class WithSymbolReturningString is not used anywhere in the test.
2011-06-08 23:35:46 -07:00
José Valim
0ad228027d Merge pull request #1579 from bradleybuda/master
Date#freeze fails when called more than once in 1.8
2011-06-08 23:34:30 -07:00
José Valim
12245a3325 Merge pull request #1577 from reu/useless-variable-set
Instance variable not used.
2011-06-08 23:33:42 -07:00
Aaron Patterson
9295053d9d Merge pull request #1593 from r00k/master
Simplify boolean logic into ternary
2011-06-08 20:29:12 -07:00
Ben Orenstein
f87888066a Simplify boolean logic into ternary. 2011-06-08 23:11:19 -04:00
Neeraj Singh
fa5b34ed22 remove unused user_controller from test 2011-06-08 22:49:38 -04:00
Sebastian Martinez
9af4e803ec Remove unused 'y' variable. 2011-06-08 23:13:59 -03:00
Sebastian Martinez
0b86ac244c Remove unused 'quoted_column_names' variable. 2011-06-08 23:13:43 -03:00
Neeraj Singh
277eb05733 class WithSymbolReturningString is not used anywhere in the test.
Secondly it seemed from the method that the intent was to test a case
where layout was declared in a symbol and the method named mention in
layout returns nil.

That case is already covered with class     class WithSymbolReturningNil .

Also the case of SymbolReturningString is covered with the class
WithSymbol.
2011-06-08 20:22:13 -04:00
David Heinemeier Hansson
54b0aeba64 Indention style + whitespace for readability 2011-06-09 01:31:06 +02:00
David Heinemeier Hansson
93f30f0536 If you set a js_compressor, it means that you want shit compressed -- we dont need a second option to turn that on 2011-06-09 01:30:58 +02:00
Bradley Buda
dd3e7a6a34 Allow a Date to be frozen more than once without an error in Ruby 1.8.x 2011-06-08 13:54:48 -07:00
Rodrigo Navarro
8166e4b0c3 No need to create a new Arel::Table, as the arel_table method already creates a new one 2011-06-08 16:56:01 -03:00
Aaron Patterson
7d537b2106 Merge pull request #1564 from sikachu/master-changelog
Update CHANGELOG to mention the new SafeBuffer change
2011-06-08 09:01:45 -07:00
Prem Sichanugrist
7ab06004da Update CHANGELOG to mention the new SafeBuffer change 2011-06-08 11:47:27 -04:00
José Valim
8bc4771291 Merge pull request #1560 from sikachu/master-escapejs
Make escape_javascript happy to handle SafeBuffers
2011-06-08 08:08:59 -07:00
Paul Gallagher
bf2f039a93 Make escape_javascript happy to handle SafeBuffers
* see GH#1553
* allow for the fact that gsub on SafeBuffer does not pass match variables $1, $2 etc to a block
2011-06-08 11:05:15 -04:00
José Valim
b937c76f0d Merge pull request #1559 from neerajdotname/before_filter_test
Make test meaningful
2011-06-08 08:05:04 -07:00
Neeraj Singh
7fff2f9fe3 Make test meaningful
If before_filter list was being called mistakenly then
the test should fail. However test will not fail because
second filter is assigning new values to @list. To truly
test that first before_filter is not called when it should
not be called then @list should not assigned value unconditionally.

This patch will make the test fail if first filter is called.
2011-06-08 10:51:10 -04:00
José Valim
b64524d6fd Merge pull request #1556 from thoefer/master
Brought back alternative convention for namespaced models in i18n
2011-06-08 05:01:24 -07:00
thoefer
0831b7e076 Brought back alternative convention for namespaced models in i18n. 2011-06-08 13:40:17 +02:00
Xavier Noria
8c1762f11e Merge pull request #1550 from rsim/fix_test_has_many_with_pluralize_table_names_false
Do not hardcode :id as it was ignored by mass assignment filter
2011-06-08 02:13:52 -07:00
Raimonds Simanovskis
e58ee1dccc Do not hardcode :id as it was ignored by mass assignment filter
As a result test was failing on Oracle where ids are assigned by default from 1000
2011-06-08 11:56:23 +03:00
José Valim
47ac896936 Merge pull request #1549 from neerajdotname/rename_to_method_for_action
test name should reflect the name of the method
2011-06-08 00:41:55 -07:00
José Valim
2bf686f4db Merge pull request #1548 from neerajdotname/remove_class_me5
remove unused class from test
2011-06-08 00:30:11 -07:00
Neeraj Singh
8f3c9e87d9 test name should reflect the name of the method
that is actually being called
2011-06-08 03:17:27 -04:00
Neeraj Singh
cbfdd0df33 remove unused class from test 2011-06-08 02:54:19 -04:00