Commit aa1ab8aa authored by Grzegorz Bizon's avatar Grzegorz Bizon

Add some tests for updating ci runner information

parent 9a8e4863
...@@ -35,7 +35,8 @@ describe Ci::API::API do ...@@ -35,7 +35,8 @@ describe Ci::API::API do
end end
end end
it "starts a build" do context 'when there is a pending build' do
it 'starts a build' do
register_builds info: { platform: :darwin } register_builds info: { platform: :darwin }
expect(response).to have_http_status(201) expect(response).to have_http_status(201)
...@@ -49,6 +50,11 @@ describe Ci::API::API do ...@@ -49,6 +50,11 @@ describe Ci::API::API do
) )
end end
it 'updates runner info' do
expect { register_builds }.to change { runner.reload.contacted_at }
end
end
context 'when builds are finished' do context 'when builds are finished' do
before do before do
build.success build.success
...@@ -159,13 +165,18 @@ describe Ci::API::API do ...@@ -159,13 +165,18 @@ describe Ci::API::API do
end end
context 'when runner is paused' do context 'when runner is paused' do
let(:inactive_runner) { create(:ci_runner, :inactive, token: "InactiveRunner") } let(:runner) { create(:ci_runner, :inactive, token: 'InactiveRunner') }
before do it 'responds with 404' do
register_builds inactive_runner.token register_builds
expect(response).to have_http_status 404
end end
it { expect(response).to have_http_status 404 } it 'does not update runner info' do
expect { register_builds }
.not_to change { runner.reload.contacted_at }
end
end end
def register_builds(token = runner.token, **params) def register_builds(token = runner.token, **params)
......
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