Skip to content

Skip issue reporting for features/sanity specs

George Koltsov requested to merge skip-sanity-spec-reporting into master

What does this MR do and why?

Skips reporting if the spec is in qa/specs/features/sanity/ as these don't test the app but the framework. These are skipped with the SKIP_REPORT_IN_ISSUES variable in sanity jobs but it needs a further check in other jobs.

Fixes an error in failure issue reporting:

`partial_file_path': undefined method `[]' for nil:NilClass (NoMethodError)

Ex: https://gitlab.com/gitlab-org/quality/nightly/-/jobs/3281887773

We only expect api or browser_ui in the file path, not sanity.

MR acceptance checklist

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

Edited by George Koltsov

Merge request reports