aboutsummaryrefslogtreecommitdiffstats
path: root/railties/test
Commit message (Collapse)AuthorAgeFilesLines
...
* | | respect `pluralize_table_names` when generate fixture file. fixes #19519yuuji.yaginuma2015-03-271-0/+10
| | |
* | | Revert "Merge pull request #19404 from dmathieu/remove_rack_env"Jeremy Kemper2015-03-206-19/+74
| | | | | | | | | | | | | | | | | | | | | Preserving RACK_ENV behavior. This reverts commit 7bdc7635b885e473f6a577264fd8efad1c02174f, reversing changes made to 45786be516e13d55a1fca9a4abaddd5781209103.
* | | Remove `#build_original_fullpath` methodeileencodes2015-03-201-27/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removing `#build_original_fullpath` because it is no longer used by the Rails' source code or called in the `#call` method becasue the previous commit now uses `fullpath` from Rack. The method was nodoc'ed so it's safe to remove. NOTE: I did this as a separate commit so if this does cause a problem with engines etc reverting is easy.
* | | Merge pull request #19404 from dmathieu/remove_rack_envAaron Patterson2015-03-196-74/+19
|\ \ \ | | | | | | | | Don't fallback to RACK_ENV when RAILS_ENV is not present
| * | | don't fallback to RACK_ENV when RAILS_ENV is not presentDamien Mathieu2015-03-196-74/+19
| | | |
* | | | Use absolute path on find_method location for the runnerArthur Neves2015-03-191-4/+5
| | | |
* | | | method_source needs to be a hard dependency on railtiesArthur Neves2015-03-191-7/+0
| | | |
* | | | Merge pull request #19216 from senny/bin_test_runnerYves Senn2015-03-194-41/+203
|\ \ \ \ | | | | | | | | | | `bin/rails test` runner (rerun snippets, run tests by line, option documentation)
| * | | | run `test_test.rb` using the `bin/rails test`.Yves Senn2015-03-181-17/+1
| | | | |
| * | | | Fix reporter test and verbose modeArthur Neves2015-03-181-9/+7
| | | | |
| * | | | move `ENV["BACKTRACE"]` support into the TestRunner.Yves Senn2015-03-181-0/+10
| | | | |
| * | | | `-p`, `--pattern` to run tests using a pattern.Yves Senn2015-03-181-2/+1
| | | | |
| * | | | use `bin/rails t` runner in `test_runner_test.rb`.Yves Senn2015-03-182-28/+21
| | | | |
| * | | | Fix relative dir call on test runnerArthur Neves2015-03-181-10/+14
| | | | |
| * | | | Add tests for runner#test_files methodArthur Neves2015-03-181-0/+14
| | | | |
| * | | | Run multiple files on runnerArthur Neves2015-03-181-2/+16
| | | | |
| * | | | `-e` / `--environment` for the test runner.Yves Senn2015-03-181-0/+14
| | | | |
| * | | | get rid of NAMED_PATTERNS in favor of running a whole directoryYves Senn2015-03-181-0/+8
| | | | |
| * | | | add tests for our minitest reporter.Yves Senn2015-03-181-0/+76
| | | | |
| * | | | Show the right file when test raisesArthur Neves2015-03-181-0/+1
| | | | |
| * | | | Better file:line parsing and filter method by file tooArthur Neves2015-03-181-0/+26
| | | | |
| * | | | spike of a `OptionParser` backed test runner.Yves Senn2015-03-181-0/+21
| |/ / /
* / / / Add a new-line to the end of route method generated code.Arthur Neves2015-03-181-0/+24
|/ / / | | | | | | | | | | | | | | | | | | | | | Fix regression on route method that was added by bac812a7ef2660a2fe2ab00822e5e66228379822. The regression was that when calling the `route` method, we were not appending a \n anymore. [fixes #19316]
* / / Fix deprecation warning in rails_info_controller testsSemyon Pupkov2015-03-171-8/+8
|/ / | | | | | | Use keyword arguments in request methods
* | add mailer suffix to comments for email previewsyuuji.yaginuma2015-03-151-5/+5
| |
* | Merge pull request #19221 from matthewd/random-testsMatthew Draper2015-03-061-5/+0
|\ \ | | | | | | Run all our tests in random order
| * | Revert "Leave all our tests as order_dependent! for now"Matthew Draper2015-03-061-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 2f52f969885b2834198de0045748436a4651a94e. Conflicts: actionmailer/test/abstract_unit.rb actionview/test/abstract_unit.rb activemodel/test/cases/helper.rb activerecord/test/cases/helper.rb activesupport/test/abstract_unit.rb railties/test/abstract_unit.rb
* | | pass a config to the route setAaron Patterson2015-03-051-0/+13
|/ / | | | | | | | | This way we can get the relative_url_root from the application without setting another global value
* | Created rake restart task.Hyonjee Joo2015-02-261-0/+31
| | | | | | | | | | | | Fixes #18876. Rake restart touches `tmp/restart.txt` to restart application on next request. Updated tests and documentation accordingly.
* | Change filter on /rails/info/routes to use an actual path regexp from railsbrainopia2015-02-231-0/+25
| | | | | | | | | | | | | | | | Change filter on /rails/info/routes to use an actual path regexp from rails and not approximate javascript version. Oniguruma supports much more extensive list of features than javascript regexp engine. Fixes #18402.
* | Require `belongs_to` by default.Josef Šimánek2015-02-212-1/+37
| | | | | | | | Deprecate `required` option in favor of `optional` for belongs_to.
* | Merge pull request #18994 from tgxworld/run_delete_middleware_operations_lastGuillermo Iguaran2015-02-191-0/+16
|\ \ | | | | | | Allow Rack::Runtime to be deleted from middleware stack.
| * | Allow Rack::Runtime to be deleted from middleware stack.Guo Xiang Tan2015-02-191-0/+16
| | | | | | | | | | | | Fixes: https://github.com/rails/rails/issues/16433.
* | | Remove unused gsub_app_file method from Bukkit.Josef Šimánek2015-02-191-6/+0
| | |
* | | Merge pull request #18393 from y-yagi/fix_mailerRafael Mendonça França2015-02-182-16/+25
|\ \ \ | |/ / |/| | follow up to #18074
| * | follow up to #18074yuuji.yaginuma2015-01-082-16/+25
| | | | | | | | | | | | | | | * ignore mailer suffix when generate test files * add mailer suffix to view files
* | | Dont check bundler output on plugin generator testArthur Neves2015-02-101-4/+2
| | |
* | | Actually fix the buildSean Griffin2015-02-101-1/+1
| | | | | | | | | | | | Like for real this time. I checked.
* | | Fix the buildSean Griffin2015-02-101-1/+1
| | |
* | | tests, reset global to previous value.Yves Senn2015-02-091-1/+3
| | | | | | | | | | | | | | | The default of the global might change. It's better to reset it to what it was than a hardcoded value.
* | | Remove warning from generator named base testYuki Nishijima2015-02-081-10/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the following warning: railties/test/generators/named_base_test.rb:9: warning: method redefined; discarding old pluralize_table_names activesupport/lib/active_support/core_ext/class/attribute.rb:86: warning: previous definition of pluralize_table_names was here railties/test/generators/named_base_test.rb:9: warning: method redefined; discarding old pluralize_table_names= activesupport/lib/active_support/core_ext/class/attribute.rb:83: warning: previous definition of pluralize_table_names= was here
* | | README.rdoc -> README.md for newly generated applicationsXavier Noria2015-02-062-3/+3
| | | | | | | | | | | | | | | | | | README.rdoc was generated to support the doc:app task. Now that this task is gone we can switch to Markdown, which is nowadays a better default.
* | | Remove documentation tasksXavier Noria2015-02-061-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch removes the tasks doc:app, doc:rails, and doc:guides. In our experience applications do not generate APIs using doc:app. Methods may be certainly documented for maintainers, annotated with YARD tags, etc. but that is intended to be read with the source code, not in a separate website. Then, teams also have typically selected topics written down in Markdown files, or in a GitHub wiki... that kind of thing. If a team absolutely needs to generate application documentation for internal purposes, they can still easily write their own task. Regarding doc:rails and doc:guides, we live in 2015. We are used to go to online docs all the time. If you really want access to the API offline RubyGems generates it for every Rails component unless you tell it not to, and you can checkout the Rails source code to read the guides as Markdown, or download them for a Kindle reader. All in all, maintaining this code does not seem to be worthwhile anymore. As a consequence of this, guides (+3 MB uncompressed) won't be distributed with the rails gem anymore. Of course, guides and API are going to be still part of releases, since documentation is maintained alongside code and tests. Also, time permitting, this will allow us to experiment with novel ways to generate documentation in the Rails docs server, since right now we were constrained by being able to generate them in the user's environment.
* | | Merge pull request #18399 from kommen/unify-structure-file-envvar-namesRafael Mendonça França2015-02-051-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Use SCHEMA instead of DB_STRUCTURE for specifiying structure file. Conflicts: activerecord/CHANGELOG.md
| * | | Use `SCHEMA` instead of `DB_STRUCTURE` for specifiying structure file.Dieter Komendera2015-01-081-1/+1
| |/ / | | | | | | | | | | | | `rake test:load_structure` already uses `SCHEMA` and there's no need to maintain two different env vars.
* | | Merge pull request #18526 from vipulnsward/add-silence-streamRafael Mendonça França2015-02-053-71/+8
|\ \ \ | | | | | | | | Extracted silence_stream method to new module in activesupport/testing
| * | | - Extracted silence_stream method to new module in activesupport/testing.Vipul A M2015-01-203-71/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added include for the same in ActiveSupport::Test. - Removed occurrences of silence_stream being used elsewhere. - Reordered activesupport testcase requires alphabetically. - Removed require of silence stream from test_case - Moved quietly method to stream helper - Moved capture output to stream helper module and setup requires for the same elsewhere
* | | | Force generated route to be insertedAndrew White2015-02-051-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Thor isn't very discerning over whether some content is present when passed to `inject_into_file`, e.g. a commented out route is detected as being present. So to prevent people scratching their heads as to why a route hasn't appeared it's better to fall on the side of having duplicate routes.
* | | | Don't remove all line endings from routes.rbAndrew White2015-02-031-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When there is a single scaffold in the routes.rb with no other lines then revoking/destroying it will create a routes.rb file with a syntax error. This is because the sentinel for the Thor `route` action didn't include the newline but the logged route code did. The fix is to add the newline to the sentinel and remove it from the the logged route code. Fixes #15913.
* | | | Removed magic comments # encoding: utf-8 , since its default from ruby 2.0 ↵Vipul A M2015-02-035-5/+0
| | | | | | | | | | | | | | | | onwards.