diff options
author | Kamil Trzcinski <ayufan@ayufan.eu> | 2017-12-05 15:22:47 +0100 |
---|---|---|
committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2017-12-05 15:22:47 +0100 |
commit | e05c1e90e9eca705ecbae13d685d674eeb8f5fb4 (patch) | |
tree | 8396cadd6fed681f72625c4e265a70c531959c83 | |
parent | c871e6970c9ae85ffeac418df430a29307b0b865 (diff) | |
download | gitlab-ce-perform-sql-matching-of-tags.tar.gz |
Fix rspecs, as we need tag_list to be persistedperform-sql-matching-of-tags
-rw-r--r-- | spec/services/ci/register_job_service_spec.rb | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/spec/services/ci/register_job_service_spec.rb b/spec/services/ci/register_job_service_spec.rb index dd36e9dada9..decdd577226 100644 --- a/spec/services/ci/register_job_service_spec.rb +++ b/spec/services/ci/register_job_service_spec.rb @@ -15,16 +15,14 @@ module Ci describe '#execute' do context 'runner follow tag list' do it "picks build with the same tag" do - pending_job.tag_list = ["linux"] - pending_job.save - specific_runner.tag_list = ["linux"] + pending_job.update(tag_list: ["linux"]) + specific_runner.update(tag_list: ["linux"]) expect(execute(specific_runner)).to eq(pending_job) end it "does not pick build with different tag" do - pending_job.tag_list = ["linux"] - pending_job.save - specific_runner.tag_list = ["win32"] + pending_job.update(tag_list: ["linux"]) + specific_runner.update(tag_list: ["win32"]) expect(execute(specific_runner)).to be_falsey end @@ -33,13 +31,12 @@ module Ci end it "does not pick build with tag" do - pending_job.tag_list = ["linux"] - pending_job.save + pending_job.update(tag_list: ["linux"]) expect(execute(specific_runner)).to be_falsey end it "pick build without tag" do - specific_runner.tag_list = ["win32"] + specific_runner.update(tag_list: ["win32"]) expect(execute(specific_runner)).to eq(pending_job) end end |