Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | promotes change_column_null to the migrations API | Xavier Noria | 2013-03-11 | 2 | -0/+27 |
| | |||||
* | quick pass over the RDoc of schema_statements.rb [ci skip] | Xavier Noria | 2013-03-11 | 1 | -122/+190 |
| | |||||
* | documents how to DROP DEFAULT [ci skip] | Xavier Noria | 2013-03-11 | 1 | -3/+7 |
| | |||||
* | Merge pull request #9668 from neerajdotname/9567-delete-all-warning | Xavier Noria | 2013-03-11 | 1 | -0/+9 |
|\ | | | | | Show warning message if delete_all is firing callbacks | ||||
| * | Show warning message if delete_all is firing callbacks | Neeraj Singh | 2013-03-11 | 1 | -0/+9 |
| | | | | | | | | | | | | `post.comments.delete_all` will fire callbacks if :dependent option is :destroy . It will be fixed in Rails 4.1 . In the meantime display a warning . Look at #9567 for details . | ||||
* | | Replace assert_(no_)match args from regexps to strings to remove warnings | Carlos Antonio da Silva | 2013-03-11 | 1 | -40/+40 |
| | | | | | | | | | | | | Using regexps as arguments without parentheses end up raising: warning: ambiguous first argument; put parentheses or even spaces | ||||
* | | Skip fetching path if the iteration is going to be skipped | Carlos Antonio da Silva | 2013-03-11 | 1 | -2/+2 |
| | | |||||
* | | There's no need to install test group in travis | Santiago Pastorino | 2013-03-11 | 2 | -2/+2 |
| | | |||||
* | | Move benchmark-ips to test group | Santiago Pastorino | 2013-03-11 | 1 | -2/+2 |
| | | |||||
* | | Use platforms instead of conditionals in Gemfile | Santiago Pastorino | 2013-03-11 | 1 | -4/+7 |
| | | |||||
* | | Merge pull request #9666 from sikachu/master-fix-test_test_failure | Rafael Mendonça França | 2013-03-11 | 1 | -1/+17 |
|\ \ | | | | | | | Make sure that the test case is run under test env | ||||
| * | | Make sure that the test case is run under test env | Prem Sichanugrist | 2013-03-11 | 1 | -1/+17 |
| | | | | | | | | | | | | | | | | | | * Unset $RAILS_ENV that got set by abstract_unit to trigger the default. * split out environment setting since Ruby 1.9.3 doesn't support inline ENV setting. | ||||
* | | | Mention that debugging can be slow. | Steve Klabnik | 2013-03-11 | 1 | -0/+2 |
|/ / | | | | | | | | | | | https://github.com/rails/rails/pull/3180\#issuecomment-14705821 has mention of rendering going from 1-2ms to 4 seconds with this on, which seems reasonable: debugging is slow. | ||||
* | | default the reaping frequency to 10 seconds | Aaron Patterson | 2013-03-11 | 1 | -1/+1 |
| | | |||||
* | | debugger does not work on trunk | Aaron Patterson | 2013-03-11 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #9664 from sikachu/master-fix-test_test_failure | Andrew White | 2013-03-11 | 1 | -5/+3 |
|\ | | | | | Fix test failure introduced in 3ed41e57 | ||||
| * | Fix test failure introduced in 3ed41e57 | Prem Sichanugrist | 2013-03-11 | 1 | -5/+3 |
|/ | | | | I forgot to run the test suit after changing the task name. :bomb: | ||||
* | Merge pull request #9080 from sikachu/master-rails-test | Rafael Mendonça França | 2013-03-11 | 8 | -132/+591 |
|\ | | | | | Add `rails test` command to run the test suite | ||||
| * | Make sure that `rails test` load test in test env | Prem Sichanugrist | 2013-03-09 | 4 | -7/+55 |
| | | |||||
| * | Update Rake tasks to call `rails test` instead | Prem Sichanugrist | 2013-03-09 | 2 | -62/+41 |
| | | | | | | | | | | Also, print out deprecation warning for other rake tasks except `rake test` and `rake` (default) | ||||
| * | Improve wording for rails test command | Dalibor Nasevic | 2013-03-09 | 5 | -39/+41 |
| | | |||||
| * | Load fixtures only when running suites, or `-f` | Prem Sichanugrist | 2013-03-09 | 4 | -3/+80 |
| | | | | | | | | | | | | * `rails test -f` will run the test suites with all fixtures loaded * New application will now generated without `fixtures :all` line enabled by default. | ||||
| * | Add support for MiniTest flags in TestRunner | Prem Sichanugrist | 2013-03-09 | 4 | -59/+93 |
| | | | | | | | | | | Any flags that got set will be passed through to MiniTest::Unit.runner, such as `-n`, `-s-, and `-v`. | ||||
| * | Update testing documentation | Prem Sichanugrist | 2013-03-09 | 1 | -63/+71 |
| | | | | | | | | | | | | | | * Update test invocation to use `rails test` instead. * Update all the test command previews (since we're now using MiniTest.) * Mentioning MiniTest instead of Test::Unit. * Update list of test suites. | ||||
| * | Add `rails test` command to run the test suite | Prem Sichanugrist and Chris Toomey | 2013-03-09 | 4 | -0/+311 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To run the whole test suite: $ rails test To run the test file(s): $ rails test test/unit/foo_test.rb [test/unit/bar_test.rb ...] To run the test suite $ rails test [models,helpers,units,controllers,mailers,...] For more information, see `rails test --help`. This command will eventually replacing `rake test:*`, and `rake test` command will actually invoking `rails test` instead. | ||||
* | | Merge pull request #9657 from vipulnsward/cleanup_tests | Rafael Mendonça França | 2013-03-11 | 3 | -4/+2 |
|\ \ | | | | | | | Cleanup tests for unused variables | ||||
| * | | Cleanup tests for unused variables | Vipul A M | 2013-03-11 | 3 | -4/+2 |
| | | | |||||
* | | | Merge pull request #9658 from ↵ | Rafael Mendonça França | 2013-03-11 | 1 | -0/+1 |
|\ \ \ | |/ / |/| | | | | | | | | tkhr/add-assert-to-test_company_descends_from_active_record add an missing assertion to inheritance_test.rb | ||||
| * | | add an missing assertion to inheritance_test.rb | Takehiro Adachi | 2013-03-11 | 1 | -0/+1 |
|/ / | | | | | | | | | | | assertion for https://github.com/rails/rails/blob/ad624345e54bd20802de67b2b5c9ef29ecf5 d5f4/activerecord/lib/active_record/inheritance.rb#L32 | ||||
* | | Merge pull request #9653 from jetthoughts/master | Carlos Antonio da Silva | 2013-03-10 | 1 | -5/+5 |
|\ \ | | | | | | | | | | | | | | | | | | | Fixed tests `to_time` returning an instance of `Time` in the local system timezone. Tests for to_time returning an instance of Time in the local system timezone did not take in account DST when they used mixed timezone declarations like 'US/Eastern' and '-0500'. Replaced timezone with DST by timezone without DST. | ||||
| * | | Fixed tests for returning an instance of `Time` in the local system timezone. | Paul Nikitochkin | 2013-03-10 | 1 | -5/+5 |
| | | | |||||
* | | | Merge pull request #9651 from vipulnsward/change_to_each_value_intest | Carlos Antonio da Silva | 2013-03-10 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Change from each to each_value on hash to avoid unused variable warning | ||||
| * | | | Change from each to each_value on hash to avoid unused variable warning | Vipul A M | 2013-03-11 | 1 | -1/+1 |
| | | | | |||||
* | | | | Merge pull request #9650 from yahonda/group_by_oracle | Rafael Mendonça França | 2013-03-10 | 1 | -2/+2 |
|\ \ \ \ | |_|/ / |/| | | | Address ActiveRecord::StatementInvalid: OCIError: ORA-00979 | ||||
| * | | | Address ActiveRecord::StatementInvalid: OCIError: ORA-00979 | Yasuo Honda | 2013-03-11 | 1 | -2/+2 |
| |/ / | |||||
* | | | Merge pull request #9645 from senny/9643_habtm_wrong_table_name | Carlos Antonio da Silva | 2013-03-10 | 1 | -0/+2 |
|\ \ \ | |/ / |/| | | document habtm join-table name with custom prefix [ci skip] | ||||
| * | | document habtm join-table name with custom prefix [ci skip]. | Yves Senn | 2013-03-10 | 1 | -0/+2 |
|/ / | | | | | | | | | | | Closes #9643. The logic was introduced with 46492949 | ||||
* | | Merge pull request #9638 from siong1987/debugger | Carlos Antonio da Silva | 2013-03-09 | 1 | -1/+1 |
|\ \ | | | | | | | Make sure that debugger isn't included in production environment. | ||||
| * | | Make sure that debugger isn't included in production environment. | Teng Siong Ong | 2013-03-09 | 1 | -1/+1 |
|/ / | |||||
* | | Change uniq validation with conditions examples to use where with hashes | Carlos Antonio da Silva | 2013-03-09 | 2 | -5/+5 |
| | | | | | | | | Take the opportunity to showcase where.not. | ||||
* | | Merge pull request #9637 from GSI/master | Carlos Antonio da Silva | 2013-03-09 | 1 | -0/+1 |
|\ \ | | | | | | | Include Montevideo time zone in list of aliases. Closes #9628 | ||||
| * | | included "Montevideo" in list of time zone aliases (closes Issue #9628) | Gernot | 2013-03-09 | 1 | -0/+1 |
| | | | |||||
* | | | Merge pull request #9633 from senny/5321_make_it_lazy | Carlos Antonio da Silva | 2013-03-09 | 3 | -4/+23 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | Uniqueness validation uses a proc to specify the `:conditions` option. This is a follow up to #5321 and follows the general direction in AR to make things lazy evaluated. | ||||
| * | | | Uniqueness validation uses a proc to specify the `:conditions` option. | Yves Senn | 2013-03-09 | 3 | -4/+23 |
| | |/ | |/| | | | | | | | | | | This is a follow up to #5321 and follows the general direction in AR to make things lazy evaluated. | ||||
* | | | Merge pull request #9618 from manvsmachine/fix-postgres-enable-extension-error | Carlos Antonio da Silva | 2013-03-09 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | Bugfix for enable_extension in postgresql_adapter. The use of quotations is required to install extensions with certain characters in them (e.g. uuid-ossp). | ||||
| * | | | fixes enable_extension bug in postgresql_adapter | Darren Woodley | 2013-03-08 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The use of quotations is required to install extensions with certain characters in them (e.g. uuid-ossp). removes CHANGELOG entry Deemed unnecessary. | ||||
* | | | | Fix "ActiveRecord" => "Active Record" in changelog [ci skip] | Carlos Antonio da Silva | 2013-03-09 | 1 | -4/+3 |
| |/ / |/| | | |||||
* | | | Merge pull request #9371 from benmoss/access-connection-via-class | Jon Leighton | 2013-03-09 | 7 | -13/+19 |
|\ \ \ | | | | | | | | | Access an instance's connection via its class, rather than via #connection | ||||
| * | | | Deprecate #connection in favour of accessing it via the class | Ben Moss | 2013-03-09 | 7 | -13/+19 |
| | | | | | | | | | | | | | | | | | | | | This allows end-users to have a `connection` method on their models without clashing with ActiveRecord internals. | ||||
* | | | | Merge pull request #9626 from dasch/dasch/instrument-strong-params | José Valim | 2013-03-09 | 2 | -1/+7 |
|\ \ \ \ | | | | | | | | | | | Use AS::Notifications to instrument Strong Params |