diff options
author | ddavison <ddavison@gitlab.com> | 2019-07-10 00:44:16 +0300 |
---|---|---|
committer | ddavison <ddavison@gitlab.com> | 2019-07-11 02:10:28 +0300 |
commit | 015b61b1dc73e2c8d6e6cf04b6b2c7fb896a452f (patch) | |
tree | 218b05eb1f0500c01a0b6e7dccd3a2ea433df73b | |
parent | 98fc3f23db648a16096994ba3b70d461b5b1ccf0 (diff) |
Change element_with_pattern cop to not use quotes
Using quotes within string validations can be messy. Let's
use a typical CSS selector for an unquoted attribute
Update the cop spec to validate appropriate new message
-rw-r--r-- | rubocop/cop/qa/element_with_pattern.rb | 2 | ||||
-rw-r--r-- | spec/rubocop/cop/qa/element_with_pattern_spec.rb | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/rubocop/cop/qa/element_with_pattern.rb b/rubocop/cop/qa/element_with_pattern.rb index f7debf13151..d48f4725488 100644 --- a/rubocop/cop/qa/element_with_pattern.rb +++ b/rubocop/cop/qa/element_with_pattern.rb @@ -30,7 +30,7 @@ module RuboCop return if args.first.nil? args.first.each_node(:str) do |arg| - add_offense(arg, message: MESSAGE % "data-qa-selector=\"#{element_name.value}\"") + add_offense(arg, message: MESSAGE % "data-qa-selector=#{element_name.value}") end end diff --git a/spec/rubocop/cop/qa/element_with_pattern_spec.rb b/spec/rubocop/cop/qa/element_with_pattern_spec.rb index ef20d9a1f26..fee390caa9f 100644 --- a/spec/rubocop/cop/qa/element_with_pattern_spec.rb +++ b/spec/rubocop/cop/qa/element_with_pattern_spec.rb @@ -23,9 +23,9 @@ describe RuboCop::Cop::QA::ElementWithPattern do expect_offense(<<-RUBY) view 'app/views/shared/groups/_search_form.html.haml' do element :groups_filter, 'search_field_tag :filter' - ^^^^^^^^^^^^^^^^^^^^^^^^^^ Don't use a pattern for element, create a corresponding `qa-groups-filter` instead. + ^^^^^^^^^^^^^^^^^^^^^^^^^^ Don't use a pattern for element, create a corresponding `data-qa-selector=groups_filter` instead. element :groups_filter_placeholder, /Search by name/ - ^^^^^^^^^^^^^^ Don't use a pattern for element, create a corresponding `qa-groups-filter-placeholder` instead. + ^^^^^^^^^^^^^^ Don't use a pattern for element, create a corresponding `data-qa-selector=groups_filter_placeholder` instead. end RUBY end |