diff options
author | Douwe Maan <douwe@selenight.nl> | 2017-07-04 18:57:13 -0500 |
---|---|---|
committer | Douwe Maan <douwe@selenight.nl> | 2017-07-05 11:12:00 -0500 |
commit | 873b40555b582638992e7cdeb2081c29d08cae05 (patch) | |
tree | e0bef2f35e1c5a44c83b7680dfa1d4c05797048d /spec/rubocop | |
parent | fe13f110412d85c05dc68e5ee1db499f681bf722 (diff) | |
download | gitlab-ce-873b40555b582638992e7cdeb2081c29d08cae05.tar.gz |
Add ProjectPathHelper cop
Diffstat (limited to 'spec/rubocop')
-rw-r--r-- | spec/rubocop/cop/project_path_helper_spec.rb | 41 |
1 files changed, 41 insertions, 0 deletions
diff --git a/spec/rubocop/cop/project_path_helper_spec.rb b/spec/rubocop/cop/project_path_helper_spec.rb new file mode 100644 index 00000000000..bc47b45cad7 --- /dev/null +++ b/spec/rubocop/cop/project_path_helper_spec.rb @@ -0,0 +1,41 @@ +require 'spec_helper' + +require 'rubocop' +require 'rubocop/rspec/support' + +require_relative '../../../rubocop/cop/project_path_helper' + +describe RuboCop::Cop::ProjectPathHelper do + include CopHelper + + subject(:cop) { described_class.new } + + context "when using namespace_project with the project's namespace" do + let(:source) { 'edit_namespace_project_issue_path(@issue.project.namespace, @issue.project, @issue)' } + let(:correct_source) { 'edit_project_issue_path(@issue.project, @issue)' } + + it 'registers an offense' do + inspect_source(cop, source) + + aggregate_failures do + expect(cop.offenses.size).to eq(1) + expect(cop.offenses.map(&:line)).to eq([1]) + expect(cop.highlights).to eq(['edit_namespace_project_issue_path']) + end + end + + it 'autocorrects to the right version' do + autocorrected = autocorrect_source(cop, source) + + expect(autocorrected).to eq(correct_source) + end + end + + context 'when using namespace_project with a different namespace' do + it 'registers no offense' do + inspect_source(cop, 'edit_namespace_project_issue_path(namespace, project)') + + expect(cop.offenses.size).to eq(0) + end + end +end |