Skip to content
Snippets Groups Projects
Commit 784401ac authored by Andrejs Cunskis's avatar Andrejs Cunskis :palm_tree:
Browse files

Merge branch 'hm-omit-deleted-specs-from-detected-changes' into 'master'

Omit deleted files from changed specs from QA_TESTS

See merge request !134592



Merged-by: default avatarAndrejs Cunskis <acunskis@gitlab.com>
Approved-by: default avatarAndrejs Cunskis <acunskis@gitlab.com>
Reviewed-by: default avatarAndrejs Cunskis <acunskis@gitlab.com>
Co-authored-by: default avatarhmuralidhar <hmuralidhar@gitlab.com>
parents 22d29cbb 2c24d86c
No related branches found
No related tags found
1 merge request!134592Omit deleted files from changed specs from QA_TESTS
Pipeline #1043799052 passed
Pipeline: E2E Omnibus GitLab EE

#1043892869

    Pipeline: GitLab

    #1043804229

      Pipeline: E2E GDK

      #1043802460

        +21
        ......@@ -29,10 +29,16 @@ def initialize(mr_diff, mr_labels, additional_group_spec_list)
        # @return [String]
        def qa_tests
        return if mr_diff.empty? || dependency_changes
        return if only_spec_changes? && mr_diff.all? { |change| change[:deleted_file] }
        # make paths relative to qa directory
        return changed_files&.map { |path| path.delete_prefix("qa/") }&.join(" ") if only_spec_changes?
        return qa_spec_directories_for_devops_stage&.join(" ") if non_qa_changes? && mr_labels.any?
        if only_spec_changes?
        return mr_diff
        .reject { |change| change[:deleted_file] }
        .map { |change| change[:path].delete_prefix("qa/") } # make paths relative to qa directory
        .join(" ")
        end
        qa_spec_directories_for_devops_stage&.join(" ") if non_qa_changes? && mr_labels.any?
        end
        # Qa framework changes
        ......
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Finish editing this message first!
        Please register or to comment