Commit 025a5b42 authored by drew cimino's avatar drew cimino

Use integer literal in spec assertion

The FactoryBot build model has no user_id, so we provide one to
ensure that a missing param (nil) doesn't also pass this spec.
parent af10069f
...@@ -181,10 +181,14 @@ RSpec.describe Ci::CreateJobArtifactsService do ...@@ -181,10 +181,14 @@ RSpec.describe Ci::CreateJobArtifactsService do
end end
context 'when artifact_type is metrics' do context 'when artifact_type is metrics' do
before do
allow(job).to receive(:user_id).and_return(123)
end
let(:params) { { 'artifact_type' => 'metrics', 'artifact_format' => 'gzip' }.with_indifferent_access } let(:params) { { 'artifact_type' => 'metrics', 'artifact_format' => 'gzip' }.with_indifferent_access }
it 'tracks the job user_id' do it 'tracks the job user_id' do
expect(service).to receive(:track_usage_event).with('i_metrics_report_artifact_uploaders', job.user_id) expect(service).to receive(:track_usage_event).with('i_metrics_report_artifact_uploaders', 123)
subject subject
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment