Skip to content
This repository was archived by the owner on Oct 28, 2020. It is now read-only.

Commit afa98db

Browse files
author
v.promzelev
committed
use real logger in tests
1 parent f6fea7b commit afa98db

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

spec/activerecord/overflow_signalizer_spec.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -90,12 +90,12 @@
9090

9191
describe '#analyse' do
9292
context 'signalize to logger' do
93-
let!(:logger) { double(:logger, warn: true) }
93+
let!(:logger) { Logger.new('/dev/null') }
9494

9595
subject { described_class.new(logger: logger, models: [TestIntModel], days_count: 10) }
9696

9797
context 'empty table' do
98-
it 'doesnt log anything' do
98+
it 'doesnt log to warn' do
9999
expect(logger).not_to receive(:warn)
100100
subject.analyse
101101
end
@@ -116,7 +116,7 @@
116116
TestIntModel.destroy_all
117117
end
118118

119-
it 'doesnt log anything' do
119+
it 'doesnt log to warn' do
120120
expect(logger).not_to receive(:warn)
121121
subject.analyse
122122
end
@@ -170,7 +170,7 @@
170170
subject { described_class.new(signalizer: signalizer, models: [TestIntModel], days_count: 10) }
171171

172172
context 'empty table' do
173-
it 'doesnt log anything' do
173+
it 'doesnt log to warn' do
174174
expect(signalizer).not_to receive(:signalize)
175175
subject.analyse
176176
end
@@ -191,7 +191,7 @@
191191
TestIntModel.destroy_all
192192
end
193193

194-
it 'doesnt log anything' do
194+
it 'doesnt log to warn' do
195195
expect(signalizer).not_to receive(:signalize)
196196
subject.analyse
197197
end

0 commit comments

Comments
 (0)