Skip to content

Commit

Permalink
Use consistent naming for request specs
Browse files Browse the repository at this point in the history
Controller generators are using a different naming scheme for requests
specs that are now generated by default. Use always
`spec/requests/posts_spec.rb` by default.

This
  • Loading branch information
eloyesp authored and pirj committed Sep 2, 2020
1 parent c360f35 commit 4b511aa
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
5 changes: 4 additions & 1 deletion example_app_generator/generate_stuff.rb
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,10 @@ def using_source_path(path)
generate('rspec:install')
generate('controller wombats index') # plural
generate('controller welcome index') # singular
generate('rspec:request wombats')

# request specs are now the default
generate('rspec:controller wombats --no-request-specs --controller-specs --no-view-specs')

generate('integration_test widgets')
generate('mailer Notifications #')

Expand Down
2 changes: 1 addition & 1 deletion lib/generators/rspec/controller/controller_generator.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ def generate_request_spec
return unless options[:request_specs]

template 'request_spec.rb',
File.join('spec/requests', class_path, "#{file_name}_request_spec.rb")
File.join('spec/requests', class_path, "#{file_name}_spec.rb")
end

def generate_controller_spec
Expand Down
4 changes: 2 additions & 2 deletions spec/generators/rspec/controller/controller_generator_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
setup_default_destination

describe 'request specs' do
subject { file('spec/requests/posts_request_spec.rb') }
subject { file('spec/requests/posts_spec.rb') }

describe 'generated by default' do
before do
Expand Down Expand Up @@ -38,7 +38,7 @@
end

describe 'with namespace and actions' do
subject { file('spec/requests/admin/external/users_request_spec.rb') }
subject { file('spec/requests/admin/external/users_spec.rb') }

before do
run_generator %w[admin::external::users index custom_action]
Expand Down

0 comments on commit 4b511aa

Please # to comment.