diff options
author | ddavison <ddavison@gitlab.com> | 2019-05-29 16:47:53 -0700 |
---|---|---|
committer | ddavison <ddavison@gitlab.com> | 2019-07-10 16:09:43 -0700 |
commit | 390e0d8906c066996e8bbb14aa5824643f9798f8 (patch) | |
tree | 8b220d1d2e0770bc96cc958ce624043e1ae0885e /rubocop | |
parent | 48fa47e07d54979b70704beb16ea55548f2190c4 (diff) | |
download | gitlab-ce-390e0d8906c066996e8bbb14aa5824643f9798f8.tar.gz |
Introduce data-qa-selector to supplant .qa-class
In order to break away from using CSS classes as
our primary method of element identification, we
need to provide the ability to search for data
attributes.
Make Test::Sanity::Selectors now work
Utilize regex to match on literal strings of the element name
Suggest the data-qa-selector pattern vs the qa-
Add data-qa-selector to login page to start
We need a page that is heavily used in order to be
confident that this functionality works. Let's start
with the Login page
Use appropriate HAML data tag practices
Diffstat (limited to 'rubocop')
-rw-r--r-- | rubocop/cop/qa/element_with_pattern.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/rubocop/cop/qa/element_with_pattern.rb b/rubocop/cop/qa/element_with_pattern.rb index d14eeaaeaf3..f7debf13151 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 % "qa-#{element_name.value.to_s.tr('_', '-')}") + add_offense(arg, message: MESSAGE % "data-qa-selector=\"#{element_name.value}\"") end end |