aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* scaffold controller_test template should use kwargs. refs #18323.Yves Senn2015-01-301-5/+5
| | | | This prevents a flood of warnings when generating a new scaffold.
* tests, use `capture` instead of custom `redirect_stderr` helper.Yves Senn2015-01-301-12/+2
|
* Merge pull request #18736 from nonsensery/autoload-guide-grammar-fixYves Senn2015-01-301-1/+1
|\ | | | | Fix grammar in Qualified Constants section [ci skip]
| * Fix grammar in Qualified Constants sectionAlex Johnson2015-01-291-1/+1
| | | | | | | | | | New wording is based on the "cref" sentence earlier in the section. [ci skip]
* | Merge pull request #18734 from andreynering/docs-dumpingRafael Mendonça França2015-01-291-5/+10
|\ \ | | | | | | Improvements about schema dumping [ci skip]
| * | Improvements about schema dumping [ci skip]Andrey Nering2015-01-291-5/+10
| | |
* | | Merge pull request #18477 from Alamoz/scaffold_css_templateRafael Mendonça França2015-01-291-0/+11
|\ \ \ | | | | | | | | Add a margin to body and padding to th, td in scaffold.css.
| * | | Add a margin to body and padding to th, td.Adrien Lamothe2015-01-291-0/+11
|/ / / | | | | | | | | | Add space to new css defs.
* | | Merge pull request #18732 from BMorearty/document-bundle-updateRafael Mendonça França2015-01-291-0/+8
|\ \ \ | | | | | | | | Doc: run bundle update after cloning Rails.
| * | | Doc: run bundle update after cloning Rails.Brian Morearty2015-01-291-0/+8
| |/ /
* | | Remove duplicated testsRafael Mendonça França2015-01-291-30/+0
| | |
* | | Merge pull request #18705 from mrgilman/skip-testRafael Mendonça França2015-01-2911-22/+26
|\ \ \ | |_|/ |/| | Update option to skip test in generators
| * | Change option description and update test namesMelanie Gilman2015-01-293-3/+3
| | |
| * | Update option to skip test in generatorsMelanie Gilman2015-01-2711-20/+24
| | | | | | | | | | | | Rails no longer generates Test::Unit files by default.
* | | Post.all.or(anything) == Post.allSean Griffin2015-01-293-7/+7
| |/ |/|
* | Merge pull request #18722 from bogdan/group-by-key-wordRafael Mendonça França2015-01-293-13/+27
|\ \ | | | | | | Fixed AR::Relation#group method when argument is a SQL reserved keyword
| * | Fixed AR::Relation#group method when argument is a SQL reserved keywordBogdan Gusiev2015-01-293-13/+27
| | |
* | | Consistent usage of spaces in hashes across our codebaseRafael Mendonça França2015-01-2925-113/+117
| | |
* | | Use fixed fonts only in the name of the parameterRafael Mendonça França2015-01-293-14/+14
| | |
* | | Improve CHANGELOGRafael Mendonça França2015-01-291-6/+5
| | |
* | | Merge pull request #18323 from kirs/controller-test-kwargsRafael Mendonça França2015-01-2945-417/+938
|\ \ \ | |/ / |/| | | | | Use kwargs in ActionController::TestCase and ActionDispatch::Integration HTTP methods
| * | Switch to kwargs in ActionController::TestCase and ActionDispatch::IntegrationKir Shatrov2015-01-2945-417/+938
|/ / | | | | | | | | | | | | | | Non-kwargs requests are deprecated now. Guides are updated as well. `post url, nil, nil, { a: 'b' }` doesn't make sense. `post url, params: { y: x }, session: { a: 'b' }` would be an explicit way to do the same
* | Merge pull request #18627 from tekin/dont-blat-default-formatAndrew White2015-01-293-3/+33
|\ \ | | | | | | Preserve default format when generating URLs
| * | Preserve default url options when generating URLsTekin Suleyman2015-01-283-3/+33
| | | | | | | | | | | | | | | Fixes an issue that would cause default_url_options to be lost when generating URLs with fewer positional arguments than parameters in the route definition.
* | | Merge pull request #18719 from ↵Yves Senn2015-01-291-0/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5t111111/add-fix-for-loading-fixtures-in-engine-tests Add fix for loading fixtures in engine tests (additional fix for #4971) Conflicts: railties/lib/rails/generators/rails/plugin/templates/test/test_helper.rb
| * | | Add fix for loading fixtures in engine testsWAKASUGI 5T1111112015-01-291-0/+1
| | | |
* | | | reassign `file_fixture_path` in plugins `test_helper.rb`. refs #18658.Yves Senn2015-01-291-0/+1
| | | |
* | | | let dependencies use Module#const_defined?Xavier Noria2015-01-281-2/+2
|/ / / | | | | | | | | | | | | | | | | | | Module#const_defined? accepts constant paths in modern Ruby, we no longer need our qualified_* extensions. References #17845.
* | | Don't error when grouped calculations return 0 recordsSean Griffin2015-01-282-1/+6
| | | | | | | | | | | | Fixes #18717
* | | Merge Pull Request #16052 Added #or to ActiveRecord::RelationSean Griffin2015-01-288-1/+166
|\ \ \
| * | | Bring the implementation of Relation#or up to speedSean Griffin2015-01-285-44/+48
| | | |
| * | | Added #or to ActiveRecord::RelationMatthew Draper2015-01-286-1/+162
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Post.where('id = 1').or(Post.where('id = 2')) # => SELECT * FROM posts WHERE (id = 1) OR (id = 2) [Matthew Draper & Gael Muller]
* | | | Always convert strings to UTF-8, regardless of column type in SQLiteSean Griffin2015-01-282-12/+7
| |/ / |/| | | | | | | | | | | | | | | | | | | | All columns which would map to a string primitive need this behavior. Binary has it's own marker type, so it won't go through this conversion. String and text, which need this, will. Fixes #18585.
* | | Merge branch 'rm-deprecate_via_redirect'Rafael Mendonça França2015-01-283-5/+31
|\ \ \
| * | | Just assert the deprecation of one methodRafael Mendonça França2015-01-281-10/+15
| | | |
| * | | Add CHANGELOG entryRafael Mendonça França2015-01-281-0/+6
| | | |
| * | | Merge pull request #18693 from aditya-kapoor/deprecate_via_redirectRafael Mendonça França2015-01-282-15/+30
|/| | | | | | | | | | | | | | | Deprecate *_via_redirect integration test methods
| * | | Deprecate *_via_redirect integration test methodsAditya Kapoor2015-01-282-15/+30
| | | |
* | | | Merge pull request #18716 from tonytonyjan/typoRafael Mendonça França2015-01-281-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | table name should be plural
| * | | | table name should be plural [ci skip]Jian Weihang2015-01-291-1/+1
|/ / / /
* | | | Merge pull request #18654 from liseki/fix-aj-test-helperRafael Mendonça França2015-01-282-2/+51
|\ \ \ \ | |_|/ / |/| | | Fix ActiveJob assertions with a GlobalID object argument
| * | | Fix ActiveJob assertions with a GlobalID object argumentRichard Manyanza2015-01-232-2/+51
| | | |
* | | | Don't redefine autosave association callbacks in nested attrsSean Griffin2015-01-286-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These callbacks will already have been defined when the association was built. The check against `reflection.autosave` happens at call time, not at define time, so simply modifying the reflection is sufficient. Fixes #18704
* | | | Merge pull request #18658 from senny/file_fixturesYves Senn2015-01-288-0/+76
|\ \ \ \ | | | | | | | | | | Add `ActiveSupport::Testing::FileFixtures`.
| * | | | introduce `ActiveSupport::Testing::FileFixtures`.Yves Senn2015-01-288-0/+76
|/ / / / | | | | | | | | | | | | | | | | | | | | It's a thin layer to provide easy access to sample files throughout test-cases. This adds the directory `test/fixtures/files` to newly generated applications.
* | | | Merge pull request #18709 from ianks/atomic-writeYves Senn2015-01-283-1/+18
|\ \ \ \ | | | | | | | | | | | | | | | Return value of yielded block in File.atomic_write
| * | | | Return value of yielded block in File.atomic_writeIan Ker-Seymer2015-01-282-1/+14
|/ / / / | | | | | | | | | | | | | | | | | | | | Staying true to Ruby convention, we now return the value of the yielded block from `File.atomic_write {...}`. This mimics the behavior of MRI's `File.open {...}`.
* | | | Merge pull request #18666 from cheunghy/auth_check_nilGuillermo Iguaran2015-01-282-2/+9
|\ \ \ \ | | | | | | | | | | Fixed undefined method error when doing http basic authentication.
| * | | | Fixed undefined method error when doing authentication.Zhang Kai Yu2015-01-241-2/+2
| | | | |
| * | | | Add test for HTTP basic authentication when no credential is given.Zhang Kai Yu2015-01-241-0/+7
| | | | |