-
-
Notifications
You must be signed in to change notification settings - Fork 269
/
named_subject_spec.rb
121 lines (100 loc) · 3.49 KB
/
named_subject_spec.rb
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
# frozen_string_literal: true
RSpec.describe RuboCop::Cop::RSpec::NamedSubject, :config do
shared_examples_for 'checking subject outside of shared examples' do
it 'checks `it` and `specify` for explicit subject usage' do
expect_offense(<<-RUBY)
RSpec.describe User do
subject { described_class.new }
it "is valid" do
expect(subject.valid?).to be(true)
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
specify do
expect(subject.valid?).to be(true)
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
end
RUBY
end
it 'checks before and after for explicit subject usage' do
expect_offense(<<-RUBY)
RSpec.describe User do
subject { described_class.new }
before(:each) do
do_something_with(subject)
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
after do
do_something_with(subject)
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
end
RUBY
end
it 'checks around(:each) for explicit subject usage' do
expect_offense(<<-RUBY)
RSpec.describe User do
subject { described_class.new }
around(:each) do |test|
do_something_with(subject)
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
end
RUBY
end
it 'ignores subject when not wrapped inside a test' do
expect_no_offenses(<<-RUBY)
def foo
it(subject)
end
RUBY
end
end
context 'when IgnoreSharedExamples is false' do
let(:cop_config) { { 'IgnoreSharedExamples' => false } }
it_behaves_like 'checking subject outside of shared examples'
it 'checks shared_examples for explicit subject usage' do
expect_offense(<<-RUBY)
RSpec.describe User do
subject(:new_user) { described_class.new }
shared_examples_for 'a valid new user' do
it "is a User" do
expect(subject).to be_a(User)
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
it "is valid" do
expect(subject.valid?).to be(true)
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
it "is new" do
expect(subject).to be_new_record
^^^^^^^ Name your test subject if you need to reference it explicitly.
end
end
end
RUBY
end
end
context 'when IgnoreSharedExamples is true' do
let(:cop_config) { { 'IgnoreSharedExamples' => true } }
it_behaves_like 'checking subject outside of shared examples'
it 'ignores explicit subject in shared_examples' do
expect_no_offenses(<<-RUBY)
RSpec.describe User do
subject(:new_user) { described_class.new }
shared_examples_for 'a valid new user' do
it "is a User" do
expect(subject).to be_a(User)
end
it "is valid" do
expect(subject.valid?).to be(true)
end
it "is new" do
expect(subject).to be_new_record
end
end
end
RUBY
end
end
end