aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* only match the default taskAaron Patterson2013-04-031-1/+1
|
* test no longer makes sense after requiring all test filesAaron Patterson2013-04-031-23/+1
|
* switch the testing tests to use rakeAaron Patterson2013-04-031-20/+23
|
* a test file can be provided to rake, e.g.:Aaron Patterson2013-04-031-1/+10
| | | | rake test path/to/test.rb
* switch to Rails::TestTaskAaron Patterson2013-04-032-25/+34
|
* default task should also be in the test envAaron Patterson2013-04-031-1/+1
|
* ensure the schema checking is done in the dev connectionAaron Patterson2013-04-031-0/+8
|
* Revert "Update Rake tasks to call `rails test` instead"Aaron Patterson2013-04-032-41/+62
| | | | This reverts commit b51673fbd9563bd3ffa22e22255ca1cef80cfb6d.
* Revert "Warning removed unused variable task_name"Aaron Patterson2013-04-031-0/+1
| | | | This reverts commit 106e15927f0dd8060fc37eff44b823a92fa94bd2.
* apps that depend on active record should load fixturesAaron Patterson2013-04-032-10/+2
|
* Merge pull request #10049 from vipulnsward/optimize_around_merge_v1Rafael Mendonça França2013-04-031-1/+1
|\ | | | | optimize some code around merge
| * optimize some code around mergeVipul A M2013-04-031-1/+1
| |
* | Merge pull request #10074 from ↵Rafael Mendonça França2013-04-036-6/+0
|\ \ | | | | | | | | | | | | trevorturk/remove-fixme-comments-about-legacy-key-generator Remove comments about removing LegacyKeyGenerator in 4.1
| * | Remove comments about removing LegacyKeyGenerator in 4.1Trevor Turk2013-04-036-6/+0
| | |
* | | Merge pull request #9932 from senny/9913_routing_problemAndrew White2013-04-032-8/+41
|\ \ \ | | | | | | | | routing bugfixes when matching multiple paths
| * | | routing shorthand syntax works with multiple pathsYves Senn2013-03-262-5/+20
| | | | | | | | | | | | | | | | | | | | | | | | Closes #9913. We need to expand the match shorthand syntax for every path.
| * | | bugfix, when matching multiple paths with `get`, `post`, ...Yves Senn2013-03-262-3/+21
| | | | | | | | | | | | | | | | | | | | | | | | This problem was introduced with: https://github.com/rails/rails/commit/d03aa104e069be4e301efa8cefb90a2a785a7bff
* | | | Merge pull request #10070 from rubys/skip_bundle_in_railties_testsAndrew White2013-04-031-1/+4
|\ \ \ \ | | | | | | | | | | Avoid running bundler on tests that don't need it
| * | | | Avoid running bundler on tests that don't need itSam Ruby2013-04-031-1/+4
| | | | |
* | | | | Merge pull request #10072 from senny/10071_format_on_scope_block_is_ignoredAndrew White2013-04-032-1/+23
|\ \ \ \ \ | |/ / / / |/| | | | fix regression in Mapper when `format:` was used in a `scope`.
| * | | | fix regression in Mapper when `format:` was used in a `scope`.Yves Senn2013-04-032-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #10071 `#normalize_path!` depends on the options so we need to call `#normalize_options!` first to make sure everything is set correctly.
* | | | | Move alias method near to the aliased one in collection proxyCarlos Antonio da Silva2013-04-031-2/+1
|/ / / /
* | | | Merge pull request #10064 from thenickcox/fix_grammatical_errorSteve Klabnik2013-04-021-2/+2
|\ \ \ \ | | | | | | | | | | Fix grammatical error on JS helper warning message [ci skip]
| * | | | Fix grammatical error on JS helper warning messagethenickcox2013-04-021-2/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | Add changelog entry Delete changelog
* | | | Merge pull request #10063 from SamSaffron/masterSantiago Pastorino2013-04-022-15/+29
|\ \ \ \ | | | | | | | | | | Per #9999 revert blank? to empty?
| * | | | Per #9999 revert the revert changing so columns are only tested for empty? ↵Sam2013-04-032-15/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | as opposed to blank? This is both faster and more correct, added tests to make sure this is not reverted again.
* | | | | true/false => truthy falsySteve Klabnik2013-04-021-2/+2
| | | | |
* | | | | Getting rid of a few other vestiges of rails.png.Steve Klabnik2013-04-023-1/+0
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't actually need a rails.png in the AP fixtures, the tests that use it don't actually try to load the file. We also don't need to get rid of it with the dummy reset either. Finally, it's not needed in the sample application that's included with the Rails Guides.
* | | | CHANGELOG entry for rails.png removal.Steve Klabnik2013-04-021-0/+5
| | | |
* | | | We don't need to remove index.html any more.Steve Klabnik2013-04-021-1/+0
| | | | | | | | | | | | | | | | | | | | Since it's not a real file any longer, no need to worry about removing it when we reset the dummy.
* | | | .gitkeep -> .keepSteve Klabnik2013-04-021-0/+0
| |/ / |/| | | | | | | | Mentioned in a comment in e90c2d029edc by @frodsan.
* | | Merge pull request #10061 from trevorturk/dummy-key-generator-renameSantiago Pastorino2013-04-028-18/+18
|\ \ \ | | | | | | | | Rename DummyKeyGenerator -> LegacyKeyGenerator
| * | | Rename DummyKeyGenerator -> LegacyKeyGeneratorTrevor Turk2013-04-028-18/+18
| | | |
* | | | Merge pull request #10060 from trevorturk/cookies-refactoringJeremy Kemper2013-04-021-28/+25
|\ \ \ \ | | | | | | | | | | Cookies refactoring
| * | | | :scissors: spacing after privateTrevor Turk2013-04-021-3/+0
| | | | |
| * | | | Be consistent when talking about cookies, key -> nameTrevor Turk2013-04-021-25/+25
| | | | |
* | | | | Merge pull request #10059 from trevorturk/perm-cookie-jar-fixSteve Klabnik2013-04-022-2/+7
|\| | | | | |/ / / |/| | | Fix permanent cookie jar accessor typo
| * | | Fix permanent cookie jar accessor typoTrevor Turk2013-04-022-2/+7
|/ / /
* | | Fix tests from rails.png removal.Jessica Lynn Suttles2013-04-021-1/+8
| | | | | | | | | | | | | | | I'm giving @jlsuttles credit on this commit because she gave me the idea and the actual converted image in 952289aabfb.
* | | Merge pull request #10056 from steveklabnik/remove_rails_pngDavid Heinemeier Hansson2013-04-023-1/+1
|\ \ \ | | | | | | | | Remove rails.png.
| * | | adding .gigkeep to default app/assets/imagesSteve Klabnik2013-04-021-0/+0
| | | |
| * | | Remove rails.png.Steve Klabnik2013-04-022-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | No reason to have a file, we've got data-URIs now! It didn't make sense to me that you no longer have to remove public/index.html, but you still need to remove rails.png. This means that you won't have to any more when making a new Rails app.
* | | | Merge pull request #10054 from rubys/use_find_by_in_rdocSantiago Pastorino2013-04-0210-23/+23
|\ \ \ \ | | | | | | | | | | Prefer find_by over dynamic finders in rdoc
| * | | | Prefer find_by over dynamic finders in rdocSam Ruby2013-04-0210-23/+23
| |/ / /
* | | | Merge pull request #10055 from vipulnsward/remove_each_with_index_in_testSantiago Pastorino2013-04-021-1/+1
|\ \ \ \ | |/ / / |/| | | each_with_index => each as we dont use index
| * | | each_with_index => each as we dont use indexVipul A M2013-04-031-1/+1
| | | |
* | | | Merge pull request #10053 from ↵Steve Klabnik2013-04-021-6/+30
|\ \ \ \ | |/ / / |/| | | | | | | | | | | wangjohn/change_docs_for_associations_distinctness Changed the guides for associations, warning about race conditions when checking for distinctness
| * | | Changed the guides to use distinct instead of uniq for associations (seewangjohn2013-04-021-6/+30
| | | | | | | | | | | | | | | | | | | | | | | | commit a1bb6c8b which changes the naming). I've also provided an example of how distinctness when inserting into an association can be achieved.
* | | | Merge pull request #3782 from arunagw/warning_mode_onSantiago Pastorino2013-04-021-1/+1
|\ \ \ \ | | | | | | | | | | Turning on warnings in "rake test" for railties
| * | | | Turning on warnings in "rake test" for railtiesArun Agrawal2013-04-011-1/+1
| | | | |