Fixed bug
This commit is contained in:
parent
7f2b6b11bd
commit
264171f72d
2 changed files with 17 additions and 2 deletions
|
@ -25,5 +25,12 @@ module Projects
|
|||
def cache_key
|
||||
['projects', 'count_service', VERSION, @project.id, cache_key_name]
|
||||
end
|
||||
|
||||
def self.query(project_ids)
|
||||
raise(
|
||||
NotImplementedError,
|
||||
'"query" must be implemented and return an ActiveRecord::Relation'
|
||||
)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -4,9 +4,17 @@ describe Projects::CountService do
|
|||
let(:project) { build(:project, id: 1) }
|
||||
let(:service) { described_class.new(project) }
|
||||
|
||||
describe '#relation_for_count' do
|
||||
describe '.query' do
|
||||
it 'raises NotImplementedError' do
|
||||
expect { service.relation_for_count }.to raise_error(NotImplementedError)
|
||||
expect { described_class.query(project.id) }.to raise_error(NotImplementedError)
|
||||
end
|
||||
end
|
||||
|
||||
describe '#relation_for_count' do
|
||||
it 'calls the class method query with the project id' do
|
||||
expect(described_class).to receive(:query).with(project.id)
|
||||
|
||||
service.relation_for_count
|
||||
end
|
||||
end
|
||||
|
||||
|
|
Loading…
Reference in a new issue