From b18fe8ce35b1378f6c20a467ca51bbfa248aff12 Mon Sep 17 00:00:00 2001 From: Ana Nunes da Silva Date: Tue, 26 Mar 2024 10:59:35 +0000 Subject: [PATCH] Fix offense constant definition in block in models/spree/preferences/configuration_spec.rb --- .rubocop_todo.yml | 1 - .../spree/preferences/configuration_spec.rb | 19 +++++++++---------- 2 files changed, 9 insertions(+), 11 deletions(-) diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index a906c62651..de77e31cd8 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -11,7 +11,6 @@ # AllowedMethods: enums Lint/ConstantDefinitionInBlock: Exclude: - - 'spec/models/spree/preferences/configuration_spec.rb' - 'spec/models/spree/preferences/preferable_spec.rb' - 'spec/validators/date_time_string_validator_spec.rb' - 'spec/validators/integer_array_validator_spec.rb' diff --git a/spec/models/spree/preferences/configuration_spec.rb b/spec/models/spree/preferences/configuration_spec.rb index 32883d3fb6..fc4b51a709 100644 --- a/spec/models/spree/preferences/configuration_spec.rb +++ b/spec/models/spree/preferences/configuration_spec.rb @@ -3,25 +3,24 @@ require 'spec_helper' describe Spree::Preferences::Configuration do - before :all do - class AppConfig < Spree::Preferences::Configuration + let(:config) do + Class.new(Spree::Preferences::Configuration) do preference :color, :string, default: :blue - end - @config = AppConfig.new + end.new end it "has named methods to access preferences" do - @config.color = 'orange' - expect(@config.color).to eq 'orange' + config.color = 'orange' + expect(config.color).to eq 'orange' end it "uses [ ] to access preferences" do - @config[:color] = 'red' - expect(@config[:color]).to eq 'red' + config[:color] = 'red' + expect(config[:color]).to eq 'red' end it "uses set/get to access preferences" do - @config.set :color, 'green' - expect(@config.get(:color)).to eq 'green' + config.set :color, 'green' + expect(config.get(:color)).to eq 'green' end end