Permalink
Browse files

Merge pull request #10448 from arunagw/bug-fix-scaffold-generator-master

Fixes Scaffold generator with --assets=false
  • Loading branch information...
2 parents 903a2c2 + f1805a6 commit 628d6f133146a034a555760008e9dfc96fb1ff88 @rafaelfranca rafaelfranca committed May 4, 2013
View
@@ -1,3 +1,7 @@
+* Fixes bug with Scaffold generator with --assets=false --resource-route=false See #9525 for more details.
+
+ *Arun Agrawal*
+
* Rails::Railtie no longer forces the Rails::Configurable module on everything
that subclassess it. Instead, the methods from Rails::Configurable have been
moved to class methods in Railtie and the Railtie has been made abstract.
@@ -8,6 +8,8 @@ class ScaffoldGenerator < ResourceGenerator # :nodoc:
class_option :stylesheets, type: :boolean, desc: "Generate Stylesheets"
class_option :stylesheet_engine, desc: "Engine for Stylesheets"
+ class_option :assets, :type => :boolean
+ class_option :resource_route, :type => :boolean
def handle_skip
@options = @options.merge(stylesheets: false) unless options[:assets]
@@ -246,6 +246,20 @@ def test_scaffold_generator_no_assets
assert_no_file "app/assets/stylesheets/posts.css"
end
+ def test_scaffold_generator_no_assets
+ run_generator [ "posts", "--assets=false" ]
+ assert_file "app/assets/stylesheets/scaffold.css"
+ assert_no_file "app/assets/javascripts/posts.js"
+ assert_no_file "app/assets/stylesheets/posts.css"
+ end
+
+ def test_scaffold_generator_no_assets
+ run_generator [ "posts", "--resource-route=false" ]
+ assert_file "config/routes.rb" do |route|
+ assert_no_match(/resources :posts$/, route)
+ end
+ end
+
def test_scaffold_generator_no_stylesheets
run_generator [ "posts", "--no-stylesheets" ]
assert_no_file "app/assets/stylesheets/scaffold.css"

0 comments on commit 628d6f1

Please sign in to comment.