Commit acfe25ed authored by Tomasz Maczukin's avatar Tomasz Maczukin

Refactorize `ci_runner` factory and `let` definitions in runners API spec

parent e4d2f997
...@@ -26,13 +26,11 @@ FactoryGirl.define do ...@@ -26,13 +26,11 @@ FactoryGirl.define do
end end
platform "darwin" platform "darwin"
is_shared false
active true
factory :ci_shared_runner do trait :shared do
is_shared true is_shared true
end end
factory :ci_specific_runner do
is_shared false
end
end end
end end
require 'spec_helper' require 'spec_helper'
describe API::API, api: true do describe API::Runners, api: true do
include ApiHelpers include ApiHelpers
let(:admin) { create(:user, :admin) } let(:admin) { create(:user, :admin) }
...@@ -10,20 +10,20 @@ describe API::API, api: true do ...@@ -10,20 +10,20 @@ describe API::API, api: true do
let(:project) { create(:project, creator_id: user.id) } let(:project) { create(:project, creator_id: user.id) }
let(:project2) { create(:project, creator_id: user.id) } let(:project2) { create(:project, creator_id: user.id) }
let!(:shared_runner) { create(:ci_shared_runner, tag_list: ['mysql', 'ruby'], active: true) } let!(:shared_runner) { create(:ci_runner, :shared) }
let!(:unused_specific_runner) { create(:ci_specific_runner) } let!(:unused_specific_runner) { create(:ci_runner) }
let!(:specific_runner) do let!(:specific_runner) do
runner = create(:ci_specific_runner, tag_list: ['mysql', 'ruby']) create(:ci_runner).tap do |runner|
create(:ci_runner_project, runner: runner, project: project) create(:ci_runner_project, runner: runner, project: project)
runner end
end end
let!(:two_projects_runner) do let!(:two_projects_runner) do
runner = create(:ci_specific_runner) create(:ci_runner).tap do |runner|
create(:ci_runner_project, runner: runner, project: project) create(:ci_runner_project, runner: runner, project: project)
create(:ci_runner_project, runner: runner, project: project2) create(:ci_runner_project, runner: runner, project: project2)
runner end
end end
before do before do
...@@ -352,14 +352,12 @@ describe API::API, api: true do ...@@ -352,14 +352,12 @@ describe API::API, api: true do
end end
describe 'POST /projects/:id/runners' do describe 'POST /projects/:id/runners' do
let(:specific_runner2) do context 'authorized user' do
runner = create(:ci_specific_runner) it 'should enable specific runner' do
specific_runner2 = create(:ci_runner).tap do |runner|
create(:ci_runner_project, runner: runner, project: project2) create(:ci_runner_project, runner: runner, project: project2)
runner
end end
context 'authorized user' do
it 'should enable specific runner' do
expect do expect do
post api("/projects/#{project.id}/runners", user), runner_id: specific_runner2.id post api("/projects/#{project.id}/runners", user), runner_id: specific_runner2.id
end.to change{ project.runners.count }.by(+1) end.to change{ project.runners.count }.by(+1)
......
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