aboutsummaryrefslogtreecommitdiffstats
path: root/railties/test
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #6690 from suginoy/fix-templates-copyRafael Mendonça França2012-06-111-0/+11
|\ | | | | Fix: 'rake rails:templates:copy' doesn't work
| * Added a testcase for #6690.kennyj2012-06-121-0/+11
| |
* | Use . instead of :: for class methods, add CHANGELOG entriesJosé Valim2012-06-101-1/+1
| |
* | Merge pull request #6665 from schneems/schneems/raise-migration-errorJosé Valim2012-06-101-0/+15
|\ \ | |/ |/| Notify A User they Have Pending Migrations
| * test errors for pending migrationsschneems2012-06-091-0/+15
| | | | | | App should raise error on page_load
* | Fix tests, active_authorizer shouldn't include blank valuesPiotr Sarnacki2012-06-091-1/+1
| | | | | | | | Fix for a test failing after: 00ff0a6776
* | Merge pull request #6681 from arunagw/plugin_gen_fixRafael Mendonça França2012-06-081-2/+8
|\ \ | | | | | | Plugin gen fix
| * | We should not include engine.rake file into rakeArun Agrawal2012-06-081-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if we are passing -T which is skip_test_unit See issue #6673 for more details. I saw that we are not creating dummy app even if we do skip_test_unit. Fixes #6673
| * | Removed protected as we already doing it above.Arun Agrawal2012-06-081-2/+0
| | |
* | | Don't add sqlite3 to gemspec with -O on rails plugin newPiotr Sarnacki2012-06-081-0/+8
|/ / | | | | | | (closes #6672)
* | stop `to_s`ing method namesAkira Matsuda2012-06-061-3/+1
| | | | | | | | Module#methods are Symbols in Ruby >= 1.9
* | Fixed the application_controller require_dependency path generated by the ↵Fred Wu2012-06-061-1/+3
| | | | | | | | app generator
* | Merge pull request #6627 from Vanuan/tests_add_precompile_failure_assertJosé Valim2012-06-051-17/+24
|\ \ | | | | | | bundle exec rake assets:precompile shouldn't fail quietly.
| * | bundle exec rake assets:precompile shouldn't fail quietly.John Yani2012-06-051-17/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | If JavaScript runtime is not installed, execjs fails with error quietly, while tests continue to run. This should not happen since it causes tests to fail for unknown reason (#6621). This commit assures that if JavaScript runtime is not installed, an assertion is raised.
* | | Fixed the Gemfile when gemspec is skipped in the 'rails plugin new' commandFred Wu2012-06-051-0/+26
|/ /
* | Set RACK_ENV to nil in the dbconsole testRafael Mendonça França2012-05-301-0/+1
| | | | | | | | This will fix the travis-ci build
* | Fix app_generator_test with the changes introduced atRafael Mendonça França2012-05-301-1/+1
| | | | | | | | f4d7af67ffc90f2542afa50c7579fc83ea4f45f2
* | Remove support for rails server RAILS_ENV=env-nameSam Oliver2012-05-301-2/+2
| |
* | Fix various bugs with console arguments.Sam Oliver2012-05-302-39/+71
|/ | | | Allow hyphens in environment names again.
* Unused variable warning removed.Arun Agrawal2012-05-301-1/+1
|
* More assert_match warnings fixed.Arun Agrawal2012-05-304-12/+12
|
* Warnings removed for "assert_match /".Arun Agrawal2012-05-301-20/+20
|
* Add support runner hook.kennyj2012-05-292-0/+26
|
* Add a testcase for #5847.kennyj2012-05-291-0/+23
|
* Fix railties_order when application object is passedPiotr Sarnacki2012-05-271-0/+4
| | | | | | | | | | | | | | | railites_order method, introduced in 40b19e0, had a bug that was causing loading application instance twice in initializers if railties_order already included application instance. So for example railties_order = [Foo::Engine, :main_app, Bar::Engine] would result in such railties array: [MyApp::Application, Foo::Engine, MyAppApplication, Bar::Engine] In order to fix it, we need to check for existence of application in both railties_order and railties arrays.
* Fixes build 4008.1Waseem Ahmad2012-05-251-2/+2
| | | | | | | | This fixex build 4008.1[1] because of the changes made in 4001835db00ce44cb75bca33ec02cd76b8ccc790 [1] http://travis-ci.org/#!/rails/rails/jobs/1429671
* remove unnecessary test from route_inspect_testschneems2012-05-241-9/+0
|
* Rails::InfoController tests passingschneems2012-05-243-6/+32
| | | | This includes new tests for /rails/info/routes
* Remove unnecessary comment.Uģis Ozols2012-05-231-1/+0
|
* Use new hash syntax for generators gem methodAlexey Vakhov2012-05-222-2/+10
| | | | | | The Gemfile of new application uses ruby 1.9 hashes. Gem method of generators should use them too. It prevents from mixing two kinds of syntax in one file.
* Use require_dependency in generated controllersPiotr Sarnacki2012-05-221-2/+2
| | | | | | | Using require in development mode will prevent required files from reloading, even if they're changed. In order to keep namespaced application_controller reloadable, we need to use require_dependency instead of require.
* Fix `rails db -h` and cosmetic fixes in usage bannersAlexey Vakhov2012-05-221-0/+18
| | | | | | | Ruby tries to use '-h' as short version of '--header' by default https://github.com/ruby/ruby/blob/trunk/lib/optparse.rb#L1372-1381. To force `rails db -h` prints an usage message we should add the `-h` options explicitly.
* Use relative path to sqlite3 db in `rails db` commandAlexey Vakhov2012-05-221-6/+11
| | | | | | | | | | Rails uses sqlit3 db file with a path relative to the rails root. It allows to execute server not from rails root only. For example you can fire `./spec/dummy/script/rails s` to start dummy application server if you develop some engine gem. Now the `rails db` command uses relative paths also and you can explore your dummy db via `./spec/dummy/script/rails db` command.
* using __method__ for the command method callsAaron Patterson2012-05-211-0/+59
|
* Fix generators to help with ambiguous `ApplicationController` issuePiotr Sarnacki2012-05-201-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In development mode, dependencies are loaded dynamically at runtime, using `const_missing`. Because of that, when one of the constants is already loaded and `const_missing` is not triggered, user can end up with unexpected results. Given such file in an Engine: ```ruby module Blog class PostsController < ApplicationController end end ``` If you load it first, before loading any application files, it will correctly load `Blog::ApplicationController`, because second line will hit `const_missing`. However if you load `ApplicationController` first, the constant will be loaded already, `const_missing` hook will not be fired and in result `PostsController` will inherit from `ApplicationController` instead of `Blog::ApplicationController`. Since it can't be fixed in `AS::Dependencies`, the easiest fix is to just explicitly load application controller. closes #6413
* Merge pull request #3359 from mrreynolds/datamapper_naming_fixRafael Mendonça França2012-05-192-11/+11
|\ | | | | Fixed DataMapper namings in symbols and constants.
| * Fixed DataMapper namings in symbols and constants.Robert Glaser2011-10-182-11/+11
| |
* | Remove warning in namespaced generator testCarlos Antonio da Silva2012-05-191-1/+1
| |
* | Fix railties testsPiotr Sarnacki2012-05-191-1/+1
| | | | | | | | | | Minitest uses different signature for assert_no_match, so we have to swap arguments.
* | Improve `rake routes` output for redirects - closes #6369.Andrew White2012-05-191-4/+7
| |
* | Fix inspecting route redirections, closes #6369Łukasz Strzałkowski2012-05-181-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | This commit fixes route inspection in `rake routes` Before: foo /foo(.:format) :controller#:action" After: foo /foo(.:format) Redirect (301)
* | More DRY route inspect testsŁukasz Strzałkowski2012-05-181-26/+18
| |
* | Revert changes related to `bundle install` fixes in `rails new`Piotr Sarnacki2012-05-171-7/+2
| | | | | | | | | | | | | | | | | | | | | | Reverted changes: f3482a9 Fix tests in railties 5904295 improve #6318 aed906a prevent using already loaded Gemfile for 'bundle install' In order to fix this, we need a fix in bundler related to GEM_PATH, which will allow to run tests properly. I will get this changes back when it happens.
* | Load all the env files available in config.paths (closes #6335)Piotr Sarnacki2012-05-161-1/+9
| |
* | Fix tests in railtiesPiotr Sarnacki2012-05-151-1/+1
| | | | | | | | | | | | | | | | Since `bundle install` was fixed in `rails plugin new`, it now requires `rails 4.0.0.beta` version in filesystem when running tests. Instead of providing it, we can run tested command with `--dev` option, to use rails from the local directory.
* | improve #6318Michal Papis2012-05-151-1/+6
| |
* | Remove not used date variable and another test warningCarlos Antonio da Silva2012-05-141-2/+1
| |
* | Remove assertion on date from humans.txt testsJosé Valim2012-05-141-1/+1
| |
* | Refactor App Generator TestOscar Del Ben2012-05-131-22/+20
| |
* | Remove useless load path modificationsSantiago Pastorino2012-05-111-3/+2
| |