aboutsummaryrefslogtreecommitdiffstats
path: root/railties/test/generators/model_generator_test.rb
Commit message (Collapse)AuthorAgeFilesLines
* Pare back default `index` option for the migration generatorPrathamesh Sonpatki2016-01-241-20/+0
| | | | | | | | | | - Using `references` or `belongs_to` in migrations will always add index for the referenced column by default, without adding `index:true` option to generated migration file. - Users can opt out of this by passing `index: false`. - Legacy migrations won't be affected by this change. They will continue to run as they were before. - Fixes #18146
* Introduce ApplicationRecord, an Active Record layer supertypeGenadi Samokovarov2015-12-161-0/+11
| | | | | | | | | | | | | | | | It's pretty common for folks to monkey patch `ActiveRecord::Base` to work around an issue or introduce extra functionality. Instead of shoving even more stuff in `ActiveRecord::Base`, `ApplicationRecord` can hold all those custom work the apps may need. Now, we don't wanna encourage all of the application models to inherit from `ActiveRecord::Base`, but we can encourage all the models that do, to inherit from `ApplicationRecord`. Newly generated applications have `app/models/application_record.rb` present by default. The model generators are smart enough to recognize that newly generated models have to inherit from `ApplicationRecord`, but only if it's present.
* Ensure generated migrations include a version numberMatthew Draper2015-12-151-8/+8
|
* Use thor class_option to make the primary_key_type option workRafael Mendonça França2015-10-231-0/+9
| | | | Also move the method to the right class
* respect `pluralize_table_names` when generate fixture file. fixes #19519yuuji.yaginuma2015-03-271-0/+10
|
* Add Secure Token Generatorrobertomiranda2015-01-111-0/+11
|
* Change the default `null` value for `timestamps` to `false`Rafael Mendonça França2015-01-041-1/+1
|
* Generated fixtures won't use parent_id when generated with parent:referencesPablo Olmos de Aguilera Corradini2015-01-021-5/+5
| | | | Fix #18301
* Use the new `foreign_key` option on `references` in generatorsSean Griffin2014-12-221-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes `rails g model Post user:references` from def change create_table :posts do |t| t.references :user, index: true end add_foreign_key :posts, :users end to def change create_table :posts do |t| t.references :user, index: true, foreign_key: true end end Changes `rails g migration add_user_to_posts user:references` from def change add_reference :posts, :users, index: true add_foreign_key :posts, :users end to def change add_reference :posts, :users, index: true, foreign_key: true end
* Generators add foreign keys on referencesDerek Prior2014-11-251-0/+21
| | | | | | | | | | | If you run a generator such as: ``` rails generate model accounts supplier:references ``` The resulting migration will now add the corresponding foreign key constraint unless the reference was specified to be polymorphic.
* Merge pull request #16062 from sgrif/sg-required-generatorsRafael Mendonça França2014-08-171-0/+44
|\ | | | | Add a `required` option to the model generator
| * Add a `required` option to the model generatorSean Griffin2014-08-081-0/+44
| | | | | | | | | | | | | | | | | | | | | | Syntax was chosen to follow the passing of multiple options to decimal/numeric types. Curly braces, and allowing any of `,`, `.`, or `-` to be used as a separator to avoid the need for shell quoting. (I'm intending to expand this to all columns, but that's another PR. The `required` option will cause 2 things to change. `required: true` will be added to the association. `null: false` will be added to the column in the migration.
* | Change the default `null` value for timestampsSean Griffin2014-08-121-1/+1
|/ | | | | | | As per discussion, this changes the model generators to specify `null: false` for timestamp columns. A warning is now emitted if `timestamps` is called without a `null` option specified, so we can safely change the behavior when no option is specified in Rails 5.
* modify model generator warning message. refs [#174c9f0]Kuldeep Aggarwal2014-02-271-1/+1
|
* include names in model generator warning message. refs #13515.Yves Senn2014-02-241-1/+1
| | | | | | This is a follow up to #13515. It includes the name given and the singularized version in the warning message. This will aide the user to see wether the detected singular was right or not.
* Add warning when user tried to create model with pluralize name.Kuldeep Aggarwal2014-02-231-0/+7
| | | | | 1. Generate model with correct_name. 2. It will help new users to avoid mistakes when tried to create model with wrong name.
* Consistent use of single and double quotesRafael Mendonça França2012-12-261-10/+10
|
* quote column names in generated fixture filesYves Senn2012-12-261-3/+26
|
* Make sure that no extra spaces are created with a non-polymorphicRafael Mendonça França2012-12-101-2/+2
| | | | attributes
* Deal with polymorphic attributes correctly in the generatorsRafael Mendonça França2012-12-101-0/+5
|
* Make references and belongs_to attributes to generate the _id column inRafael Mendonça França2012-12-101-0/+5
| | | | fixtures
* Use Ruby 1.9 Hash syntax in railtiesRobin Dupret2012-10-141-3/+3
|
* Update test locationsMike Moore2012-10-091-3/+3
| | | | | | | | | | | | | | Change the default test locations to avoid confusion around the common testing terms "unit" and "functional". Add new rake tasks for the new locations, while maintaining backwards compatibility with the old rake tasks. New testing locations are as follows: app/models -> test/models (was test/units) app/helpers -> test/helpers (was test/units/helpers) app/controllers -> test/controllers (was test/functional) app/mailers -> test/mailers (was test/functional)
* Remove all references to attr_accessible/protected and old ↵Guillermo Iguaran2012-09-161-10/+0
| | | | mass_assignment_sanitizers
* Add polymorphic option to model generatorAleksey Magusev2012-06-271-3/+14
| | | | | | | | | | For instance, $ rails g model Product supplier:references{polymorphic} generate model with `belongs_to :supplier, polymorphic: true` association and appropriate migration. Also fix model_generator_test.rb#L196 and #L201
* fix scaffold_generator_test.rb and model_generator_test.rbArun Agrawal2012-04-161-4/+4
| | | | Broken after this 6a054b0038bac288a1f6e45feb5470f4ee492081
* Whitelist all attribute assignment by default.Michael Koziarski2012-03-051-0/+10
| | | | Change the default for newly generated applications to whitelist all attribute assignment. Also update the generated model classes so users are reminded of the importance of attr_accessible.
* Use 1.9 hash syntax instead.José Valim2011-12-241-6/+6
|
* assert_not_match -> assert_no_match.José Valim2011-12-241-4/+4
|
* Tidy up migration types.José Valim2011-12-241-1/+1
|
* added ability to specify from cli when generating a model/migration whether ↵Dmitrii Samoilov2011-12-241-0/+68
| | | | particular property should be an index like this 'rails g model person name:string:index profile:string'
* Attributes on scaffold and model generators default to string.José Valim2011-06-171-3/+9
| | | | This allows the following: "rails g scaffold Post title body:text author"
* be sure to parenthesize the arguments when the first one is a RegExp literalAkira Matsuda2011-05-181-17/+16
| | | | | | this fixes: "warning: ambiguous first argument; put parentheses or even spaces" because: you need this to tell the parser that you're not calling :/ method (division) details (Japanese!): http://blade.nagaokaut.ac.jp/cgi-bin/vframe.rb/ruby/ruby-dev/42445?42370-43277
* Correct migration name the test is looking to match.Ben Orenstein2011-02-041-1/+1
| | | | | | [#6375 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Configurable generation of add_index for references columnsMichał Łomnicki2011-02-011-0/+41
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Use Rails 3.1 `change` method in model generatorPrem Sichanugrist2011-01-041-6/+2
|
* Skip creating migration if --skip option is passed to model generator [#6144 ↵Piotr Sarnacki2010-12-161-3/+15
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Facepunch failing tests.José Valim2010-11-181-3/+3
|
* Remove existing migration when using 'rails generate model' with --force ↵David Trasbo2010-09-181-0/+9
| | | | | | [#5526 state:committed] Signed-off-by: José Valim <jose.valim@gmail.com>
* Add two integration tests for GeneratedAttribute missing type error. [#5461 ↵rohit2010-09-021-0/+6
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Generate module file for namespaced models [#4230 state:resolved]Andrew White2010-04-121-0/+8
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Load generators from both lib/rails/generators and lib/generators. Using the ↵José Valim2010-03-231-1/+1
| | | | former since it's less obstrusive.
* Get generators tests running on Ruby 1.9.1José Valim2010-01-191-1/+2
|
* Generators load path now will be Ruby load path. If you want to use ↵José Valim2010-01-181-1/+1
| | | | rspec:install generator, you need generators/rspec/install_generator in your load path.
* Move all generators tests to use new test case syntax.José Valim2010-01-031-8/+1
|
* Create Rails::Generators::TestCase.José Valim2010-01-031-3/+3
|
* CI breakageYehuda Katz2009-11-021-1/+1
| | | | This reverts commit a288b74f1c75c6f100de7611a5093a421f1ad6d1.
* Generators should use Rails.root instead of Dir.pwd [#3408 status:resolved]José Valim2009-10-281-1/+1
| | | | Signed-off-by: Yehuda Katz <wycats@gmail.com>
* Move railties/lib/* into railties/lib/*Yehuda Katz + Carl Lerche2009-09-241-1/+1
|
* Fixed generating a namespaced model with table pluralization turned off. Add ↵Hugo Peixoto2009-08-091-0/+44
| | | | | | | | tests for namespaced model generation. [#767 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-07 05:41:08 +0000