Skip to content

Commit a8579cf

Browse files
authored
Fix RSpec slides array changes matcher using (#247)
1 parent b640cba commit a8579cf

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

slides/rspec.markdown

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1080,6 +1080,7 @@ RSpec.describe User, 'with some elements' do
10801080
10811081
describe 'with another name' do
10821082
let(:params) { { name: 'Mark', position: 'PM' } }
1083+
10831084
it { expect(user.name).to eql('Mark') }
10841085
end
10851086
end
@@ -1698,7 +1699,7 @@ end
16981699
16991700
```ruby
17001701
it 'removes the last element' do
1701-
expect { @array.pop }.to change{ @array.size }.by(1)
1702+
expect { @array.pop }.to change{ @array.size }.by(-1)
17021703
end
17031704
17041705
it 'removes the last element' do
@@ -1710,11 +1711,11 @@ it 'removes the last element' do
17101711
end
17111712
17121713
it 'changes size by at least 1' do
1713-
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_least(1)
1714+
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_least(-1)
17141715
end
17151716
17161717
it 'changes size by at most 2' do
1717-
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_most(2)
1718+
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_most(-2)
17181719
end
17191720
```
17201721

slides/rspec_new.markdown

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -596,7 +596,7 @@ end
596596
597597
```ruby
598598
it 'removes the last element' do
599-
expect { @array.pop }.to change{ @array.size }.by(1)
599+
expect { @array.pop }.to change{ @array.size }.by(-1)
600600
end
601601
602602
it 'removes the last element' do
@@ -608,11 +608,11 @@ it 'removes the last element' do
608608
end
609609
610610
it 'changes size by at least 1' do
611-
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_least(1)
611+
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_least(-1)
612612
end
613613
614614
it 'changes size by at most 2' do
615-
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_most(2)
615+
expect { 2.times { @array.pop } }.to change{ @array.size }.by_at_most(-2)
616616
end
617617
```
618618

0 commit comments

Comments
 (0)