aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #21105 from os97673/allow-minitest-pluginsRafael Mendonça França2015-08-031-0/+1
|\ | | | | Allow Minitest to load plugins. Fixes #21102
| * Allow Minitest to load plugins. Fixes #21102Oleg Sukhodolsky2015-08-031-0/+1
| |
* | Inform user to add script in correct location into the application.js ↵Mehmet Emin İNAÇ2015-08-032-2/+2
| | | | | | | | | | | | | | | | manifest file [ci skip] use it change the warning
* | Revert "test runner should crash with non existing file argument."Yves Senn2015-08-032-6/+1
| | | | | | | | | | | | | | | | | | | | | | This reverts commit 465f0fbca3d4a1c269038b84ec9cc248fdab5fab. This breaks some cases where non file / directory arguments are passed to the runner (for example db:migrate). I still think that we can get this to work. From what I can tell there is no reason why db:migrate is passed along to `Minitest.run`. I'll revert and investigate possible solutions.
* | Merge pull request #21104 from atul-shimpi/masterYves Senn2015-08-032-2/+4
|\ \ | | | | | | | | | | | | | | | Inform user to add styles in correct location in application.css [ci skip]
| * | Inform user to add styles in correct location in application.cssatul-shimpi2015-08-032-2/+4
| |/
* / test runner should crash with non existing file argument.Yves Senn2015-08-032-1/+6
|/ | | | | | | | Before this patch, using `bin/rails test` with a non existing file or directory argument would silently swallow the argument and run the whole test suite. After the patch the command fails with `cannot load such file --`.
* Merge pull request #20992 from JuanitoFatas/fix/bin-setup-scriptYves Senn2015-07-311-1/+1
|\ | | | | Don't fail when checking dependencies in bin/setup script
| * Don't fail when checking dependencies in bin/setup scriptJuanito Fatas2015-07-231-1/+1
| |
* | Fix rake/notes_testschneems2015-07-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | Presumably due to https://github.com/rails/sprockets-rails/pull/265 sprockets was trying to load the "scss" gem but it isn't in the gemfile: ``` ApplicationTests::RakeTests::RakeNotesTest#test_register_a_new_extension: LoadError: cannot load such file -- sass ``` If we use an empty precompile list, it won't try to load sass.
* | Fix reported regression rails/sprockets-rails#265schneems2015-07-291-0/+1
| | | | | | | | We can prevent the UglifierCompressor from being instantiated prematurely by setting precompile to an empty array in this test.
* | Add `:nodoc:` for internal testing methods [ci skip]Robin Dupret2015-07-281-2/+2
| |
* | Merge pull request #20759 from vngrs/fix_wrong_docRafael Mendonça França2015-07-271-2/+2
|\ \ | | | | | | Change wrong usage in documentation about autoload_paths [ci skip]
| * | Change wrong usage in documentation about autoload_paths [ci skip]Mehmet Emin İNAÇ2015-07-021-2/+2
| | | | | | | | | | | | We can't use this configuration outside of the application.rb
* | | Merge pull request #21003 from robin850/api-pluginsRafael Mendonça França2015-07-275-9/+84
|\ \ \ | | | | | | | | Allow generating plugins for API applications
| * | | Enable the `api_only` option for API plugins' generatorsRobin Dupret2015-07-232-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This way, running a generator inside the plugin's directory, files that are not relevant won't be generated (e.g. views or assets). This won't interfere with the application's generators configuration.
| * | | Generate an API dummy application for API pluginsRobin Dupret2015-07-232-0/+9
| | | |
| * | | Make ApplicationController inherit from AC::API for API pluginsRobin Dupret2015-07-232-1/+9
| | | |
| * | | Skip assets, helpers and views generation for API pluginsRobin Dupret2015-07-232-8/+27
| | | |
| * | | Add the `--api` option for the plugin generatorRobin Dupret2015-07-232-0/+12
| | |/ | |/|
* | | Merge pull request #20928 from matthewd/unload-interlockMatthew Draper2015-07-241-1/+1
|\ \ \ | |/ / |/| | We need stricter locking before we can unload
| * | We need stricter locking before we can unloadMatthew Draper2015-07-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Specifically, the "loose upgrades" behaviour that allows us to obtain an exclusive right to load things while other requests are in progress (but waiting on the exclusive lock for themselves) prevents us from treating load & unload interchangeably: new things appearing is fine, but they do *not* expect previously-present constants to vanish. We can still use loose upgrades for unloading -- once someone has decided to unload, they don't really care if someone else gets there first -- it just needs to be tracked separately.
* | | Merge pull request #20926 from rsanheim/fail-fast-bin-setupRafael Mendonça França2015-07-211-5/+9
|\ \ \ | | | | | | | | add system! to fail fast in bin/setup
| * | | add system! to fail fast in bin/setupRob Sanheim2015-07-181-5/+9
| |/ /
* | | Merge pull request #20819 from y-yagi/rails_api_gemfileRafael Mendonça França2015-07-212-1/+4
|\ \ \ | | | | | | | | remove `web-console` from API app generated Gemfile
| * | | remove `web-console` from API app generated Gemfileyuuji.yaginuma2015-07-092-1/+4
| | | |
* | | | titleizing the New LinkThiago Pinto2015-07-201-1/+1
| | | |
* | | | Merge pull request #20384 from kaspth/per-request-cacheDavid Heinemeier Hansson2015-07-201-0/+63
|\ \ \ \ | |_|/ / |/| | | Use digest cache in development.
| * | | Make digest cache work in development.Kasper Timm Hansen2015-07-181-0/+63
| | | | | | | | | | | | | | | | | | | | | | | | Avoid computing the same fragment digest many times when looping over templates. The cache is cleared on every request so template changes are still picked up.
* | | | [skip ci] Lookup can be a noun but it is not a verbJon Atack2015-07-171-3/+3
| | | | | | | | | | | | | | | | Various grammar corrections and wrap to 80 characters.
* | | | make test runner work correctly inside engineyuuji.yaginuma2015-07-124-7/+13
| | | |
* | | | remove Rack::Lock for webrickAaron Patterson2015-07-101-11/+1
|/ / / | | | | | | | | | constant loading should be thread safe now, so lets remove this
* | | Rely on the load interlock for non-caching reloads, tooMatthew Draper2015-07-093-24/+18
| | |
* | | Soften the lock requirements when eager_load is disabledMatthew Draper2015-07-092-13/+50
|/ / | | | | | | | | We don't need to fully disable concurrent requests: just ensure that loads are performed in isolation.
* | Merge pull request #20767 from y-yagi/remove_unnecessary_fixture_methodYves Senn2015-07-061-4/+0
|\ \ | | | | | | remove unnecessary loading fixtures from `navigation_test.rb`
| * | remove unnecessary loading fixtures from `navigation_test.rb`yuuji.yaginuma2015-07-031-4/+0
| |/ | | | | | | | | it has been corrected in 0176aef1ebaa9f69001c7045a51727a8ea9b61b8 as for loading of fixtures in `test_helper.rb`, loading fixtures for each test is unnecessary.
* / Display a more human readable list of reserved namesRobin Dupret2015-07-014-4/+8
|/ | | | | Interpolating an array inside a String will call #inspect on it. Let's call #join to display a more human-readable error message.
* Avoid crashing when minitest-rails is loaded.Victor Costan2015-07-011-3/+3
| | | | | | | | The improvments to the test runner's integration with minitest in commit b6fc8e25a10cc4abdd03018798b180270d6c5d7f add methods to the Minitest module that refer to the Rails module. Unfortunately, when the minitest-rails gem is loaded, the reference is incorrectly resolved to the Minitest::Rails module.
* `dump_schema_after_migration` applies migration tasks other than db:migrateYves Senn2015-06-301-6/+8
| | | | | | | | | | | | | Closes #20743. The task `db:_dump` now only dumps the schema if `ActiveRecord::Base.dump_schema_after_migration` is true. This has effects: - `db:migrate:up` - `db:migrate:down` - `db:forward` - `db:rollback`
* tests, railties tests should use `bin/` executables when possible.Yves Senn2015-06-309-121/+121
| | | | | We recommend using the `bin/` executables in our docs and guides. Let's make sure that our tests execute the same code path.
* Merge pull request #20724 from y-yagi/fix_scaffold_in_full_engineYves Senn2015-06-297-3/+36
|\ | | | | | | fix NoMethodError that occurs when generating scaffold inside full mode engine
| * fix NoMethodError that occurs when generating scaffold inside full mode engineyuuji.yaginuma2015-06-287-3/+36
| |
* | Merge pull request #20712 from nhodges/dev/nh/20345Rafael Mendonça França2015-06-271-2/+1
|\ \ | |/ |/| Remove `config` options from `default_options` in `railties`
| * Remove `config` options from `default_options` in `railties`Nuri Hodges2015-06-261-2/+1
| | | | | | | | Fixes #20345.
* | Merge pull request #20714 from y-yagi/fix_file_fixture_pathRafael Mendonça França2015-06-271-1/+1
|\ \ | |/ |/| correct `file_fixture_path` in plugins `test_helper.rb`
| * correct `file_fixture_path` in plugins `test_helper.rb`yuuji.yaginuma2015-06-271-1/+1
| | | | | | | | `fixture_path` does not end with a slash
* | cherry-pick test and source doc from #20050Hirofumi Wakasugi2015-06-252-0/+12
| |
* | modify add_source in application template to take a block for gem entriesHirofumi Wakasugi2015-06-251-2/+16
|/
* do not show "Failed tests" message when a failed test is notyuuji.yaginuma2015-06-232-3/+10
|
* remove unused variable in reporter testyuuji.yaginuma2015-06-221-1/+0
|