diff options
author | Thong Kuah <tkuah@gitlab.com> | 2019-04-12 10:27:19 +0000 |
---|---|---|
committer | James Lopez <james@gitlab.com> | 2019-04-12 10:27:19 +0000 |
commit | f38f160a1b0212d5ce0c0402ae07011466753f44 (patch) | |
tree | e583e5a13158b0e3e7b8d7dcb5d03385ec7c7704 | |
parent | ca8c35285eedb0ae6e9a52fe377ec0b3ae9ada1a (diff) | |
download | gitlab-ce-f38f160a1b0212d5ce0c0402ae07011466753f44.tar.gz |
Enable RSpec/ExampleWording cop
-rw-r--r-- | Gemfile | 2 | ||||
-rw-r--r-- | Gemfile.lock | 4 | ||||
-rw-r--r-- | qa/spec/specs/runner_spec.rb | 4 | ||||
-rw-r--r-- | spec/requests/api/pipelines_spec.rb | 6 |
4 files changed, 9 insertions, 7 deletions
@@ -352,7 +352,7 @@ group :development, :test do gem 'spring', '~> 2.0.0' gem 'spring-commands-rspec', '~> 1.0.4' - gem 'gitlab-styles', '~> 2.4', require: false + gem 'gitlab-styles', '~> 2.5', require: false # Pin these dependencies, otherwise a new rule could break the CI pipelines gem 'rubocop', '~> 0.54.0' gem 'rubocop-rspec', '~> 1.22.1' diff --git a/Gemfile.lock b/Gemfile.lock index 3314a769949..baa7be6258a 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -289,7 +289,7 @@ GEM gitlab-markup (1.7.0) gitlab-sidekiq-fetcher (0.4.0) sidekiq (~> 5) - gitlab-styles (2.5.1) + gitlab-styles (2.5.2) rubocop (~> 0.54.0) rubocop-gitlab-security (~> 0.1.0) rubocop-rspec (~> 1.19) @@ -1025,7 +1025,7 @@ DEPENDENCIES gitlab-default_value_for (~> 3.1.1) gitlab-markup (~> 1.7.0) gitlab-sidekiq-fetcher (~> 0.4.0) - gitlab-styles (~> 2.4) + gitlab-styles (~> 2.5) gitlab_omniauth-ldap (~> 2.1.1) gon (~> 6.2) google-api-client (~> 0.23) diff --git a/qa/spec/specs/runner_spec.rb b/qa/spec/specs/runner_spec.rb index 741821ddf8c..5c86c102105 100644 --- a/qa/spec/specs/runner_spec.rb +++ b/qa/spec/specs/runner_spec.rb @@ -69,7 +69,7 @@ describe QA::Specs::Runner do subject { described_class.new } - it 'it includes default args and excludes the skip_signup_disabled tag' do + it 'includes default args and excludes the skip_signup_disabled tag' do expect_rspec_runner_arguments(['--tag', '~orchestrated', '--tag', '~skip_signup_disabled', *described_class::DEFAULT_TEST_PATH_ARGS]) subject.perform @@ -83,7 +83,7 @@ describe QA::Specs::Runner do subject { described_class.new } - it 'it includes default args and excludes the requires_git_protocol_v2 tag' do + it 'includes default args and excludes the requires_git_protocol_v2 tag' do expect_rspec_runner_arguments(['--tag', '~orchestrated', '--tag', '~requires_git_protocol_v2', *described_class::DEFAULT_TEST_PATH_ARGS]) subject.perform diff --git a/spec/requests/api/pipelines_spec.rb b/spec/requests/api/pipelines_spec.rb index 0d46463312b..26158231444 100644 --- a/spec/requests/api/pipelines_spec.rb +++ b/spec/requests/api/pipelines_spec.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'spec_helper' describe API::Pipelines do @@ -493,7 +495,7 @@ describe API::Pipelines do context 'pipeline created is not created by the developer user' do let(:api_user) { create(:user) } - it 'should not return pipeline variables' do + it 'does not return pipeline variables' do subject expect(response).to have_gitlab_http_status(403) @@ -502,7 +504,7 @@ describe API::Pipelines do end context 'user is not a project member' do - it 'should not return pipeline variables' do + it 'does not return pipeline variables' do get api("/projects/#{project.id}/pipelines/#{pipeline.id}/variables", non_member) expect(response).to have_gitlab_http_status(404) |