Address comments

Author: David Revelo

lib/generators/rspec/controller/controller_generator.rb

@@ -7,11 +7,12 @@ class ControllerGenerator < Base
       argument :actions, :type => :array, :default => [], :banner => "action action"
 
       class_option :template_engine, :desc => "Template engine to generate view files"
-      class_option :controller_specs, :type => :boolean, :default => true,  :desc => "Generate controller specs"
-      class_option :view_specs,       :type => :boolean, :default => true,  :desc => "Generate view specs"
-      class_option :routing_specs,    :type => :boolean, :default => false,  :desc => "Generate routing specs"
+      class_option :controller_specs, :type => :boolean, :default => true, :desc => "Generate controller specs"
+      class_option :view_specs,       :type => :boolean, :default => true, :desc => "Generate view specs"
+      class_option :routing_specs,    :type => :boolean, :default => false, :desc => "Generate routing specs"
 
       def generate_controller_spec
+        return if actions.empty?
         return unless options[:controller_specs]
 
         template 'controller_spec.rb',

lib/generators/rspec/controller/templates/routing_spec.rb

@@ -1,7 +1,7 @@
 require 'rails_helper'
 
 <% module_namespacing do -%>
-RSpec.describe <%= class_name %>Controller, <%= type_metatag(:routing) %> do
+RSpec.describe '<%= class_name %>Controller', <%= type_metatag(:routing) %> do
   describe 'routing' do
 <% for action in actions -%>
     it 'routes to #<%= action %>' do

spec/generators/rspec/controller/controller_generator_spec.rb

@@ -15,7 +15,7 @@
       describe 'the spec' do
         it { is_expected.to exist }
         it { is_expected.to contain(/require 'rails_helper'/) }
-        it { is_expected.to contain(/^RSpec.describe PostsController, #{type_metatag(:controller)}/) }
+        it { is_expected.to contain(/^RSpec.describe 'PostsController', #{type_metatag(:controller)}/) }
       end
     end
     describe 'skipped with a flag' do
@@ -105,15 +105,15 @@
         before { run_generator %w(posts --routing-specs) }
 
         it { is_expected.to contain(/require 'rails_helper'/) }
-        it { is_expected.to contain(/^RSpec.describe PostsController, #{type_metatag(:routing)}/) }
+        it { is_expected.to contain(/^RSpec.describe 'PostsController', #{type_metatag(:routing)}/) }
         it { is_expected.to contain(/describe 'routing'/) }
       end
 
       describe 'with action parameter' do
         before { run_generator %w(posts seek --routing-specs) }
 
         it { is_expected.to contain(/require 'rails_helper'/) }
-        it { is_expected.to contain(/^RSpec.describe PostsController, #{type_metatag(:routing)}/) }
+        it { is_expected.to contain(/^RSpec.describe 'PostsController', #{type_metatag(:routing)}/) }
         it { is_expected.to contain(/describe 'routing'/) }
         ``
         it { is_expected.to contain(/it 'routes to #seek'/) }