diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-01-20 15:08:53 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-01-20 15:08:53 +0000 |
commit | 709948b7a69597b1efe24df9b0f388cc0b493dd9 (patch) | |
tree | a6cbb0b7a1243c5308f8d5afb703d1980edb4595 /danger | |
parent | 50ea04b6c6823aa1bd8d64cd9a77dcbd03b19053 (diff) | |
download | gitlab-ce-709948b7a69597b1efe24df9b0f388cc0b493dd9.tar.gz |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'danger')
-rw-r--r-- | danger/qa_selector/Dangerfile | 29 |
1 files changed, 25 insertions, 4 deletions
diff --git a/danger/qa_selector/Dangerfile b/danger/qa_selector/Dangerfile index a20e626c694..b781f390a1a 100644 --- a/danger/qa_selector/Dangerfile +++ b/danger/qa_selector/Dangerfile @@ -1,9 +1,13 @@ # frozen_string_literal: true -def filter_changed_lines(files) +data_qa_selectors = /qa_selector|data-qa-selector/ + +deprecated_qa_selectors = /(?!.*\bdata-qa-)(?=class=.*qa-.*|class: .*qa-.*)/ + +def filter_changed_lines(files, pattern) lines = [] files.each do |file| - qa_selector_changed_lines = helper.changed_lines(file).select { |line| line =~ /qa_selector|data-qa-selector/ } + qa_selector_changed_lines = helper.changed_lines(file).select { |line| line =~ pattern } next unless qa_selector_changed_lines.any? lines += ["file `#{file}`:", qa_selector_changed_lines] @@ -15,9 +19,11 @@ changed_code_files = helper.changed_files(/\.(vue|haml|js|rb)$/) return if changed_code_files.empty? -lines_with_qa_selectors = filter_changed_lines(changed_code_files) +lines_with_qa_selectors = filter_changed_lines(changed_code_files, data_qa_selectors) + +deprecated_qa_class = filter_changed_lines(changed_code_files, deprecated_qa_selectors) -return if lines_with_qa_selectors.empty? +return if (lines_with_qa_selectors + deprecated_qa_class).empty? markdown(<<~MARKDOWN) ## QA Selectors @@ -40,3 +46,18 @@ if lines_with_qa_selectors.any? warn "This merge request contains lines with QA selectors. Please ensure `e2e:package-and-test` job is run." end + +if deprecated_qa_class.any? + markdown(<<~MARKDOWN) + ### Deprecated .qa-selector class + + The following lines in this MR contain deprecated qa class selectors: + + * #{deprecated_qa_class.join("\n* ")} + + Please ensure all deprecated qa class selectors are replaced with data-qa-selectors in accordance with our [Testing Guide](https://docs.gitlab.com/ee/development/testing_guide/end_to_end/page_objects.html#data-qa-selector-vs-qa-selector). + + MARKDOWN + + warn "This merge request contains deprecated .qa-selector CSS class. Please use data-qa-selector attribute instead." +end |