aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | Merge pull request #16234 from ↵Yves Senn2014-07-223-5/+66
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | tgxworld/fix_template_assertion_for_integration_test Fix AC::TemplateAssertions instance variables not resetting.
| * | | | | | | Fix AC::TemplateAssertions instance variables not resetting.Guo Xiang Tan2014-07-213-5/+66
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Fixes https://github.com/rails/rails/issues/16119.
* | | | | | | Merge pull request #16256 from akshay-vishnoi/doc-changeRafael Mendonça França2014-07-221-1/+1
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | | Fixes #16255 [ci skip]
| * | | | | | Fixes #16255 [ci skip]Akshay Vishnoi2014-07-221-1/+1
|/ / / / / /
* | | | | | Merge pull request #16231 from Envek/type_in_referencesYves Senn2014-07-226-5/+56
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Allow to specify a type for foreign key column in migrations * unified the docs * some cleanup in CHANGELOG
| * | | | | | Allow to specify a type for foreign key column in migrationsAndrey Novikov2014-07-226-3/+46
|/ / / / / / | | | | | | | | | | | | | | | | | | [Andrey Novikov & Łukasz Sarnacki]
* | | | | | Merge pull request #16253 from ↵Yves Senn2014-07-221-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | zuhao/refactor_activerecord_nested_through_associations_test Make sure assertion passes for postgresql.
| * | | | | | Make sure assertion passes for postgresql.Zuhao Wan2014-07-221-1/+1
| | | | | | |
* | | | | | | Merge pull request #16254 from ↵Yves Senn2014-07-221-1/+0
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | zuhao/refactor_activerecord_attribute_decorators_test Remove redundant reset that already happened in teardown.
| * | | | | | Remove redundant that already happened in teardown.Zuhao Wan2014-07-221-1/+0
|/ / / / / /
* | | | | | build fix, remove not null constraint.Yves Senn2014-07-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fixtures are still in play, adding a new column without a default and `null: true` is not possible. This reverts back to leaking global state, as our `schema.rb` adds the `null: false` constraint on this field. A future solution would be to make the `migration_test.rb` run independent of fixture tables. This way we can simply drop the state after test execution, without worrying about side effects. /cc @zuhao
* | | | | | Merge pull request #16250 from zuhao/refactor_activerecord_migration_testYves Senn2014-07-221-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove length limit that results failure in other tests.
| * | | | | | Remove length limit that results failure in other tests.Zuhao Wan2014-07-221-1/+1
| | | | | | |
* | | | | | | docs, add example for running a single test against core adapters.Yves Senn2014-07-221-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | Merge pull request #16248 from ↵Yves Senn2014-07-221-2/+3
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | zuhao/refactor_activerecord_presence_validation_test Remove the side-effects of validates_presence_of.
| * | | | | | Remove the side-effects of validates_presence_of.Zuhao Wan2014-07-221-2/+3
|/ / / / / /
* | | | | | Merge pull request #16232 from egilburg/activesupport_coverageRafael Mendonça França2014-07-2111-4/+114
|\ \ \ \ \ \ | | | | | | | | | | | | | | Added some missing activesupport test coverage
| * | | | | | missing activesupport test coverageEugene Gilburg2014-07-1911-4/+114
| |/ / / / /
* | | | | | Merge pull request #16239 from akshay-vishnoi/remove-requireRafael Mendonça França2014-07-211-1/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove not-required file class/subclasses
| * | | | | | Remove not-required file class/subclassesAkshay Vishnoi2014-07-211-1/+0
| | | | | | |
* | | | | | | Merge pull request #16242 from vadivelan-k/content_verifierArthur Nogueira Neves2014-07-211-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Touch option description grammatical error fixed [ci skip]
| * | | | | | | Touch option description grammatical error fixed [ci skip]vadivelan2014-07-211-1/+1
|/ / / / / / /
* | | | | | | Merge pull request #16240 from siddharthbhagwan/masterEileen M. Uchitelle2014-07-211-1/+1
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Generators guide grammatical error fixed [ci skip]
| * | | | | | Generator guide grammatical error fixed [ci skip]Siddharth Bhagwan2014-07-211-1/+1
|/ / / / / /
* | | | | | Merge pull request #16236 from y-yagi/patch-1Yves Senn2014-07-211-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | [ci skip] Fix code in Routing Guide
| * | | | | [ci skip] Fix code in Routing Guideyuuji.yaginuma2014-07-211-1/+1
|/ / / / /
* | | | | Merge branch 'rm-remove-mocha'Rafael Mendonça França2014-07-1920-84/+182
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: actionpack/test/abstract_unit.rb
| * | | | | Stop requiring mocha automaticallyRafael Mendonça França2014-07-1914-5/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are planning to remove mocha from our test suite because of performance problems. To make this possible we should stop require mocha on ActionSupport::TestCase. This should not affect applications since users still need to add mocha to Gemfile and this already load mocha. Added FIXME notes to place that still need mocha removal
| * | | | | Stop using mocha on dbconsole_testRafael Mendonça França2014-07-151-44/+68
| | | | | |
| * | | | | Stop using mocha on console_testRafael Mendonça França2014-07-071-15/+40
| | | | | |
| * | | | | Set the application logger using configurationRafael Mendonça França2014-07-071-3/+5
| | | | | |
| * | | | | Stop using mocha on configuration_testRafael Mendonça França2014-07-071-7/+23
| | | | | |
| * | | | | Stop using mocha on app_rails_loader_testRafael Mendonça França2014-07-072-10/+27
| | | | | |
* | | | | | Merge pull request #16230 from egilburg/before_type_cast_coverageRafael Mendonça França2014-07-191-0/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | Missing test for form values before type cast
| * | | | | | adding missing test for text area value before type castEugene Gilburg2014-07-191-0/+7
|/ / / / / /
* | | | | | Merge pull request #16224 from egilburg/renderer_refactorsRafael Mendonça França2014-07-194-17/+32
|\ \ \ \ \ \ | | | | | | | | | | | | | | Some ActionView renderer refactors
| * | | | | | adding missing test coverageEugene Gilburg2014-07-181-0/+12
| | | | | | |
| * | | | | | marking private methods which dont work if called on their own anywaysEugene Gilburg2014-07-182-1/+5
| | | | | | |
| * | | | | | small refactors to actionview renderersEugene Gilburg2014-07-183-16/+15
| | | | | | |
* | | | | | | Merge pull request #16227 from Aguynamedsteve/typo_correctionYves Senn2014-07-191-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fixed typo in comment [ci skip]
| * | | | | | | Fixed typo in commentSteve2014-07-191-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #16219 from skanev/fix-create-join-table-with-common-prefixYves Senn2014-07-195-2/+44
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Make create_join_table use the same logic as HABTM reflections
| * | | | | | | create_join_table uses same logic as HABTM reflectionsStefan Kanev2014-07-185-2/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this change, create_join_table would not remove the common prefix in the join table name, unlike ActiveRecord::Reflections. A HABTM between Music::Artist and Music::Record would use a table music_artists_records, while create_join table would create music_artists_music_records.
* | | | | | | | Merge pull request #16098 from tgxworld/fixes_to_request_testRafael Mendonça França2014-07-181-22/+14
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fixes to request method test.
| * | | | | | | | Remove redundant test.Guo Xiang Tan2014-07-181-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we're stubbing the request, the test is actually just asserting that `@method = env['REQUEST_METHOD']`. In order to the test against the methodoverride middleware, we should test it against an actual request. However, Rack is already covering this scenario so we can remove this test.
| * | | | | | | | Fix duplicated test.Guo Xiang Tan2014-07-181-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is already another test covering Request#request_method. This test should cover Request#method.
| * | | | | | | | Update test to clearly reflect what it is testing for.Guo Xiang Tan2014-07-181-5/+7
| | | | | | | | |
| * | | | | | | | Update outdated test.Guo Xiang Tan2014-07-181-6/+3
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current test is asserting against an outdated version of Request#method where HEAD requests are treated as GET requests.
* | | | | | | | Merge pull request #16212 from ↵Rafael Mendonça França2014-07-181-1/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | aantix/additional_migration_conflict_help_messaging Additional help messaging to help the user resolve a conflicted migration
| * | | | | | | | Modified migration conflict message to remove the string concatenation.Jim Jones2014-07-181-3/+3
| | | | | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-22 07:41:46 +0000