diff options
Diffstat (limited to 'qa/qa/specs/features/api/4_verify/remove_runner_spec.rb')
| -rw-r--r-- | qa/qa/specs/features/api/4_verify/remove_runner_spec.rb | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/qa/qa/specs/features/api/4_verify/remove_runner_spec.rb b/qa/qa/specs/features/api/4_verify/remove_runner_spec.rb index eb1b085c35c..7aaaa7137ed 100644 --- a/qa/qa/specs/features/api/4_verify/remove_runner_spec.rb +++ b/qa/qa/specs/features/api/4_verify/remove_runner_spec.rb @@ -17,17 +17,16 @@ module QA # Removing a runner via the UI is covered by `spec/features/runners_spec.rb`` it 'removes the runner', testcase: 'https://gitlab.com/gitlab-org/gitlab/-/quality/test_cases/354828' do - runners = nil - expect { (runners = runner.list_of_runners(tag_list: runner_tags)).size } - .to eventually_eq(1).within(max_duration: 10, sleep_interval: 1) - expect(runners.first[:description]).to eq(executor) + runners_list = runner.runners_list + expect(runners_list.size).to eq(1) + expect(runners_list.first[:description]).to eq(executor) - request = Runtime::API::Request.new(api_client, "runners/#{runners.first[:id]}") + request = Runtime::API::Request.new(api_client, "runners/#{runner.id}") response = delete(request.url) expect(response.code).to eq(Support::API::HTTP_STATUS_NO_CONTENT) expect(response.body).to be_empty - expect(runner.list_of_runners(tag_list: runner_tags)).to be_empty + expect(runner).to be_not_found_by_tags end end end |