Skip to content

Skip remove resources after QA unit tests

Mark Lapierre requested to merge ml-skip-remove-in-unit-tests into master

What does this MR do and why?

Skips removing reusable resources when running QA unit tests, because they don't create any.

Fixes failures as in: https://gitlab.com/gitlab-org/gitlab/-/jobs/2301802593

An error occurred in an `after(:suite)` hook.
Failure/Error: QA::Resource::ReusableCollection.remove_all_via_api! if !suite.reporter.failed_examples.present? && !QA::Runtime::Env.running_on_dot_com?
ArgumentError:
  Empty `gitlab_address` attribute!
# ./qa/runtime/scenario.rb:22:in `block (2 levels) in define'
# ./qa/runtime/scenario.rb:21:in `tap'
# ./qa/runtime/scenario.rb:21:in `block in define'
# ./qa/runtime/env.rb:106:in `running_on_dot_com?'
# ./spec/spec_helper.rb:82:in `block (2 levels) in <top (required)>'

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports