296 lines
13 KiB
Ruby
296 lines
13 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rspec-parameterized'
|
|
require 'gitlab-dangerfiles'
|
|
require 'danger'
|
|
require 'danger/plugins/helper'
|
|
require 'gitlab/dangerfiles/spec_helper'
|
|
|
|
require_relative '../../../danger/plugins/project_helper'
|
|
|
|
RSpec.describe Tooling::Danger::ProjectHelper do
|
|
include_context "with dangerfile"
|
|
|
|
let(:fake_danger) { DangerSpecHelper.fake_danger.include(described_class) }
|
|
let(:fake_helper) { Danger::Helper.new(project_helper) }
|
|
|
|
subject(:project_helper) { fake_danger.new(git: fake_git) }
|
|
|
|
before do
|
|
allow(project_helper).to receive(:helper).and_return(fake_helper)
|
|
end
|
|
|
|
describe '#changes' do
|
|
it 'returns an array of Change objects' do
|
|
expect(project_helper.changes).to all(be_an(Gitlab::Dangerfiles::Change))
|
|
end
|
|
|
|
it 'groups changes by change type' do
|
|
changes = project_helper.changes
|
|
|
|
expect(changes.added.files).to eq(added_files)
|
|
expect(changes.modified.files).to eq(modified_files)
|
|
expect(changes.deleted.files).to eq(deleted_files)
|
|
expect(changes.renamed_before.files).to eq([renamed_before_file])
|
|
expect(changes.renamed_after.files).to eq([renamed_after_file])
|
|
end
|
|
end
|
|
|
|
describe '#categories_for_file' do
|
|
using RSpec::Parameterized::TableSyntax
|
|
|
|
before do
|
|
allow(fake_git).to receive(:diff_for_file).with('usage_data.rb') { double(:diff, patch: "+ count(User.active)") }
|
|
end
|
|
|
|
where(:path, :expected_categories) do
|
|
'usage_data.rb' | [:database, :backend, :product_intelligence]
|
|
'doc/foo.md' | [:docs]
|
|
'CONTRIBUTING.md' | [:docs]
|
|
'LICENSE' | [:docs]
|
|
'MAINTENANCE.md' | [:docs]
|
|
'PHILOSOPHY.md' | [:docs]
|
|
'PROCESS.md' | [:docs]
|
|
'README.md' | [:docs]
|
|
|
|
'ee/doc/foo' | [:unknown]
|
|
'ee/README' | [:unknown]
|
|
|
|
'app/assets/foo' | [:frontend]
|
|
'app/views/foo' | [:frontend]
|
|
'public/foo' | [:frontend]
|
|
'scripts/frontend/foo' | [:frontend]
|
|
'spec/frontend/bar' | [:frontend]
|
|
'spec/frontend_integration/bar' | [:frontend]
|
|
'vendor/assets/foo' | [:frontend]
|
|
'babel.config.js' | [:frontend]
|
|
'jest.config.js' | [:frontend]
|
|
'package.json' | [:frontend]
|
|
'yarn.lock' | [:frontend]
|
|
'config/foo.js' | [:frontend]
|
|
'config/deep/foo.js' | [:frontend]
|
|
|
|
'ee/app/assets/foo' | [:frontend]
|
|
'ee/app/views/foo' | [:frontend]
|
|
'ee/spec/frontend/bar' | [:frontend]
|
|
'ee/spec/frontend_integration/bar' | [:frontend]
|
|
|
|
'.gitlab/ci/frontend.gitlab-ci.yml' | %i[frontend tooling]
|
|
|
|
'app/models/foo' | [:backend]
|
|
'bin/foo' | [:backend]
|
|
'config/foo' | [:backend]
|
|
'lib/foo' | [:backend]
|
|
'rubocop/foo' | [:backend]
|
|
'.rubocop.yml' | [:backend]
|
|
'.rubocop_todo.yml' | [:backend]
|
|
'.rubocop_manual_todo.yml' | [:backend]
|
|
'spec/foo' | [:backend]
|
|
'spec/foo/bar' | [:backend]
|
|
|
|
'ee/app/foo' | [:backend]
|
|
'ee/bin/foo' | [:backend]
|
|
'ee/spec/foo' | [:backend]
|
|
'ee/spec/foo/bar' | [:backend]
|
|
|
|
'spec/migrations/foo' | [:database]
|
|
'ee/spec/migrations/foo' | [:database]
|
|
|
|
'spec/features/foo' | [:test]
|
|
'ee/spec/features/foo' | [:test]
|
|
'spec/support/shared_examples/features/foo' | [:test]
|
|
'ee/spec/support/shared_examples/features/foo' | [:test]
|
|
'spec/support/shared_contexts/features/foo' | [:test]
|
|
'ee/spec/support/shared_contexts/features/foo' | [:test]
|
|
'spec/support/helpers/features/foo' | [:test]
|
|
'ee/spec/support/helpers/features/foo' | [:test]
|
|
|
|
'generator_templates/foo' | [:backend]
|
|
'vendor/languages.yml' | [:backend]
|
|
'file_hooks/examples/' | [:backend]
|
|
|
|
'Gemfile' | [:backend]
|
|
'Gemfile.lock' | [:backend]
|
|
'Rakefile' | [:backend]
|
|
'FOO_VERSION' | [:backend]
|
|
|
|
'Dangerfile' | [:tooling]
|
|
'danger/bundle_size/Dangerfile' | [:tooling]
|
|
'ee/danger/bundle_size/Dangerfile' | [:tooling]
|
|
'danger/bundle_size/' | [:tooling]
|
|
'ee/danger/bundle_size/' | [:tooling]
|
|
'.gitlab-ci.yml' | [:tooling]
|
|
'.gitlab/ci/cng.gitlab-ci.yml' | [:tooling]
|
|
'.gitlab/ci/ee-specific-checks.gitlab-ci.yml' | [:tooling]
|
|
'scripts/foo' | [:tooling]
|
|
'tooling/danger/foo' | [:tooling]
|
|
'ee/tooling/danger/foo' | [:tooling]
|
|
'lefthook.yml' | [:tooling]
|
|
'.editorconfig' | [:tooling]
|
|
'tooling/bin/find_foss_tests' | [:tooling]
|
|
'.codeclimate.yml' | [:tooling]
|
|
'.gitlab/CODEOWNERS' | [:tooling]
|
|
|
|
'lib/gitlab/ci/templates/Security/SAST.gitlab-ci.yml' | [:ci_template]
|
|
'lib/gitlab/ci/templates/dotNET-Core.yml' | [:ci_template]
|
|
|
|
'ee/FOO_VERSION' | [:unknown]
|
|
|
|
'db/schema.rb' | [:database]
|
|
'db/structure.sql' | [:database]
|
|
'db/migrate/foo' | [:database, :migration]
|
|
'db/post_migrate/foo' | [:database, :migration]
|
|
'ee/db/geo/migrate/foo' | [:database, :migration]
|
|
'ee/db/geo/post_migrate/foo' | [:database, :migration]
|
|
'app/models/project_authorization.rb' | [:database, :backend]
|
|
'app/services/users/refresh_authorized_projects_service.rb' | [:database, :backend]
|
|
'app/services/authorized_project_update/find_records_due_for_refresh_service.rb' | [:database, :backend]
|
|
'lib/gitlab/background_migration.rb' | [:database, :backend]
|
|
'lib/gitlab/background_migration/foo' | [:database, :backend]
|
|
'ee/lib/gitlab/background_migration/foo' | [:database, :backend]
|
|
'lib/gitlab/database.rb' | [:database, :backend]
|
|
'lib/gitlab/database/foo' | [:database, :backend]
|
|
'ee/lib/gitlab/database/foo' | [:database, :backend]
|
|
'lib/gitlab/github_import.rb' | [:database, :backend]
|
|
'lib/gitlab/github_import/foo' | [:database, :backend]
|
|
'lib/gitlab/sql/foo' | [:database, :backend]
|
|
'rubocop/cop/migration/foo' | [:database]
|
|
|
|
'db/fixtures/foo.rb' | [:backend]
|
|
'ee/db/fixtures/foo.rb' | [:backend]
|
|
|
|
'qa/foo' | [:qa]
|
|
'ee/qa/foo' | [:qa]
|
|
|
|
'workhorse/main.go' | [:workhorse]
|
|
'workhorse/internal/upload/upload.go' | [:workhorse]
|
|
|
|
'locale/gitlab.pot' | [:none]
|
|
|
|
'FOO' | [:unknown]
|
|
'foo' | [:unknown]
|
|
|
|
'foo/bar.rb' | [:backend]
|
|
'foo/bar.js' | [:frontend]
|
|
'foo/bar.txt' | [:none]
|
|
'foo/bar.md' | [:none]
|
|
|
|
'ee/config/metrics/counts_7d/20210216174919_g_analytics_issues_weekly.yml' | [:product_intelligence]
|
|
'lib/gitlab/usage_data_counters/aggregated_metrics/common.yml' | [:product_intelligence]
|
|
'lib/gitlab/usage_data_counters/hll_redis_counter.rb' | [:backend, :product_intelligence]
|
|
'lib/gitlab/tracking.rb' | [:backend, :product_intelligence]
|
|
'spec/lib/gitlab/tracking_spec.rb' | [:backend, :product_intelligence]
|
|
'app/helpers/tracking_helper.rb' | [:backend, :product_intelligence]
|
|
'spec/helpers/tracking_helper_spec.rb' | [:backend, :product_intelligence]
|
|
'lib/generators/rails/usage_metric_definition_generator.rb' | [:backend, :product_intelligence]
|
|
'spec/lib/generators/usage_metric_definition_generator_spec.rb' | [:backend, :product_intelligence]
|
|
'config/metrics/schema.json' | [:product_intelligence]
|
|
'app/assets/javascripts/tracking/foo.js' | [:frontend, :product_intelligence]
|
|
'spec/frontend/tracking/foo.js' | [:frontend, :product_intelligence]
|
|
'spec/frontend/tracking_spec.js' | [:frontend, :product_intelligence]
|
|
'lib/gitlab/usage_database/foo.rb' | [:backend]
|
|
end
|
|
|
|
with_them do
|
|
subject { project_helper.categories_for_file(path) }
|
|
|
|
it { is_expected.to eq(expected_categories) }
|
|
end
|
|
|
|
context 'having specific changes' do
|
|
where(:expected_categories, :patch, :changed_files) do
|
|
[:database, :backend, :product_intelligence] | '+ count(User.active)' | ['usage_data.rb', 'lib/gitlab/usage_data.rb', 'ee/lib/ee/gitlab/usage_data.rb']
|
|
[:database, :backend, :product_intelligence] | '+ estimate_batch_distinct_count(User.active)' | ['usage_data.rb']
|
|
[:backend, :product_intelligence] | '+ alt_usage_data(User.active)' | ['lib/gitlab/usage_data.rb']
|
|
[:backend, :product_intelligence] | '+ count(User.active)' | ['lib/gitlab/usage_data/topology.rb']
|
|
[:backend, :product_intelligence] | '+ foo_count(User.active)' | ['lib/gitlab/usage_data.rb']
|
|
[:backend] | '+ count(User.active)' | ['user.rb']
|
|
end
|
|
|
|
with_them do
|
|
it 'has the correct categories' do
|
|
changed_files.each do |file|
|
|
allow(fake_git).to receive(:diff_for_file).with(file) { double(:diff, patch: patch) }
|
|
|
|
expect(project_helper.categories_for_file(file)).to eq(expected_categories)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '.local_warning_message' do
|
|
it 'returns an informational message with rules that can run' do
|
|
expect(described_class.local_warning_message).to eq('==> Only the following Danger rules can be run locally: changelog, database, documentation, duplicate_yarn_dependencies, eslint, gitaly, pajamas, pipeline, prettier, product_intelligence, utility_css, vue_shared_documentation')
|
|
end
|
|
end
|
|
|
|
describe '.success_message' do
|
|
it 'returns an informational success message' do
|
|
expect(described_class.success_message).to eq('==> No Danger rule violations!')
|
|
end
|
|
end
|
|
|
|
describe '#rule_names' do
|
|
context 'when running locally' do
|
|
before do
|
|
expect(fake_helper).to receive(:ci?).and_return(false)
|
|
end
|
|
|
|
it 'returns local only rules' do
|
|
expect(project_helper.rule_names).to match_array(described_class::LOCAL_RULES)
|
|
end
|
|
end
|
|
|
|
context 'when running under CI' do
|
|
before do
|
|
expect(fake_helper).to receive(:ci?).and_return(true)
|
|
end
|
|
|
|
it 'returns all rules' do
|
|
expect(project_helper.rule_names).to eq(described_class::LOCAL_RULES | described_class::CI_ONLY_RULES)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#all_ee_changes' do
|
|
subject { project_helper.all_ee_changes }
|
|
|
|
it 'returns all changed files starting with ee/' do
|
|
changes = double
|
|
expect(project_helper).to receive(:changes).and_return(changes)
|
|
expect(changes).to receive(:files).and_return(%w[fr/ee/beer.rb ee/wine.rb ee/lib/ido.rb ee.k])
|
|
|
|
is_expected.to match_array(%w[ee/wine.rb ee/lib/ido.rb])
|
|
end
|
|
end
|
|
|
|
describe '#project_name' do
|
|
subject { project_helper.project_name }
|
|
|
|
it 'returns gitlab if ee? returns true' do
|
|
expect(project_helper).to receive(:ee?) { true }
|
|
|
|
is_expected.to eq('gitlab')
|
|
end
|
|
|
|
it 'returns gitlab-ce if ee? returns false' do
|
|
expect(project_helper).to receive(:ee?) { false }
|
|
|
|
is_expected.to eq('gitlab-foss')
|
|
end
|
|
end
|
|
|
|
describe '#file_lines' do
|
|
let(:filename) { 'spec/foo_spec.rb' }
|
|
let(:file_spy) { spy }
|
|
|
|
it 'returns the chomped file lines' do
|
|
expect(project_helper).to receive(:read_file).with(filename).and_return(file_spy)
|
|
|
|
project_helper.file_lines(filename)
|
|
|
|
expect(file_spy).to have_received(:lines).with(chomp: true)
|
|
end
|
|
end
|
|
end
|