aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge pull request #20326 from hderms/dh/fix_task_bugRafael Mendonça França2015-05-272-16/+66
|\ \ | | | | | | | | | Fix rake method definition leaking onto Object
| * | add fixed fileDermot Haughey2015-05-272-16/+66
| | | | | | | | | | | | | | | | | | add framework_test add another test
* | | :bomb: Fix another leftoverRafael Mendonça França2015-05-271-1/+1
| | |
* | | Fix file nameRafael Mendonça França2015-05-271-1/+1
| | | | | | | | | | | | This is a leftover of #19867
* | | Merge pull request #19867 from radar/rename-app-rails-loaderRafael Mendonça França2015-05-273-11/+11
|\ \ \ | |/ / |/| | Remove redundant 'Rails' from Rails::AppRailsLoader constant
| * | Remove redundant 'Rails' from Rails::AppRailsLoader constantRyan Bigg2015-05-213-11/+11
| | |
* | | Remove web-console and spring from test group on default Gemfile.Kassio Borges2015-05-271-2/+4
| | |
* | | Merge pull request #20262 from arunagw/aa-remove-broken-unused-release-taskYves Senn2015-05-271-9/+0
|\ \ \ | | | | | | | | Remove broken and unused release task
| * | | Remove broken and unused release taskArun Agrawal2015-05-221-9/+0
| | | | | | | | | | | | | | | | | | | | - We do release with release.rb - There is no `rake/gemcutter`
* | | | Merge pull request #20248 from ronakjangir47/remove_mochaRafael Mendonça França2015-05-251-18/+34
|\ \ \ \ | | | | | | | | | | Remove use of mocha in the railties generators tests
| * | | | Remove use of mocha in the railties generators tests Ronak Jangir2015-05-221-18/+34
| |/ / /
* | | | Add missing type reference hereZachary Scott2015-05-241-1/+1
| | | |
* | | | Omit .keep in .gitignore when generated with --skip-keeps.Ben Pickles2015-05-223-2/+12
|/ / /
* | | Remove use of mocha in the railties path testsRoque Pinel2015-05-181-59/+88
| | |
* | | [ci skip] Remove comments about Rails 3.1claudiob2015-05-112-11/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | Stems from https://github.com/rails/rails/pull/20105#issuecomment-100900939 where @senny said: > From my point of view, all the docs (guides, API) are version bound. > They should describe that version and continue to be available when newer versions are released. > The cross referencing can be done by the interested user.
* | | removing unused and already required require'sAnkit Gupta2015-05-112-3/+0
| | | | | | | | | | | | | | | custom test - Does not include EnvHelpers and the require is not needed path generation test - require abstract_unit which has . rails/all requires rails and corresponding
* | | Refactor railties console and dbconsole commandsMehmet Emin İNAÇ2015-05-104-109/+103
| | | | | | | | | | | | fix minor convention problems
* | | Merge pull request #20063 from trayo/masterRafael Mendonça França2015-05-071-0/+10
|\ \ \ | | | | | | | | Adds documentaion for the Rails.root and Rails.public_path methods
| * | | better docs for Rails.root and Rails.public_pathTravis Yoder2015-05-071-0/+10
| | | |
* | | | require object/blank not usedAnkit Gupta2015-05-061-1/+0
| | | | | | | | | | | | | | | | the current class is not the blank?, present? as well as other inheriting, the test suite runs
* | | | add test coverage for `bin/setup`.Yves Senn2015-05-061-0/+54
| | | | | | | | | | | | | | | | Make sure this script keeps working and has consistent output.
* | | | Merge pull request #20023 from mohnish/patch-1Yves Senn2015-05-062-2/+5
|\ \ \ \ | | | | | | | | | | | | | | | Use the built-in rake tasks
| * | | | Use the built-in rake tasksMohnish Thallavajhula2015-05-041-2/+2
| | | | | | | | | | | | | | | Use the `log:clear` and `tmp:clear` rake tasks in the setup file template
* | | | | list reserved words in generator errorSean Linsley2015-05-054-4/+4
| | | | |
* | | | | Merge pull request #19756 from georgemillo/docsRobin Dupret2015-05-051-17/+26
|\ \ \ \ \ | |/ / / / |/| | | | fixing English in Rails::Engine docs
| * | | | fixing English in Rails::Engine docsGeorge Millo2015-04-231-17/+26
| | |/ / | |/| | | | | | | | | | [ci skip]
* | | | Fix constant in void context warningsAndrew White2015-05-051-2/+2
| | | |
* | | | Fix URI.escape is obsolete warningsAndrew White2015-05-051-1/+1
| |/ / |/| |
* | | Document inline image mailer preview interceptorAndrew White2015-05-042-7/+20
| | | | | | | | | | | | Explain what the interceptor is used for and how to remove it.
* | | Tiny documentation edits [ci skip]Robin Dupret2015-05-041-2/+3
| | | | | | | | | | | | | | | | | | * Fix a few typos * Wrap lines to 80 chars * Use `+` instead of `<tt>`
* | | Fix railties configuration testAndrew White2015-05-041-2/+2
| | |
* | | Add support for inline images to mailer previewsAndrew White2015-05-042-8/+10
| | | | | | | | | | | | | | | Use a preview interceptor to search for inline cid: urls in src attributes and convert them to data urls.
* | | Improve display of attachment names in mailer previewsAndrew White2015-05-041-1/+1
| | | | | | | | | | | | | | | Display attachment filenames as a comma separated list rather than showing the inspect output for the array.
* | | Fix mailer previews with attachmentsAndrew White2015-05-044-7/+206
| | | | | | | | | | | | | | | | | | Use the mail gem's own API to locate the correct part. Fixes #14435.
* | | Fix generator testsRafael Mendonça França2015-05-031-1/+1
| | | | | | | | | | | | In my machine the output is different
* | | Remove unneeded base fileRafael Mendonça França2015-05-032-57/+53
| | | | | | | | | | | | We are only using for one test class
* | | Remove unused private classesRafael Mendonça França2015-05-032-186/+0
| | | | | | | | | | | | | | | | | | | | | The usage of these classes where removed at 8017e6af31caa58a58787274ff0ca01397219e49. cc @arthurnn @senny
* | | Merge pull request #19093 from ↵Rafael Mendonça França2015-05-033-3/+33
|\ \ \ | | | | | | | | | | | | | | | | remomueller/fix-actionmailer-preview-links-on-subdirectories Mailer preview now uses `url_for` to fix links to emails for apps runnin...
| * | | Mailer preview now uses `url_for` to fix links to emails for apps running on ↵Remo Mueller2015-04-303-3/+33
| | | | | | | | | | | | | | | | a subdirectory, closes #19092.
* | | | Merge pull request #19976 from prathamesh-sonpatki/rm-assignsRafael Mendonça França2015-05-032-5/+4
|\ \ \ \ | | | | | | | | | | Removed `assigns` from functional_test templates
| * | | | Removed `assigns` from functional_test templatesPrathamesh Sonpatki2015-05-022-5/+4
| |/ / / | | | | | | | | | | | | | | | | - Based on DHH's suggestion about deprecating `assigns` in https://github.com/rails/rails/pull/18305#issuecomment-68605166.
* | | | Use `def before_setup` instead of `setup do`eileencodes2015-05-021-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `setup do` creates unnecessary allocations of proc objects in test callbacks. This prevents that from happening and results in faster code. Originally I had done this as `def setup` and all Railties tests passed. See 044f9ab. Later it was reported there was an issue with this that caused routes in tests to be nil because devs don't generally call `super` in their test setups. Because of that I reverted the commit until I could find a suitble replacement. `before_setup` esentially does the same thing but without the requirement that applications call `super` in their test setups.
* | | | Added missing fullstop in engine API docs [ci skip]Prathamesh Sonpatki2015-05-021-2/+2
|/ / /
* | | Merge pull request #19949 from thiagoaugusto/create-thiago-punctuation-fixYves Senn2015-04-291-1/+1
|\ \ \ | | | | | | | | | | | | CHANGELOG punctuation fix
| * | | CHANGELOG punctuation fixThiago Augusto2015-04-291-1/+1
|/ / / | | | | | | Punctuation fix
* | | Merge pull request #19939 from artofhuman/remove-mocha-from-some-testsRafael Mendonça França2015-04-281-10/+13
|\ \ \ | | | | | | | | Remove use of mocha in the named base tests
| * | | Remove use of mocha in the named base testsSemyon Pupkov2015-04-281-10/+13
| | | |
* | | | mailer previews for `NullMail` instances. Closes #19849.Yves Senn2015-04-283-6/+39
| | | |
* | | | mailer previews, make sure labels and values line up.Yves Senn2015-04-281-0/+4
|/ / / | | | | | | | | | | | | | | | While this was true before when every `dd` had a value, this patch makes sure that everything keeps lining up even when the `dd` node is blank.
* | | Revert "Merge pull request #17920 from ↵Yves Senn2015-04-271-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | calebthompson/dont-rely-on-environment-task-for-schema-load" This reverts commit 08ff4ccbbb3fb143a02e6752efb974a4bcfcd3bb, reversing changes made to 6c9ed6dbc62450cdb87559afd15798305e069146. Caused by #17920. Closes #19545. This patch introduced regressions because initializers were no longer loaded. Specifically missing inflections result in broken restores of the database.