aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Move benchmark-ips to test groupSantiago Pastorino2013-03-111-2/+2
| | |
* | | Use platforms instead of conditionals in GemfileSantiago Pastorino2013-03-111-4/+7
| | |
* | | Merge pull request #9666 from sikachu/master-fix-test_test_failureRafael Mendonça França2013-03-111-1/+17
|\ \ \ | | | | | | | | Make sure that the test case is run under test env
| * | | Make sure that the test case is run under test envPrem Sichanugrist2013-03-111-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 Klabnik2013-03-111-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 secondsAaron Patterson2013-03-111-1/+1
| | |
* | | debugger does not work on trunkAaron Patterson2013-03-111-1/+1
|/ /
* | Merge pull request #9664 from sikachu/master-fix-test_test_failureAndrew White2013-03-111-5/+3
|\ \ | | | | | | Fix test failure introduced in 3ed41e57
| * | Fix test failure introduced in 3ed41e57Prem Sichanugrist2013-03-111-5/+3
|/ / | | | | | | I forgot to run the test suit after changing the task name. :bomb:
* | Merge pull request #9080 from sikachu/master-rails-testRafael Mendonça França2013-03-118-132/+591
|\ \ | | | | | | Add `rails test` command to run the test suite
| * | Make sure that `rails test` load test in test envPrem Sichanugrist2013-03-094-7/+55
| | |
| * | Update Rake tasks to call `rails test` insteadPrem Sichanugrist2013-03-092-62/+41
| | | | | | | | | | | | | | | Also, print out deprecation warning for other rake tasks except `rake test` and `rake` (default)
| * | Improve wording for rails test commandDalibor Nasevic2013-03-095-39/+41
| | |
| * | Load fixtures only when running suites, or `-f`Prem Sichanugrist2013-03-094-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 TestRunnerPrem Sichanugrist2013-03-094-59/+93
| | | | | | | | | | | | | | | Any flags that got set will be passed through to MiniTest::Unit.runner, such as `-n`, `-s-, and `-v`.
| * | Update testing documentationPrem Sichanugrist2013-03-091-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 suitePrem Sichanugrist and Chris Toomey2013-03-094-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_testsRafael Mendonça França2013-03-113-4/+2
|\ \ \ | | | | | | | | Cleanup tests for unused variables
| * | | Cleanup tests for unused variablesVipul A M2013-03-113-4/+2
| | |/ | |/|
* | | Merge pull request #9658 from ↵Rafael Mendonça França2013-03-111-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.rbTakehiro Adachi2013-03-111-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/masterCarlos Antonio da Silva2013-03-101-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 Nikitochkin2013-03-101-5/+5
| | |
* | | Merge pull request #9651 from vipulnsward/change_to_each_value_intestCarlos Antonio da Silva2013-03-101-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 warningVipul A M2013-03-111-1/+1
| | | |
* | | | Merge pull request #9650 from yahonda/group_by_oracleRafael Mendonça França2013-03-101-2/+2
|\ \ \ \ | |_|/ / |/| | | Address ActiveRecord::StatementInvalid: OCIError: ORA-00979
| * | | Address ActiveRecord::StatementInvalid: OCIError: ORA-00979Yasuo Honda2013-03-111-2/+2
| |/ /
* | | Merge pull request #9645 from senny/9643_habtm_wrong_table_nameCarlos Antonio da Silva2013-03-101-0/+2
|\ \ \ | |/ / |/| | document habtm join-table name with custom prefix [ci skip]
| * | document habtm join-table name with custom prefix [ci skip].Yves Senn2013-03-101-0/+2
|/ / | | | | | | | | | | Closes #9643. The logic was introduced with 46492949
* | Merge pull request #9638 from siong1987/debuggerCarlos Antonio da Silva2013-03-091-1/+1
|\ \ | | | | | | Make sure that debugger isn't included in production environment.
| * | Make sure that debugger isn't included in production environment.Teng Siong Ong2013-03-091-1/+1
|/ /
* | Change uniq validation with conditions examples to use where with hashesCarlos Antonio da Silva2013-03-092-5/+5
| | | | | | | | Take the opportunity to showcase where.not.
* | Merge pull request #9637 from GSI/masterCarlos Antonio da Silva2013-03-091-0/+1
|\ \ | | | | | | Include Montevideo time zone in list of aliases. Closes #9628
| * | included "Montevideo" in list of time zone aliases (closes Issue #9628)Gernot2013-03-091-0/+1
| | |
* | | Merge pull request #9633 from senny/5321_make_it_lazyCarlos Antonio da Silva2013-03-093-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 Senn2013-03-093-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-errorCarlos Antonio da Silva2013-03-091-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_adapterDarren Woodley2013-03-081-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 Silva2013-03-091-4/+3
| |/ / |/| |
* | | Merge pull request #9371 from benmoss/access-connection-via-classJon Leighton2013-03-097-13/+19
|\ \ \ | | | | | | | | Access an instance's connection via its class, rather than via #connection
| * | | Deprecate #connection in favour of accessing it via the classBen Moss2013-03-097-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-paramsJosé Valim2013-03-092-1/+7
|\ \ \ \ | | | | | | | | | | Use AS::Notifications to instrument Strong Params
| * | | | Use the instrumentation framework to instrument Strong ParamsDaniel Schierbeck2013-03-072-1/+7
| | | | |
* | | | | Fix race condition in testJon Leighton2013-03-091-2/+2
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should fix travis. For real this time! This is the one! The readpartial(100) meant that an earlier assert_stdout could chomp up the output that a later assert_stdout wants, meaning that the later assertion fails. Reading only 1 byte at a time ensure that we don't read any more than is necessary to verify the assertion.
* | | | Merge pull request #9632 from senny/unify_the_changelogsCarlos Antonio da Silva2013-03-094-36/+37
|\ \ \ \ | | | | | | | | | | Use "Fixes" in place of "Fix" in changelogs for consistency [ci skip]
| * | | | Use "Fixes" in place of "Fix" in changelogs for consistency [ci skip].Yves Senn2013-03-094-36/+37
|/ / / /
* | | | Merge pull request #9630 from robertjwhitney/update_collection_docsCarlos Antonio da Silva2013-03-091-2/+2
|\ \ \ \ | | | | | | | | | | Update documentation for collection<< to reflect behavior for new parent objects.
| * | | | Update documentation for collection<< to reflect << on new records.Robert Whitney2013-03-091-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | collection<< does not automatically fire update sql unless the parent object is not a new record (and thus has a primary key). eg: p = Post.new p.comments << Comment.new does not call update sql until: p.save at which point it saves the parent object and then the collection object. changes that to the
* | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-03-109-16/+92
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | copy edits [ci skip]Vijay Dev2013-03-102-4/+4
| | | | |