diff --git a/spec/factory_bot/disallows_duplicates_registry_spec.rb b/spec/factory_bot/disallows_duplicates_registry_spec.rb index 9597cf9..4cf8590 100644 --- a/spec/factory_bot/disallows_duplicates_registry_spec.rb +++ b/spec/factory_bot/disallows_duplicates_registry_spec.rb @@ -1,17 +1,19 @@ describe FactoryBot::Decorator::DisallowsDuplicatesRegistry do - let(:registry) { double("registry", name: "Great thing", register: true) } - - subject { described_class.new(registry) } - it "delegates #register to the registry when not registered" do + registry = double("registry", name: "Great thing", register: true) + decorator = FactoryBot::Decorator::DisallowsDuplicatesRegistry.new(registry) allow(registry).to receive(:registered?).and_return false - subject.register(:awesome, {}) + decorator.register(:awesome, {}) + expect(registry).to have_received(:register).with(:awesome, {}) end it "raises when attempting to #register a previously registered strategy" do + registry = double("registry", name: "Great thing", register: true) + decorator = FactoryBot::Decorator::DisallowsDuplicatesRegistry.new(registry) allow(registry).to receive(:registered?).and_return true - expect { subject.register(:same_name, {}) }. + + expect { decorator.register(:same_name, {}) }. to raise_error(FactoryBot::DuplicateDefinitionError, "Great thing already registered: same_name") end end