diff --git a/spec/matchers/receive_matcher_spec.cr b/spec/matchers/receive_matcher_spec.cr index 6dfdd28..6ddbfe7 100644 --- a/spec/matchers/receive_matcher_spec.cr +++ b/spec/matchers/receive_matcher_spec.cr @@ -169,7 +169,7 @@ Spectator.describe Spectator::Matchers::ReceiveMatcher do end context "with method calls" do - before_each do + before do dbl.test_method dbl.test_method(1, "wrong", :xyz, foo: "foobarbaz") dbl.irrelevant("foo") @@ -289,7 +289,7 @@ Spectator.describe Spectator::Matchers::ReceiveMatcher do pre_condition { expect(match_data).to be_a(failed_match) } - before_each do + before do dbl.test_method dbl.test_method(1, "test", :xyz, foo: "foobarbaz") dbl.irrelevant("foo") diff --git a/spec/rspec/core/explicit_subject_spec.cr b/spec/rspec/core/explicit_subject_spec.cr index 3775b8a..075bad7 100644 --- a/spec/rspec/core/explicit_subject_spec.cr +++ b/spec/rspec/core/explicit_subject_spec.cr @@ -52,7 +52,7 @@ Spectator.describe "Explicit Subject" do describe Array(Int32) do # TODO: Multiple arguments to describe/context. subject { [] of Int32 } - before_each { subject.push(1, 2, 3) } + before { subject.push(1, 2, 3) } it "has the prescribed elements" do expect(subject).to eq([1, 2, 3]) diff --git a/spec/spectator/dsl/mocks/double_spec.cr b/spec/spectator/dsl/mocks/double_spec.cr index d3eda6d..89f652c 100644 --- a/spec/spectator/dsl/mocks/double_spec.cr +++ b/spec/spectator/dsl/mocks/double_spec.cr @@ -312,7 +312,7 @@ Spectator.describe "Double DSL", :smoke do let(override) { :override } let(dbl) { double(:context_double, override: override) } - before_each { allow(dbl).to receive(:memoize).and_return(memoize) } + before { allow(dbl).to receive(:memoize).and_return(memoize) } it "doesn't change predefined values" do expect(dbl.predefined).to eq(:predefined) diff --git a/spec/spectator/dsl/mocks/mock_spec.cr b/spec/spectator/dsl/mocks/mock_spec.cr index a2b67f3..db63dd9 100644 --- a/spec/spectator/dsl/mocks/mock_spec.cr +++ b/spec/spectator/dsl/mocks/mock_spec.cr @@ -947,7 +947,7 @@ Spectator.describe "Mock DSL", :smoke do let(override) { :override } let(fake) { mock(Dummy, override: override) } - before_each { allow(fake).to receive(:memoize).and_return(memoize) } + before { allow(fake).to receive(:memoize).and_return(memoize) } it "doesn't change predefined values" do expect(fake.predefined).to eq(:predefined) diff --git a/spec/spectator/mocks/double_spec.cr b/spec/spectator/mocks/double_spec.cr index c93a723..1b1abc9 100644 --- a/spec/spectator/mocks/double_spec.cr +++ b/spec/spectator/mocks/double_spec.cr @@ -309,7 +309,7 @@ Spectator.describe Spectator::Double do subject(dbl) { ClassDouble } let(foo_stub) { Spectator::ValueStub.new(:foo, :override) } - after_each { dbl._spectator_clear_stubs } + after { dbl._spectator_clear_stubs } it "overrides an existing method" do expect { dbl._spectator_define_stub(foo_stub) }.to change { dbl.foo }.from(:stub).to(:override) @@ -357,7 +357,7 @@ Spectator.describe Spectator::Double do end describe "._spectator_clear_stubs" do - before_each { dbl._spectator_define_stub(foo_stub) } + before { dbl._spectator_define_stub(foo_stub) } it "removes previously defined stubs" do expect { dbl._spectator_clear_stubs }.to change { dbl.foo }.from(:override).to(:stub) @@ -365,7 +365,7 @@ Spectator.describe Spectator::Double do end describe "._spectator_calls" do - before_each { dbl._spectator_clear_calls } + before { dbl._spectator_clear_calls } # Retrieves symbolic names of methods called on a double. def called_method_names(dbl) @@ -440,7 +440,7 @@ Spectator.describe Spectator::Double do subject(dbl) { FooBarDouble.new } let(stub) { Spectator::ValueStub.new(:foo, 5) } - before_each { dbl._spectator_define_stub(stub) } + before { dbl._spectator_define_stub(stub) } it "removes previously defined stubs" do expect { dbl._spectator_clear_stubs }.to change { dbl.foo }.from(5).to(42) @@ -451,7 +451,7 @@ Spectator.describe Spectator::Double do subject(dbl) { FooBarDouble.new } let(stub) { Spectator::ValueStub.new(:foo, 5) } - before_each { dbl._spectator_define_stub(stub) } + before { dbl._spectator_define_stub(stub) } # Retrieves symbolic names of methods called on a double. def called_method_names(dbl) diff --git a/spec/spectator/mocks/lazy_double_spec.cr b/spec/spectator/mocks/lazy_double_spec.cr index 902922f..c3402b5 100644 --- a/spec/spectator/mocks/lazy_double_spec.cr +++ b/spec/spectator/mocks/lazy_double_spec.cr @@ -246,7 +246,7 @@ Spectator.describe Spectator::LazyDouble do subject(dbl) { Spectator::LazyDouble.new(foo: 42, bar: "baz") } let(stub) { Spectator::ValueStub.new(:foo, 5) } - before_each { dbl._spectator_define_stub(stub) } + before { dbl._spectator_define_stub(stub) } it "removes previously defined stubs" do expect { dbl._spectator_clear_stubs }.to change { dbl.foo }.from(5).to(42) @@ -257,7 +257,7 @@ Spectator.describe Spectator::LazyDouble do subject(dbl) { Spectator::LazyDouble.new(foo: 42, bar: "baz") } let(stub) { Spectator::ValueStub.new(:foo, 5) } - before_each { dbl._spectator_define_stub(stub) } + before { dbl._spectator_define_stub(stub) } # Retrieves symbolic names of methods called on a double. def called_method_names(dbl) diff --git a/spec/spectator/mocks/mock_spec.cr b/spec/spectator/mocks/mock_spec.cr index 0c19759..40b4bf0 100644 --- a/spec/spectator/mocks/mock_spec.cr +++ b/spec/spectator/mocks/mock_spec.cr @@ -312,7 +312,7 @@ Spectator.describe Spectator::Mock do let(mock) { MockThing } let(foo_stub) { Spectator::ValueStub.new(:foo, :override) } - after_each { mock._spectator_clear_stubs } + after { mock._spectator_clear_stubs } it "overrides an existing method" do expect { mock._spectator_define_stub(foo_stub) }.to change { mock.foo }.from(:stub).to(:override) @@ -368,7 +368,7 @@ Spectator.describe Spectator::Mock do end describe "._spectator_clear_stubs" do - before_each { mock._spectator_define_stub(foo_stub) } + before { mock._spectator_define_stub(foo_stub) } it "removes previously defined stubs" do expect { mock._spectator_clear_stubs }.to change { mock.foo }.from(:override).to(:stub) @@ -376,7 +376,7 @@ Spectator.describe Spectator::Mock do end describe "._spectator_calls" do - before_each { mock._spectator_clear_calls } + before { mock._spectator_clear_calls } # Retrieves symbolic names of methods called on a mock. def called_method_names(mock) @@ -410,7 +410,7 @@ Spectator.describe Spectator::Mock do let(mock) { MockThing.new } - after_each { mock._spectator_clear_stubs } + after { mock._spectator_clear_stubs } it "raises a TypeCastError when using a value-based stub" do stub = Spectator::ValueStub.new(:oops, nil).as(Spectator::Stub) @@ -461,7 +461,7 @@ Spectator.describe Spectator::Mock do let(mock) { MockedClass.new } # Necessary to clear stubs to prevent leakages between tests. - after_each { mock._spectator_clear_stubs } + after { mock._spectator_clear_stubs } it "overrides responses from methods with keyword arguments" do expect(mock.method1).to eq(123) @@ -571,8 +571,8 @@ Spectator.describe Spectator::Mock do let(mock) { MockedStruct.new } # Necessary to clear stubs to prevent leakages between tests. - after_each { mock._spectator_clear_stubs } - after_each { MockedStruct._spectator_invocations.clear } + after { mock._spectator_clear_stubs } + after { MockedStruct._spectator_invocations.clear } it "overrides responses from methods with keyword arguments" do expect(mock.method1).to eq(123) @@ -656,7 +656,7 @@ Spectator.describe Spectator::Mock do let(mock) { Thing } let(foo_stub) { Spectator::ValueStub.new(:foo, :override) } - after_each { mock._spectator_clear_stubs } + after { mock._spectator_clear_stubs } it "overrides an existing method" do expect { mock._spectator_define_stub(foo_stub) }.to change { mock.foo }.from(:stub).to(:override) @@ -712,7 +712,7 @@ Spectator.describe Spectator::Mock do end describe "._spectator_clear_stubs" do - before_each { mock._spectator_define_stub(foo_stub) } + before { mock._spectator_define_stub(foo_stub) } it "removes previously defined stubs" do expect { mock._spectator_clear_stubs }.to change { mock.foo }.from(:override).to(:stub) @@ -720,7 +720,7 @@ Spectator.describe Spectator::Mock do end describe "._spectator_calls" do - before_each { mock._spectator_clear_calls } + before { mock._spectator_clear_calls } # Retrieves symbolic names of methods called on a mock. def called_method_names(mock) @@ -756,7 +756,7 @@ Spectator.describe Spectator::Mock do let(mock) { NoReturnThing.new } - after_each { mock._spectator_clear_stubs } + after { mock._spectator_clear_stubs } it "raises a TypeCastError when using a value-based stub" do stub = Spectator::ValueStub.new(:oops, nil).as(Spectator::Stub) diff --git a/spec/spectator/mocks/null_double_spec.cr b/spec/spectator/mocks/null_double_spec.cr index 1aa86ca..4a8ef3a 100644 --- a/spec/spectator/mocks/null_double_spec.cr +++ b/spec/spectator/mocks/null_double_spec.cr @@ -270,7 +270,7 @@ Spectator.describe Spectator::NullDouble do subject(dbl) { ClassDouble } let(foo_stub) { Spectator::ValueStub.new(:foo, :override) } - after_each { dbl._spectator_clear_stubs } + after { dbl._spectator_clear_stubs } it "overrides an existing method" do expect { dbl._spectator_define_stub(foo_stub) }.to change { dbl.foo }.from(:stub).to(:override) @@ -318,7 +318,7 @@ Spectator.describe Spectator::NullDouble do end describe "._spectator_clear_stubs" do - before_each { dbl._spectator_define_stub(foo_stub) } + before { dbl._spectator_define_stub(foo_stub) } it "removes previously defined stubs" do expect { dbl._spectator_clear_stubs }.to change { dbl.foo }.from(:override).to(:stub) @@ -326,7 +326,7 @@ Spectator.describe Spectator::NullDouble do end describe "._spectator_calls" do - before_each { dbl._spectator_clear_calls } + before { dbl._spectator_clear_calls } # Retrieves symbolic names of methods called on a double. def called_method_names(dbl) @@ -401,7 +401,7 @@ Spectator.describe Spectator::NullDouble do subject(dbl) { FooBarDouble.new } let(stub) { Spectator::ValueStub.new(:foo, 5) } - before_each { dbl._spectator_define_stub(stub) } + before { dbl._spectator_define_stub(stub) } it "removes previously defined stubs" do expect { dbl._spectator_clear_stubs }.to change { dbl.foo }.from(5).to(42) @@ -412,7 +412,7 @@ Spectator.describe Spectator::NullDouble do subject(dbl) { FooBarDouble.new } let(stub) { Spectator::ValueStub.new(:foo, 5) } - before_each { dbl._spectator_define_stub(stub) } + before { dbl._spectator_define_stub(stub) } # Retrieves symbolic names of methods called on a double. def called_method_names(dbl)