Parcourir la source

Enforce no_space for Ruby hash literal

Andrew Swistak il y a 6 ans
Parent
commit
aa04ff9096

+ 1 - 0
.rubocop.yml

@@ -181,6 +181,7 @@ Layout/SpaceInsideBlockBraces:
 # Use `{ a: 1 }` not `{a:1}`.
 Layout/SpaceInsideHashLiteralBraces:
   Enabled: true
+  EnforcedStyle: no_space
 
 Layout/SpaceInsideParens:
   Enabled: true

+ 1 - 1
config/routes.rb

@@ -5,7 +5,7 @@ Rails.application.routes.draw do
 
   root to: 'welcome#root'
 
-  namespace :api, defaults: { formats: :json } do
+  namespace :api, defaults: {formats: :json} do
     namespace :v1 do
       resources :pokemon do
         collection do

+ 2 - 2
spec/capybara_helper.rb

@@ -4,7 +4,7 @@ require 'capybara/rspec'
 require 'selenium/webdriver'
 
 Capybara.default_max_wait_time = 10
-Capybara.server = :puma, { Silent: true }
+Capybara.server = :puma, {Silent: true}
 
 # Headless firefox and chrome configuration. Useful for CI matrices.
 # See https://gist.github.com/bbonamin/4b01be9ed5dd1bdaf909462ff4fdca95
@@ -23,7 +23,7 @@ end
 options = Selenium::WebDriver::Chrome::Options.new
 options.add_preference(:download, prompt_for_download: false,
                                   default_directory: '/tmp/downloads')
-options.add_preference(:browser, set_download_behavior: { behavior: 'allow' })
+options.add_preference(:browser, set_download_behavior: {behavior: 'allow'})
 
 Capybara.register_driver :chrome do |app|
   Capybara::Selenium::Driver.new(app, browser: :chrome, options: options)

+ 4 - 4
spec/controllers/api/v1/pokemon_controller_spec.rb

@@ -11,7 +11,7 @@ RSpec.describe API::V1::PokemonController, type: :controller do
 
   describe 'DELETE #destroy' do
     let!(:pokemon) { FactoryBot.create(:pokemon) }
-    subject { delete :destroy, params: { id: pokemon.id } }
+    subject { delete :destroy, params: {id: pokemon.id} }
 
     context 'the pokemon is successfully deleted' do
       it 'deletes the pokemon' do
@@ -40,11 +40,11 @@ RSpec.describe API::V1::PokemonController, type: :controller do
   end
 
   describe 'POST #upload' do
-    subject { post :upload, params: { pokemon: double } }
+    subject { post :upload, params: {pokemon: double} }
     let(:client_response) { double(pokemon: [pokemon]) }
 
     let(:pokemon) do
-      double(pokedex_number: 10, nickname: 'pyukuchu', to_h: { pokedex_number: 10, nickname: 'pyukuchu' })
+      double(pokedex_number: 10, nickname: 'pyukuchu', to_h: {pokedex_number: 10, nickname: 'pyukuchu'})
     end
 
     before do
@@ -89,7 +89,7 @@ RSpec.describe API::V1::PokemonController, type: :controller do
   describe 'GET #show' do
     context 'the pokemon exists' do
       let!(:pokemon) { FactoryBot.create(:pokemon) }
-      subject { get :show, params: { id: pokemon.id }, format: :json }
+      subject { get :show, params: {id: pokemon.id}, format: :json }
 
       it 'renders the pokemon JSON' do
         subject