Commit 1a1c366c authored by Dmytro Zaporozhets (DZ)'s avatar Dmytro Zaporozhets (DZ)

Merge branch 'fix-ci-reports-test-case-spec' into 'master'

Use parent strategy for associations in factories - CI Reports Test Case spec

See merge request gitlab-org/gitlab!41353
parents 3958c78d 9bf2fff5
...@@ -4,7 +4,7 @@ require 'spec_helper' ...@@ -4,7 +4,7 @@ require 'spec_helper'
RSpec.describe Gitlab::Ci::Reports::TestCase do RSpec.describe Gitlab::Ci::Reports::TestCase do
describe '#initialize' do describe '#initialize' do
let(:test_case) { described_class.new(params)} let(:test_case) { described_class.new(params) }
context 'when both classname and name are given' do context 'when both classname and name are given' do
context 'when test case is passed' do context 'when test case is passed' do
...@@ -62,7 +62,9 @@ RSpec.describe Gitlab::Ci::Reports::TestCase do ...@@ -62,7 +62,9 @@ RSpec.describe Gitlab::Ci::Reports::TestCase do
end end
context 'when attachment is present' do context 'when attachment is present' do
let(:attachment_test_case) { build(:test_case, :failed_with_attachment) } let_it_be(:job) { create(:ci_build) }
let(:attachment_test_case) { build(:test_case, :failed_with_attachment, job: job) }
it "initializes the attachment if present" do it "initializes the attachment if present" do
expect(attachment_test_case.attachment).to eq("some/path.png") expect(attachment_test_case.attachment).to eq("some/path.png")
......
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