app_spec.rb
7.78 KB
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
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
describe App, type: 'model' do
context "Attributes" do
it { is_expected.to have_field(:_id).of_type(String) }
it { is_expected.to have_field(:name).of_type(String) }
it { is_expected.to have_fields(:api_key, :github_repo, :bitbucket_repo, :asset_host, :repository_branch) }
it { is_expected.to have_fields(:notify_all_users, :notify_on_errs).of_type(Mongoid::Boolean) }
it { is_expected.to have_field(:email_at_notices).of_type(Array).with_default_value_of(Errbit::Config.email_at_notices) }
end
context 'validations' do
it 'requires a name' do
app = Fabricate.build(:app, name: nil)
expect(app).to_not be_valid
expect(app.errors[:name]).to include("can't be blank")
end
it 'requires unique names' do
Fabricate(:app, name: 'Errbit')
app = Fabricate.build(:app, name: 'Errbit')
expect(app).to_not be_valid
expect(app.errors[:name]).to include('is already taken')
end
it 'requires unique api_keys' do
Fabricate(:app, api_key: 'APIKEY')
app = Fabricate.build(:app, api_key: 'APIKEY')
expect(app).to_not be_valid
expect(app.errors[:api_key]).to include('is already taken')
end
end
describe '<=>' do
it 'is compared by unresolved count' do
app_0 = stub_model(App, name: 'app', unresolved_count: 1, problem_count: 1)
app_1 = stub_model(App, name: 'app', unresolved_count: 0, problem_count: 1)
expect(app_0).to be < app_1
expect(app_1).to be > app_0
end
it 'is compared by problem count' do
app_0 = stub_model(App, name: 'app', unresolved_count: 0, problem_count: 1)
app_1 = stub_model(App, name: 'app', unresolved_count: 0, problem_count: 0)
expect(app_0).to be < app_1
expect(app_1).to be > app_0
end
it 'is compared by name' do
app_0 = stub_model(App, name: 'app_0', unresolved_count: 0, problem_count: 0)
app_1 = stub_model(App, name: 'app_1', unresolved_count: 0, problem_count: 0)
expect(app_0).to be < app_1
expect(app_1).to be > app_0
end
end
context 'being created' do
it 'generates a new api-key' do
app = Fabricate.build(:app)
expect(app.api_key).to be_nil
app.save
expect(app.api_key).to_not be_nil
end
it 'generates a correct api-key' do
app = Fabricate(:app)
expect(app.api_key).to match(/^[a-f0-9]{32}$/)
end
it 'is fine with blank github repos' do
app = Fabricate.build(:app, github_repo: "")
app.save
expect(app.github_repo).to eq ""
end
it 'doesnt touch github user/repo' do
app = Fabricate.build(:app, github_repo: "errbit/errbit")
app.save
expect(app.github_repo).to eq "errbit/errbit"
end
it 'removes domain from https github repos' do
app = Fabricate.build(:app, github_repo: "https://github.com/errbit/errbit")
app.save
expect(app.github_repo).to eq "errbit/errbit"
end
it 'normalizes public git repo as a github repo' do
app = Fabricate.build(:app, github_repo: "https://github.com/errbit/errbit.git")
app.save
expect(app.github_repo).to eq "errbit/errbit"
end
it 'normalizes private git repo as a github repo' do
app = Fabricate.build(:app, github_repo: "git@github.com:errbit/errbit.git")
app.save
expect(app.github_repo).to eq "errbit/errbit"
end
end
context '#github_url_to_file' do
it 'resolves to full path to file' do
app = Fabricate(:app, github_repo: "errbit/errbit")
expect(app.github_url_to_file('path/to/file')).to eq "https://github.com/errbit/errbit/blob/master/path/to/file"
end
end
context '#github_repo?' do
it 'is true when there is a github_repo' do
app = Fabricate(:app, github_repo: "errbit/errbit")
expect(app.github_repo?).to be(true)
end
it 'is false when no github_repo' do
app = Fabricate(:app)
expect(app.github_repo?).to be(false)
end
end
context "notification recipients" do
it "should send notices to either all users plus watchers, or the configured watchers" do
@app = Fabricate(:app)
3.times { Fabricate(:user) }
5.times { Fabricate(:watcher, app: @app) }
@app.notify_all_users = true
expect(@app.notification_recipients.size).to eq 8
@app.notify_all_users = false
expect(@app.notification_recipients.size).to eq 5
end
end
context "emailable?" do
it "should be true if notify on errs and there are notification recipients" do
app = Fabricate(:app, notify_on_errs: true, notify_all_users: false)
2.times { Fabricate(:watcher, app: app) }
expect(app.emailable?).to be(true)
end
it "should be false if notify on errs is disabled" do
app = Fabricate(:app, notify_on_errs: false, notify_all_users: false)
2.times { Fabricate(:watcher, app: app) }
expect(app.emailable?).to be(false)
end
it "should be false if there are no notification recipients" do
app = Fabricate(:app, notify_on_errs: true, notify_all_users: false)
expect(app.watchers).to be_empty
expect(app.emailable?).to be(false)
end
end
context "copying attributes from existing app" do
it "should only copy the necessary fields" do
@app = Fabricate(:app, name: "app", github_repo: "url")
@copy_app = Fabricate(:app, name: "copy_app", github_repo: "copy url")
@copy_watcher = Fabricate(:watcher, email: "copywatcher@example.com", app: @copy_app)
@app.copy_attributes_from(@copy_app.id)
expect(@app.name).to eq "app"
expect(@app.github_repo).to eq "copy url"
expect(@app.watchers.first.email).to eq "copywatcher@example.com"
end
end
context '#find_or_create_err!' do
let(:app) { Fabricate(:app) }
let(:conditions) do
{
error_class: 'Whoops',
environment: 'production',
fingerprint: 'some-finger-print'
}
end
it 'returns the correct err if one already exists' do
existing = Fabricate(
:err,
problem: Fabricate(:problem, app: app),
fingerprint: conditions[:fingerprint]
)
expect(Err.where(fingerprint: conditions[:fingerprint]).first).to eq existing
expect(app.find_or_create_err!(conditions)).to eq existing
end
it 'assigns the returned err to the given app' do
expect(app.find_or_create_err!(conditions).app).to eq app
end
it 'creates a new problem if a matching one does not already exist' do
expect(Err.where(conditions).first).to be_nil
expect do
app.find_or_create_err!(conditions)
end.to change(Problem, :count).by(1)
end
context "without error_class" do
let(:conditions) do
{
environment: 'production',
fingerprint: 'some-finger-print'
}
end
it 'save the err' do
expect(Err.where(conditions).first).to be_nil
expect do
app.find_or_create_err!(conditions)
end.to change(Problem, :count).by(1)
end
end
end
describe ".find_by_api_key!" do
it 'return the app with api_key' do
app = Fabricate(:app)
expect(App.find_by_api_key!(app.api_key)).to eq app
end
it 'raise Mongoid::Errors::DocumentNotFound if not found' do
expect do
App.find_by_api_key!('foo')
end.to raise_error(Mongoid::Errors::DocumentNotFound)
end
end
describe '#notice_fingerprinter' do
it 'app acquires a notice_fingerprinter when it doesn\'t have one' do
app = Fabricate(:app, name: 'Errbit')
app.notice_fingerprinter.delete
# has a notice_fingerprinter because it's been accessed when blank
expect(app.reload.notice_fingerprinter).to be_a(NoticeFingerprinter)
end
it 'brand new app has a notice_fingerprinter' do
app = Fabricate(:app, name: 'Errbit')
expect(app.notice_fingerprinter).to be_a(NoticeFingerprinter)
end
end
end