Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Ensure generated migrations include a version number | Matthew Draper | 2015-12-15 | 3 | -14/+14 |
| | |||||
* | Revert "Merge pull request #22569 from seuros/remove-test-case" | eileencodes | 2015-12-13 | 4 | -12/+12 |
| | | | | | | | | | This reverts commit 9712a7a12b7f9e4dcef751ceda8a73c3f4beb11f, reversing changes made to 3e65c3d3886336e9145438cdeacaf4ebec6a48b8. Reverting because this caused test failures and while we have a followup branch there is still one failure that happens randomly and isn't straight forward to fix. | ||||
* | Merge pull request #22569 from seuros/remove-test-case | Eileen M. Uchitelle | 2015-12-13 | 4 | -12/+12 |
|\ | | | | | Controller generators should be creating IntegrationTest stubs instea… | ||||
| * | Controller generators should be creating IntegrationTest stubs instead of ↵ | Abdelkader Boudih | 2015-12-13 | 4 | -12/+12 |
| | | | | | | | | ControllerTest | ||||
* | | Merge pull request #22574 from gsamokovarov/scaffold-500 | Kasper Timm Hansen | 2015-12-13 | 1 | -1/+1 |
|\ \ | | | | | | | Prevent a 500 in the default controller scaffold | ||||
| * | | Prevent a 500 in the default controller scaffold | Genadi Samokovarov | 2015-12-13 | 1 | -1/+1 |
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you update a record with no attributes, you'll hit a 500 from a guard in `assign_attributes` raising: ```ruby ArgumentError: When assigning attributes, you must pass a hash as an argument. app/controllers/users_controller.rb:44:in `block in update' app/controllers/users_controller.rb:43:in `update' test/controllers/users_controller_test.rb:37:in `block in <class:UsersControllerTest>' ``` Not a biggie, but may be quite confusing for the folks new to the framework. | ||||
* / | Update web-console to 3.0.0 | Genadi Samokovarov | 2015-12-13 | 1 | -2/+2 |
|/ | | | | | Web Console 3.0.0 is compatible with Rails 5, while the 2.x.x releases aren't. | ||||
* | show relative path the rerun snippet of test runner in rails engine | yuuji.yaginuma | 2015-12-10 | 3 | -22/+58 |
| | | | | | | | | | | | | | Since the absolute path is not required to re-run the test, modified so that unnecessary information is not displayed. ```ruby # before bin/rails test /path/to/blorgh/test/integration/navigation_test.rb:5 # after bin/rails test test/integration/navigation_test.rb:5 ``` | ||||
* | Merge pull request #22508 from ↵ | Santiago Pastorino | 2015-12-07 | 1 | -0/+1 |
|\ | | | | | | | | | y-yagi/remove_request_forgery_protection_from_rails_api remove `request_forgery_protection` initializer from rails api | ||||
| * | remove `request_forgery_protection` initializer from rails api | yuuji.yaginuma | 2015-12-06 | 1 | -0/+1 |
| | | | | | | | | | | In rails api, since default is so as not to use the `protect_from_forgery`, `request_forgery_protection` initializer I think that's unnecessary | ||||
* | | Mac OS X use /private as directory to temp file | Rafael Mendonça França | 2015-12-04 | 1 | -1/+1 |
| | | | | | | | | Related with e62ddac05b7d827ae870898d7cd9eaf76e0730aa | ||||
* | | Remove test_files check from plugin checks | Rafael Mendonça França | 2015-12-04 | 1 | -4/+0 |
|/ | | | | Related with 668c006cf33a63ac3f8a8f064c772a339341e70c. | ||||
* | add `bin/test` script to rails plugin | yuuji.yaginuma | 2015-11-28 | 2 | -1/+129 |
| | |||||
* | fix application_controller require_dependency path generated by api scaffold ↵ | yuuji.yaginuma | 2015-11-08 | 1 | -0/+24 |
| | | | | | | | | | generator In the app generator, regardless of the namespace, it is adapted to read the same `application_controller.rb`, modified to api generator also be the same behavior. ref 686966a1861e6761021d45f6795de753e8a62216, 06ac63b4c602e4b26cb3bfbf0dfd2bed8b00e786 | ||||
* | Route generator should be idempotent - closes #22082 | Thiago Pinto | 2015-10-31 | 1 | -0/+15 |
| | |||||
* | Add version constraint to database gem in generated application | Rafael Mendonça França | 2015-10-29 | 1 | -4/+8 |
| | | | | | | We are using the same version constraint in the database adapters so when a new version of the adapter that doesn't work with the version of rails is released we don't break new applications. | ||||
* | Use thor class_option to make the primary_key_type option work | Rafael Mendonça França | 2015-10-23 | 2 | -6/+10 |
| | | | | Also move the method to the right class | ||||
* | Move default uuid generation to active_record | Jon McCartie | 2015-10-23 | 1 | -3/+4 |
| | |||||
* | remove unnecessary readme in dummy application | yuuji.yaginuma | 2015-10-23 | 1 | -0/+13 |
| | | | | `README` it is changed to `README.rdoc` in 6b126e2, it has been changed to` README.md` further 89a12c9. | ||||
* | Set active_record config for always creating uuids in generators | Jon McCartie | 2015-10-20 | 1 | -0/+13 |
| | |||||
* | Merge pull request #20908 from y-yagi/generate_application_job_inside_engine | Arthur Nogueira Neves | 2015-10-13 | 1 | -0/+4 |
|\ | | | | | add application_job.rb to template of mountable engine | ||||
| * | add application_job.rb to template of mountable engine | yuuji.yaginuma | 2015-07-17 | 1 | -0/+4 |
| | | | | | | | | | | since cb012467214f6e4bb1ac3987554bb75020b4796b, generated job inherents from ApplicationJob, ApplicationJob is required in the mountable engine. | ||||
* | | do not generate manifest.js in plugins | yuuji.yaginuma | 2015-10-12 | 1 | -0/+1 |
| | | | | | | | | Since the plugin generator do not generate assets, I think manifest.js also that it unnecessary. | ||||
* | | Removed Mocha from app generators tests | Ronak Jangir | 2015-09-23 | 1 | -87/+106 |
| | | |||||
* | | Remove unneeded generator variable. | Kasper Timm Hansen | 2015-09-22 | 1 | -1/+0 |
| | | |||||
* | | Removed mocha from Railites PluginGeneratorTest | Ronak Jangir | 2015-09-22 | 2 | -15/+35 |
| | | |||||
* | | fix application_controller require_dependency path generated by the scaffold ↵ | yuuji.yaginuma | 2015-09-20 | 1 | -0/+1 |
| | | | | | | | | | | | | | | generator This is follow up to #6643. In #6643, the controller generator only had been fixed, in this commit to fix the scaffold generator. | ||||
* | | Removed duplicate requiring minitest/mock as it is already required in ↵ | Ronak Jangir | 2015-08-26 | 1 | -1/+0 |
| | | | | | | | | method_call_assertions | ||||
* | | Cleaned up generators tests using internal assertion helper | Ronak Jangir | 2015-08-20 | 2 | -26/+13 |
| | | |||||
* | | Updated tests for the generated version number change | Daniel Morris | 2015-08-16 | 1 | -3/+3 |
| | | |||||
* | | make generated scaffold functional tests work inside API engines | yuuji.yaginuma | 2015-08-09 | 1 | -0/+28 |
| | | | | | | | | | | * set engine's route in setup * use fixture with engine namespace | ||||
* | | Merge pull request #21043 from ↵ | Rafael Mendonça França | 2015-08-06 | 1 | -0/+1 |
|\ \ | | | | | | | | | | | | | y-yagi/loading_fixtures_in_engine_integration_tests set the correct path to `ActionDispatch::IntegrationTest.fixture_path` | ||||
| * | | set the correct path to `ActionDispatch::IntegrationTest.fixture_path` | yuuji.yaginuma | 2015-07-28 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | `ActionDispatch::IntegrationTest.fixture_path` set by `test_help.rb`, but if the engine, path under the dummy is will be set, fixtures under test was not loaded. | ||||
* | | | Enable the `api_only` option for API plugins' generators | Robin Dupret | 2015-07-23 | 1 | -0/+26 |
| | | | | | | | | | | | | | | | | | | | | | 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 plugins | Robin Dupret | 2015-07-23 | 1 | -0/+8 |
| | | | |||||
* | | | Make ApplicationController inherit from AC::API for API plugins | Robin Dupret | 2015-07-23 | 1 | -0/+8 |
| | | | |||||
* | | | Skip assets, helpers and views generation for API plugins | Robin Dupret | 2015-07-23 | 1 | -0/+12 |
|/ / | |||||
* | | Merge pull request #20819 from y-yagi/rails_api_gemfile | Rafael Mendonça França | 2015-07-21 | 1 | -0/+1 |
|\ \ | |/ |/| | remove `web-console` from API app generated Gemfile | ||||
| * | remove `web-console` from API app generated Gemfile | yuuji.yaginuma | 2015-07-09 | 1 | -0/+1 |
| | | |||||
* | | make test runner work correctly inside engine | yuuji.yaginuma | 2015-07-12 | 2 | -4/+4 |
|/ | |||||
* | Display a more human readable list of reserved names | Robin Dupret | 2015-07-01 | 2 | -2/+2 |
| | | | | | Interpolating an array inside a String will call #inspect on it. Let's call #join to display a more human-readable error message. | ||||
* | fix NoMethodError that occurs when generating scaffold inside full mode engine | yuuji.yaginuma | 2015-06-28 | 2 | -0/+25 |
| | |||||
* | cherry-pick test and source doc from #20050 | Hirofumi Wakasugi | 2015-06-25 | 1 | -0/+8 |
| | |||||
* | Merge pull request #20605 from dcrec1/assert_file | Yves Senn | 2015-06-19 | 1 | -1/+1 |
|\ | | | | | | | assert_file understands paths with special characters | ||||
| * | assert_file understands paths with special characters | Diego Carrion | 2015-06-18 | 1 | -1/+1 |
| | | | | | | fixes #20042 | ||||
* | | fix railties tests | Mehmet Emin İNAÇ | 2015-06-18 | 1 | -2/+2 |
|/ | |||||
* | Enable wrap_parameter by default in rails api applications | Jorge Bejar | 2015-06-11 | 1 | -3/+1 |
| | |||||
* | Add rake-cors gem (commented) in Gemfile for rails api apps | Jorge Bejar | 2015-06-11 | 1 | -0/+3 |
| | |||||
* | Add AMS 0.10.0.rc1 by default for api apps | Santiago Pastorino | 2015-06-11 | 1 | -0/+1 |
| | |||||
* | Fix scaffold generator test for resource routes | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| |