Commit 220b9d7b authored by Robert Speicher's avatar Robert Speicher

Merge branch 'consider-added-specs-in-danger-checks' into 'master'

Also count specs in new files in Danger check

See merge request gitlab-org/gitlab-ce!22543
parents 821e3959 0e616d71
# frozen_string_literal: true
NO_SPECS_LABELS = %w[backstage Documentation QA].freeze NO_SPECS_LABELS = %w[backstage Documentation QA].freeze
NO_NEW_SPEC_MESSAGE = <<~MSG.freeze NO_NEW_SPEC_MESSAGE = <<~MSG.freeze
You've made some app changes, but didn't add any tests. You've made some app changes, but didn't add any tests.
...@@ -9,8 +11,8 @@ def presented_no_changelog_labels ...@@ -9,8 +11,8 @@ def presented_no_changelog_labels
NO_SPECS_LABELS.map { |label| "~#{label}" }.join(', ') NO_SPECS_LABELS.map { |label| "~#{label}" }.join(', ')
end end
has_app_changes = !git.modified_files.grep(%r{\A(ee/)?(app|lib|db/(geo/)?(post_)?migrate)/}).empty? has_app_changes = !helper.all_changed_files.grep(%r{\A(ee/)?(app|lib|db/(geo/)?(post_)?migrate)/}).empty?
has_spec_changes = !git.modified_files.grep(%r{\A(ee/)?spec/}).empty? has_spec_changes = !helper.all_changed_files.grep(%r{\A(ee/)?spec/}).empty?
new_specs_needed = (gitlab.mr_labels & NO_SPECS_LABELS).empty? new_specs_needed = (gitlab.mr_labels & NO_SPECS_LABELS).empty?
if has_app_changes && !has_spec_changes && new_specs_needed if has_app_changes && !has_spec_changes && new_specs_needed
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment