aboutsummaryrefslogtreecommitdiffstats
path: root/railties/test/generators/app_generator_test.rb
Commit message (Collapse)AuthorAgeFilesLines
...
| * Enforce frozen string in RubocopKir Shatrov2017-07-011-0/+1
| |
* | Add system tests to generate scaffoldDinah Shi2017-06-301-2/+2
|/
* Replace therubyracer with mini_racerSam2017-05-301-1/+1
|
* Remove redundant test methodAndy Atkinson2017-05-251-7/+1
|
* Use recyclable cache keys (#29092)David Heinemeier Hansson2017-05-181-3/+3
|
* Should escape meta characters in regexpRyuta Kamizono2017-05-071-7/+7
|
* Merge pull request #28708 from y-yagi/dont_generate_system_test_filesRafael França2017-04-191-0/+11
|\ | | | | Don't generate system test files if `skip_system_test` option is specified
| * Don't generate system test files if `skip_system_test` option is specifiedyuuji.yaginuma2017-04-121-0/+11
| |
* | Add `form_with_generates_remote_forms` config.Kasper Timm Hansen2017-04-161-1/+0
|/ | | | | | | Allows users to not have remote forms by default, since there's more JS harness, e.g. bundling rails-ujs, otherwise. Also don't skip creating defaults file anymore. Sprockets isn't the only new config.
* Merge pull request #28546 from claudiob/drop-j-optionRafael Mendonça França2017-03-271-8/+0
|\ | | | | | | Remove -j (--javascript) option from `rails new`
| * Remove -j (--javascript) option from `rails new`claudiob2017-03-231-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | The "-j" option was added 5 years ago (https://github.com/rails/rails/commit/d9c39c3a) when we wanted to support prototype-rails and jquery-rails. Prototype is not as popular and jQuery is not a requirement anymore. Still the "-j" option can be used to install *any* gem that ends in "-rails". This "might" open security issues and does not bring great benefits anymore. If you know which "-rails"-ending gem you want to install, you can manually add it to the Gemfile just like any other gem.
* | ignore system test gems on Gemfile when execute with --skip-test optionota42y2017-03-271-0/+24
|/
* Don't comment out config.file_watcher during Rails upgradeyuuji.yaginuma2017-03-221-14/+0
| | | | | | | This is necessary only when updating to Rails 5.0, it is not necessary for updating to 5.1. Related #24243
* Correctly check whether key is defined in configurationyuuji.yaginuma2017-03-191-0/+12
| | | | | | | | | | | | | It can not check correctly with `defined?` ```ruby irb(main):001:0> Rails.application.config.active_record => {:maintain_test_schema=>true, :belongs_to_required_by_default=>true} irb(main):002:0> defined?(Rails.application.config.active_record) => nil ``` Follow up to #28469
* Track the version-compatible config settings inside railtiesMatthew Draper2017-03-181-13/+8
| | | | | | | | | | | | | | | | Instead of forcing new applications to carry an initializer that just switches things to what their default "should" be, we can handle it internally. The initializer is then only used by upgraders: it shows what the new default would be (commented out), while their upgraded application continues to operate as it did before. Under this model, a multiply-upgraded application could accumulate several new_framework_defaults_*.rb files, for each release series it has traversed. A given release series only needs to generate the latest, though, because we don't support `rails app:upgrade` while skipping releases.
* Rename test of `app:update` to more appropriate nameyuuji.yaginuma2017-03-151-7/+7
| | | | | `rails:update` was changed to `app:update` in 6fb31638c8b61731103d4963272755b217a2df87. Therefore, I think that it is better that the test name is also `app_update`.
* Revert "Revert "Add encrypted secrets""Kasper Timm Hansen2017-02-231-0/+1
|
* Revert "Add encrypted secrets" (#28127)David Heinemeier Hansson2017-02-231-1/+0
|
* Add encrypted secrets (#28038)Kasper Timm Hansen2017-02-231-0/+1
|
* Revert back to the original of using package.json in the root of the project ↵David Heinemeier Hansson2017-02-211-5/+5
| | | | (#28093)
* Merge pull request #28055 from robertomiranda/patch-3Guillermo Iguaran2017-02-211-1/+1
|\ | | | | Use Puma 3.7
| * Use Puma 3.7.xRoberto Miranda2017-02-201-1/+1
| | | | | | | | ref this commit seems that has not been merged into 3.7 https://github.com/puma/puma/commit/42bec4600c51ab8a1c1ee5a0e1b738a4ffd82bf2
* | Add tests for system testingeileencodes2017-02-201-1/+20
|/ | | | | | | | | | * Adds test case test * Adds driver adapter test * Adds tests for capybara seleium driver (testing the settings not actually opening the browser to test capybara w/ selenium because that would so so so slow) * Adds tests for rack test driver * Adds tests for generators
* remove `ActiveSupport.halt_callback_chains_on_return_false` from everywhereyuuji.yaginuma2017-02-081-1/+0
|
* Ignore yarn error log fileyuuji.yaginuma2017-02-031-0/+5
|
* do not run `git init` when specify the `pretend` optionyuuji.yaginuma2017-01-231-0/+1
| | | | | When specifying the `pretend` option, expect that no processing will be done, so do not execute `git init` as well.
* Initalize git repo when creatin new rails appdixpac2017-01-121-2/+7
| | | | | * If `--skip-git` is not specified initalize git repo when creating new rails app
* Revert "Merge pull request #27619 from dixpac/add_missing_generator_test"yuuji.yaginuma2017-01-101-5/+0
| | | | | | | | | | This reverts commit 8cb3cdffdb70f3575518d24ea96ec891e40d21d0, reversing changes made to 3bc747bd8676dc940b531067e2861dcd4ac28efc. Reason: This test already exists in `SharedGeneratorTests`. Ref: https://github.com/rails/rails/blob/master/railties/test/generators/shared_generator_tests.rb#L109..L112 https://github.com/rails/rails/blob/master/railties/test/generators/app_generator_test.rb#L55..L56
* Test `--skip-git` generator optiondixpac2017-01-091-0/+5
|
* Remove bin/yarn if yarn is skipped, tidy up testsPrathamesh Sonpatki2017-01-011-0/+7
| | | | | | - No need to remove bin/yarn separately for API only apps because :skip_yarn is set to true for API only apps. - Added a test for :skip_yarn config.
* Privatize unneededly protected methods in Railties testsAkira Matsuda2016-12-241-1/+1
|
* fix broken tests (#27308)Yuji Yaginuma2016-12-081-3/+2
| | | Follow up to 0d20530e5edfd7d00fbc2a38ef5f87eca6ccc924
* Make Yarn the default, drop default vendor/asset directories (#27300)David Heinemeier Hansson2016-12-081-24/+11
|
* Yarn: Move node_modules, package.json, and yarn.lock file to vendor (#27245)David Heinemeier Hansson2016-12-011-1/+1
| | | Move node_modules, package.json, and yarn.lock file to vendor
* Merge pull request #26836 from Liceth/npmGuillermo Iguaran2016-11-291-0/+24
|\ | | | | Add Yarn support in new apps using --yarn option
| * Add node_modules path to assets load paths when --yarn option is usedLiceth Ovalles2016-11-281-0/+1
| |
| * Add package.json for Yarn if --yarn option is addedLiceth Ovalles2016-11-281-0/+23
| |
* | rails-ujs is now shipped with Action ViewGuillermo Iguaran2016-11-291-2/+0
|/
* Don't add jQuery by default in new apps and use rails-ujs as UJS adapter insteadGuillermo Iguaran2016-11-211-12/+8
|
* Add test for #25248Sean Griffin2016-11-121-0/+10
|
* Don't put db:migrate and db:setup in binfiles if activerecord is excludedBouke van der Bijl2016-11-011-0/+6
|
* remove assets config from `new_framework_defaults` if `--skip-sprockets` is trueyuuji.yaginuma2016-10-251-0/+3
| | | | If `sprockets` is not loaded, `Rails.application.config.assets` is not defined.
* don't create db directory when skip active_recordJosé Gomes Júnior2016-10-211-0/+1
|
* Fix test for the edge generatorRafael Mendonça França2016-09-141-1/+1
|
* Merge pull request #26157 from ↵Rafael França2016-08-161-0/+1
|\ | | | | | | | | y-yagi/remove_test_mailers_when_skipping_action-mailer remove "test/mailers" directory when skipping action-mailer
| * remove "test/mailers" directory when skipping action-maileryuuji.yaginuma2016-08-141-0/+1
| | | | | | | | Related to #26146
* | Add three new rubocop rulesRafael Mendonça França2016-08-161-1/+1
|/ | | | | | | | Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository.
* Ensure app/mailers is removed when skipping action-mailerBrian Cardarella2016-08-121-0/+1
| | | | Closes #26145
* normalizes indentation and whitespace across the projectXavier Noria2016-08-061-1/+1
|
* applies new string literal convention in railties/testXavier Noria2016-08-061-49/+49
| | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.