aboutsummaryrefslogtreecommitdiffstats
path: root/railties/test/application
Commit message (Collapse)AuthorAgeFilesLines
...
| * Merge pull request #27089 from erickueen/erickueen_fix_26606Rafael Mendonça França2016-11-181-3/+3
| |\ | | | | | | | | | Fix incorrect output from rails routes when using singular resources …
| | * Fix incorrect output from rails routes when using singular resources issue ↵Erick Reyna2016-11-182-3/+36
| |/ | | | | | | | | | | | | | | | | | | #26606 Rails routes (even rake routes in previous versions) output showed incorrect routes when an application use resource :controller, implying that edit_controller_path match with controller#show. The order of the output has changed to correct this. View #26606 for more information. Added a test case, change unit test in rake to expect the new output. Since the output of resource :controller is changing, the string spected of the railties/test/application/rake_test.rb test_rails_routes_with_controller_environment had to be modified.
| * Add more rubocop rules about whitespacesRafael Mendonça França2016-10-295-14/+14
| |
| * Prevent the test framework from being loaded in production modeAaron Patterson2016-10-211-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | The test framework should not be autoloaded in production mode. Before this commit, the testing railtie would extend AS::TestCase. This caused AS::TestCase to be preloaded regardless of the environment in which we were running. This commit just moves the code that adds line filtering support in to the test command where we actually execute the test runner. That allows us to maintain the line runner feature but only load the minimal amount of code we need.
* | clear all environments log files by defaultyuuji.yaginuma2016-10-141-0/+33
|/ | | | | | | In #22703, `log:clear` task has been changed to clear only standard environment log files. However, it is often to add a non-standard environment(e.g. "staging"). Therefore, I think than it is better to clear all environments log files by default.
* Remove deprecated code in ssl middlewareRafael Mendonça França2016-10-101-2/+2
|
* Remove deprecated support to :text in renderRafael Mendonça França2016-10-107-46/+46
|
* Initial command structure.Kasper Timm Hansen2016-09-251-1/+1
|
* Merge pull request #26515 from grosser/grosser/after_runArthur Nogueira Neves2016-09-231-1/+33
|\ | | | | support minitest after_run
| * support minitest after_run for parity to regular minitestMichael Grosser2016-09-161-0/+18
| | | | | | | | see https://github.com/seattlerb/minitest/blob/f9605387e4af7d657921a83aaf0ae364f6d26a57/lib/minitest.rb#L51-L65
| * improve test coverageMichael Grosser2016-09-161-2/+16
| |
* | use rails command in restart task testyuuji.yaginuma2016-09-191-7/+7
| |
* | improve error message when include assertions failMichael Grosser2016-09-168-30/+30
|/ | | | | | assert [1, 3].includes?(2) fails with unhelpful "Asserting failed" message assert_includes [1, 3], 2 fails with "Expected [1, 3] to include 2" which makes it easier to debug and more obvious what went wrong
* use correct Gemfile in `bin/setup` testyuuji.yaginuma2016-09-111-0/+18
| | | | | | | Currently, `bin/setup` test uses Gemfile of Rails. But this Gemfile is not a file to be used in Rails application. Add a Gemfile to Rails application that is created for test, it has been modified to use the Gemfile.
* Fix styleschneems2016-08-301-26/+26
|
* Address comment via @dhh, better option namingschneems2016-08-301-4/+4
|
* Thanks rubocopschneems2016-08-291-1/+1
|
* Favor `public_folder: true` over `public_*`schneems2016-08-291-25/+44
| | | | Adding all those `public_*` methods is a bit heavy handed, we can change the API to instead use `public_folder: true`. Change was pretty easy since it was already implemented that way.
* Fix formattingschneems2016-08-291-3/+3
|
* Add `public_*` helpers to all the `_tag` methods.schneems2016-08-291-9/+15
|
* Test `public_` methods.schneems2016-08-291-0/+72
|
* Add three new rubocop rulesRafael Mendonça França2016-08-169-28/+28
| | | | | | | | Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository.
* applies project convention for string literalsXavier Noria2016-08-071-1/+1
|
* Merge pull request #24260 from ↵Kasper Timm Hansen2016-08-071-4/+6
|\ | | | | | | | | y-yagi/show_error_message_when_error_raised_in_rails_runner show error message when error raised in rails runner
| * show error message when error raised in rails runneryuuji.yaginuma2016-08-031-4/+6
| |
* | Add `Style/EmptyLines` in `.rubocop.yml` and remove extra empty linesRyuta Kamizono2016-08-071-1/+0
| |
* | applies remaining conventions across the projectXavier Noria2016-08-061-1/+1
| |
* | normalizes indentation and whitespace across the projectXavier Noria2016-08-065-68/+68
| |
* | remove redundant curlies from hash argumentsXavier Noria2016-08-063-6/+6
| |
* | modernizes hash syntax in railtiesXavier Noria2016-08-063-6/+6
| |
* | applies new string literal convention in railties/testXavier Noria2016-08-0639-858/+858
|/ | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
* There are some cases where @@app is not definedSantiago Pastorino2016-07-261-0/+28
|
* Merge pull request #25438 from prathamesh-sonpatki/rm-session-store-initializerRafael Mendonça França2016-07-201-0/+16
|\ | | | | | | Setup default session store internally, no longer through an initializer
| * Setup default session store internally, no longer through an application ↵Prathamesh Sonpatki2016-07-171-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | initializer - By default the session store will be set to cookie store with application name as session key. - Older apps are not affected as they will have the session store initializer generated by Rails in older versions, and Rails will not overwrite the session store if it is already set or disabled. - But new apps will not have the initializer, instead the session store will be set to cookie store by default. - Based on comment by DHH here - https://github.com/rails/rails/issues/25181#issuecomment-222312764.
* | Fix typo in mailer previews test description [ci skip]Gale Shafer2016-07-141-1/+1
| | | | | | | | | | A mailer preview test description misspelled the word configuration. This commit updates the test description to spell the word correctly.
* | rails -> Rails [ci skip]Santosh Wadghule2016-07-121-1/+1
| |
* | remove unnecessary sendyuuji.yaginuma2016-07-101-5/+5
|/ | | | | `class_variable_get` is public from Ruby 1.9. Ref: http://ruby-doc.org/core-2.2.0/Module.html#method-i-class_variable_get
* Merge pull request #25652 from prathamesh-sonpatki/rm-boot-railsGuillermo Iguaran2016-07-0834-39/+3
|\ | | | | Remove unused boot_rails method and it's usage
| * Remove unused boot_rails method and it's usagePrathamesh Sonpatki2016-07-0434-39/+3
| | | | | | | | | | - The `boot_rails` method from abstract_unit.rb is empty after 2abcdfd978fdcd491576a237e8c6b. - So let's remove it and its usage.
* | Merge pull request #25730 from prathamesh-sonpatki/rm-cookie-onlyRafael França2016-07-071-0/+6
|\ \ | | | | | | No need to set `cookie_only` option from Rails
| * | Add a test case for verifying `cookie_only` is set even if user tries to set ↵Prathamesh Sonpatki2016-07-071-0/+6
| | | | | | | | | | | | it false
* | | Fix typo in test caseAbhishek Jain2016-07-081-1/+1
|/ /
* | Merge pull request #25704 from ↵Rafael França2016-07-051-0/+7
|\ \ | | | | | | | | | | | | fledman/add_respond_to_missing_to_rails_application_configuration_custom implement respond_to_missing? to match method_missing
| * | implement respond_to_missing? to match method_missingDavid Feldman2016-07-051-0/+7
| |/
* | Fix test to try `spec/` directory as opposed to `app/spec` which I hadJohn Meehan2016-07-051-2/+2
| | | | | | | | | | | | accidently set it to. Made the change and test still passes.
* | removed extra blank lineJohn Meehan2016-07-051-1/+0
| |
* | Allow rake notes to work with other directories.John Meehan2016-07-051-0/+13
|/ | | | | | | Additional directories can be added using SourceAnnotationExtractor::Annotation.register_directories("spec", "other_dir") Result: rake notes will now extract notes from these directories.
* Use sass-rails in our test suiteRafael Mendonça França2016-06-301-1/+1
|
* Fix rails/info routes for apps with globbing routeNicholas Firth-McCoy2016-06-201-0/+19
| | | | | | | | The /rails/info routes were inaccessible in apps with a catch-all globbing route, as they were being appended after the globbing route and would never be matched. See also ccc3ddb7762bae0df7e2f8d643b19b6a4769d5be.
* Fix assertion count after 69e1bb52.Kasper Timm Hansen2016-05-281-2/+2
| | | | | | | | | | | | | | | When changing the generated integration tests to assert redirects with an absolute path this redirection check in `assert_redirected_to` against the absolute response location would now pass: https://github.com/rails/rails/blob/af245aaf3a1c447752a1b5895adddc66e5f73c00/actionpack/lib/action_dispatch/testing/assertions/response.rb#L55 Thus we'd break early instead of hitting `assert_operator` and have 2 fewer assertions per `assert_redirected_to` as `assert_operator` is composed of 2 assertions internally: https://github.com/seattlerb/minitest/blob/4e146b1515b19a5c474e39c174b037510f6dbc6e/lib/minitest/assertions.rb#L254-L258 Deduct 2 assertions for 2 redirects taking the expected count down by 4 in total for the two failing tests.