Fix generate request spec name has / (#2057)

Use `name.underscore.pluralize` instead of file and table name to handle the case where generating request specs for name spaced files

Author: koheisg

lib/generators/rspec/integration/integration_generator.rb

@@ -15,7 +15,7 @@ def generate_request_spec
         return unless options[:request_specs]
 
         template 'request_spec.rb',
-                 File.join('spec/requests', class_path, "#{table_name}_spec.rb")
+                 File.join('spec/requests', "#{name.underscore.pluralize}_spec.rb")
       end
     end
   end

lib/generators/rspec/integration/templates/request_spec.rb

@@ -1,7 +1,7 @@
 require 'rails_helper'
 
 RSpec.describe "<%= class_name.pluralize %>", <%= type_metatag(:request) %> do
-  describe "GET /<%= table_name %>" do
+  describe "GET /<%= name.underscore.pluralize %>" do
     it "works! (now write some real specs)" do
       get <%= index_helper %>_path
       expect(response).to have_http_status(200)

spec/support/generators.rb

@@ -34,32 +34,65 @@ def self.included(klass)
 
           describe 'generated with no flags' do
             before do
-              run_generator %w(posts)
+              run_generator name
             end
 
-            subject(:request_spec) { file('spec/requests/posts_spec.rb') }
+            subject(:request_spec) { file(spec_file_name) }
 
-            it "creates the request spec" do
-              expect(request_spec).to exist
-            end
+            context 'When NAME=posts' do
+              let(:name) { %w(posts) }
+              let(:spec_file_name) { 'spec/requests/posts_spec.rb' }
 
-            it "the generator requires 'rails_helper'" do
-              expect(request_spec).to contain(/require 'rails_helper'/)
-            end
+              it "creates the request spec" do
+                expect(request_spec).to exist
+              end
 
-            it "the generator describes the provided NAME without monkey " \
-               "patching setting the type to `:request`" do
-              expect(request_spec).to contain(
-                /^RSpec.describe \"Posts\", #{type_metatag(:request)}/
-              )
-            end
+              it "the generator requires 'rails_helper'" do
+                expect(request_spec).to contain(/require 'rails_helper'/)
+              end
+
+              it "the generator describes the provided NAME without monkey " \
+                 "patching setting the type to `:request`" do
+                expect(request_spec).to contain(
+                  /^RSpec.describe \"Posts\", #{type_metatag(:request)}/
+                )
+              end
+
+              it "the generator includes a sample GET request" do
+                expect(request_spec).to contain(/describe "GET \/posts"/)
+              end
 
-            it "the generator includes a sample GET request" do
-              expect(request_spec).to contain(/describe "GET \/posts"/)
+              it "the generator sends the GET request to the index path" do
+                expect(request_spec).to contain(/get posts_index_path/)
+              end
             end
 
-            it "the generator sends the GET request to the index path" do
-              expect(request_spec).to contain(/get posts_index_path/)
+            context 'When NAME=api/posts' do
+              let(:name) { %w(api/posts) }
+              let(:spec_file_name) { 'spec/requests/api/posts_spec.rb' }
+
+              it "creates the request spec" do
+                expect(request_spec).to exist
+              end
+
+              it "the generator requires 'rails_helper'" do
+                expect(request_spec).to contain(/require 'rails_helper'/)
+              end
+
+              it "the generator describes the provided NAME without monkey " \
+                 "patching setting the type to `:request`" do
+                expect(request_spec).to contain(
+                  /^RSpec.describe \"Api::Posts\", #{type_metatag(:request)}/
+                )
+              end
+
+              it "the generator includes a sample GET request" do
+                expect(request_spec).to contain(/describe "GET \/api\/posts\"/)
+              end
+
+              it "the generator sends the GET request to the index path" do
+                expect(request_spec).to contain(/get api_posts_index_path\n/)
+              end
             end
           end
         end