watir 7.1.0 → 7.2.0

data/spec/unit/selector_builder/button_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../unit_helper'
 
 describe Watir::Locators::Button::SelectorBuilder do
@@ -8,9 +10,9 @@ describe Watir::Locators::Button::SelectorBuilder do
   let(:lowercase) { 'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ' }
   let(:default_types) do
     "translate(@type,'#{uppercase}','#{lowercase}')=translate('button','#{uppercase}','#{lowercase}') or " \
-    "translate(@type,'#{uppercase}','#{lowercase}')=translate('reset','#{uppercase}','#{lowercase}') or "\
-    "translate(@type,'#{uppercase}','#{lowercase}')=translate('submit','#{uppercase}','#{lowercase}') or "\
-    "translate(@type,'#{uppercase}','#{lowercase}')=translate('image','#{uppercase}','#{lowercase}')"
+      "translate(@type,'#{uppercase}','#{lowercase}')=translate('reset','#{uppercase}','#{lowercase}') or " \
+      "translate(@type,'#{uppercase}','#{lowercase}')=translate('submit','#{uppercase}','#{lowercase}') or " \
+      "translate(@type,'#{uppercase}','#{lowercase}')=translate('image','#{uppercase}','#{lowercase}')"
   end
 
   describe '#build' do
@@ -30,7 +32,7 @@ describe Watir::Locators::Button::SelectorBuilder do
       it 'true locates button or input with a type' do
         selector = {type: true}
         built = {xpath: ".//*[(local-name()='button' and @type) or " \
-"(local-name()='input' and (#{default_types}))]"}
+                        "(local-name()='input' and (#{default_types}))]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
@@ -38,8 +40,8 @@ describe Watir::Locators::Button::SelectorBuilder do
         selector = {type: 'reset'}
         type = "translate('reset','#{uppercase}','#{lowercase}')"
         built = {xpath: ".//*[(local-name()='button' and " \
-"translate(@type,'#{uppercase}','#{lowercase}')=#{type}) or " \
-"(local-name()='input' and (translate(@type,'#{uppercase}','#{lowercase}')=#{type}))]"}
+                        "translate(@type,'#{uppercase}','#{lowercase}')=#{type}) or " \
+                        "(local-name()='input' and (translate(@type,'#{uppercase}','#{lowercase}')=#{type}))]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
@@ -63,49 +65,49 @@ describe Watir::Locators::Button::SelectorBuilder do
       it 'locates value of input element with String' do
         selector = {text: 'Button'}
         built = {xpath: ".//*[(local-name()='button' and normalize-space()='Button') or " \
-"(local-name()='input' and (#{default_types}) and @value='Button')]"}
+                        "(local-name()='input' and (#{default_types}) and @value='Button')]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'locates text of button element with String' do
         selector = {text: 'Button 2'}
         built = {xpath: ".//*[(local-name()='button' and normalize-space()='Button 2') or " \
-"(local-name()='input' and (#{default_types}) and @value='Button 2')]"}
+                        "(local-name()='input' and (#{default_types}) and @value='Button 2')]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'locates value of input element with simple Regexp' do
         selector = {text: /Button/}
-        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), 'Button')) or " \
-"(local-name()='input' and (#{default_types}) and contains(@value, 'Button'))]"}
+        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), \"Button\")) or " \
+                        "(local-name()='input' and (#{default_types}) and contains(@value, \"Button\"))]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'locates text of button element with simple Regexp' do
         selector = {text: /Button 2/}
-        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), 'Button 2')) or " \
-"(local-name()='input' and (#{default_types}) and contains(@value, 'Button 2'))]"}
+        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), \"Button 2\")) or " \
+                        "(local-name()='input' and (#{default_types}) and contains(@value, \"Button 2\"))]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'Simple Regexp for text' do
         selector = {text: /n 2/}
-        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), 'n 2')) or " \
-"(local-name()='input' and (#{default_types}) and contains(@value, 'n 2'))]"}
+        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), \"n 2\")) or " \
+                        "(local-name()='input' and (#{default_types}) and contains(@value, \"n 2\"))]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'Simple Regexp for value' do
         selector = {text: /Prev/}
-        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), 'Prev')) or " \
-"(local-name()='input' and (#{default_types}) and contains(@value, 'Prev'))]"}
+        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), \"Prev\")) or " \
+                        "(local-name()='input' and (#{default_types}) and contains(@value, \"Prev\"))]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'returns complex Regexp to the locator' do
         selector = {text: /^foo$/}
-        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), 'foo')) or " \
-"(local-name()='input' and (#{default_types}) and contains(@value, 'foo'))]", text: /^foo$/}
+        built = {xpath: ".//*[(local-name()='button' and contains(normalize-space(), \"foo\")) or " \
+                        "(local-name()='input' and (#{default_types}) and contains(@value, \"foo\"))]", text: /^foo$/}
         expect(selector_builder.build(selector)).to eq built
       end
     end
@@ -114,49 +116,49 @@ describe Watir::Locators::Button::SelectorBuilder do
       it 'input element value with String' do
         selector = {value: 'Preview'}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[normalize-space()='Preview' or @value='Preview']"}
+                        "[normalize-space()='Preview' or @value='Preview']"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'button element value with String' do
         selector = {value: 'button_2'}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[normalize-space()='button_2' or @value='button_2']"}
+                        "[normalize-space()='button_2' or @value='button_2']"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'input element value with simple Regexp' do
         selector = {value: /Prev/}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[contains(normalize-space(), 'Prev') or contains(@value, 'Prev')]"}
+                        '[contains(normalize-space(), "Prev") or contains(@value, "Prev")]'}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'button element value with simple Regexp' do
         selector = {value: /on_2/}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[contains(normalize-space(), 'on_2') or contains(@value, 'on_2')]"}
+                        '[contains(normalize-space(), "on_2") or contains(@value, "on_2")]'}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'button element text with String' do
         selector = {value: 'Button 2'}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[normalize-space()='Button 2' or @value='Button 2']"}
+                        "[normalize-space()='Button 2' or @value='Button 2']"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'button element text with simple Regexp' do
         selector = {value: /ton 2/}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[contains(normalize-space(), 'ton 2') or contains(@value, 'ton 2')]"}
+                        '[contains(normalize-space(), "ton 2") or contains(@value, "ton 2")]'}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'returns complex Regexp to the locator' do
         selector = {value: /^foo$/}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[contains(normalize-space(), 'foo') or contains(@value, 'foo')]", value: /^foo$/}
+                        '[contains(normalize-space(), "foo") or contains(@value, "foo")]', value: /^foo$/}
         expect(selector_builder.build(selector)).to eq built
       end
     end
@@ -171,14 +173,14 @@ describe Watir::Locators::Button::SelectorBuilder do
       it 'negative' do
         selector = {index: -4}
         built = {xpath: "(.//*[(local-name()='button') or " \
-"(local-name()='input' and (#{default_types}))])[last()-3]"}
+                        "(local-name()='input' and (#{default_types}))])[last()-3]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'last' do
         selector = {index: -1}
         built = {xpath: "(.//*[(local-name()='button') or " \
-"(local-name()='input' and (#{default_types}))])[last()]"}
+                        "(local-name()='input' and (#{default_types}))])[last()]"}
         expect(selector_builder.build(selector)).to eq built
       end
 
@@ -199,7 +201,8 @@ describe Watir::Locators::Button::SelectorBuilder do
       it 'locates using class and attributes' do
         selector = {class: 'image', name: 'new_user_image', src: true}
         built = {xpath: ".//*[(local-name()='button') or (local-name()='input' and (#{default_types}))]" \
-"[contains(concat(' ', @class, ' '), ' image ')][@name='new_user_image' and @src]"}
+                        "[contains(concat(' ', normalize-space(@class), ' '), ' image ')]" \
+                        "[@name='new_user_image' and @src]"}
         expect(selector_builder.build(selector)).to eq built
       end
     end

data/spec/unit/selector_builder/cell_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../unit_helper'
 
 describe Watir::Locators::Cell::SelectorBuilder do
@@ -54,7 +56,7 @@ describe Watir::Locators::Cell::SelectorBuilder do
       it 'attribute and text' do
         selector = {headers: /before_tax/, text: '5 934'}
         built = {xpath: "./*[local-name()='th' or local-name()='td']" \
-"[normalize-space()='5 934'][contains(@headers, 'before_tax')]"}
+                        "[normalize-space()='5 934'][contains(@headers, \"before_tax\")]"}
 
         expect(selector_builder.build(selector)).to eq built
       end

data/spec/unit/selector_builder/element_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../unit_helper'
 
 describe Watir::Locators::Element::SelectorBuilder do
@@ -67,7 +69,7 @@ describe Watir::Locators::Element::SelectorBuilder do
 
       it 'with simple Regexp contains' do
         selector = {tag_name: /div/}
-        built = {xpath: ".//*[contains(local-name(), 'div')]"}
+        built = {xpath: './/*[contains(local-name(), "div")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -90,51 +92,51 @@ describe Watir::Locators::Element::SelectorBuilder do
     context 'with class names' do
       it 'class_name is converted to class' do
         selector = {class_name: 'user'}
-        built = {xpath: ".//*[contains(concat(' ', @class, ' '), ' user ')]"}
+        built = {xpath: ".//*[contains(concat(' ', normalize-space(@class), ' '), ' user ')]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'single String concatenates' do
         selector = {class: 'user'}
-        built = {xpath: ".//*[contains(concat(' ', @class, ' '), ' user ')]"}
+        built = {xpath: ".//*[contains(concat(' ', normalize-space(@class), ' '), ' user ')]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'Array of String concatenates with and' do
         selector = {class: %w[multiple here]}
-        built = {xpath: ".//*[contains(concat(' ', @class, ' '), ' multiple ') and " \
-"contains(concat(' ', @class, ' '), ' here ')]"}
+        built = {xpath: ".//*[contains(concat(' ', normalize-space(@class), ' '), ' multiple ') and " \
+                        "contains(concat(' ', normalize-space(@class), ' '), ' here ')]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'merges values when class and class_name are both used' do
         selector = {class: 'foo', class_name: 'bar'}
-        built = {xpath: ".//*[contains(concat(' ', @class, ' '), ' foo ') and " \
-"contains(concat(' ', @class, ' '), ' bar ')]"}
+        built = {xpath: ".//*[contains(concat(' ', normalize-space(@class), ' '), ' foo ') and " \
+                        "contains(concat(' ', normalize-space(@class), ' '), ' bar ')]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'simple Regexp contains' do
         selector = {class_name: /use/}
-        built = {xpath: ".//*[contains(@class, 'use')]"}
+        built = {xpath: './/*[contains(@class, "use")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'Array of Regexp contains with and' do
         selector = {class: [/mult/, /her/]}
-        built = {xpath: ".//*[contains(@class, 'mult') and contains(@class, 'her')]"}
+        built = {xpath: './/*[contains(@class, "mult") and contains(@class, "her")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'single negated String concatenates with not' do
         selector = {class: '!multiple'}
-        built = {xpath: ".//*[not(contains(concat(' ', @class, ' '), ' multiple '))]"}
+        built = {xpath: ".//*[not(contains(concat(' ', normalize-space(@class), ' '), ' multiple '))]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -155,8 +157,9 @@ describe Watir::Locators::Element::SelectorBuilder do
 
       it 'Array of mixed String, Regexp and Boolean contains and concatenates with and and not' do
         selector = {class: [/mult/, 'classes', '!here']}
-        built = {xpath: ".//*[contains(@class, 'mult') and contains(concat(' ', @class, ' '), ' classes ') " \
-"and not(contains(concat(' ', @class, ' '), ' here '))]"}
+        built = {xpath: './/*[contains(@class, "mult") ' \
+                        "and contains(concat(' ', normalize-space(@class), ' '), ' classes ') " \
+                        "and not(contains(concat(' ', normalize-space(@class), ' '), ' here '))]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -251,14 +254,14 @@ describe Watir::Locators::Element::SelectorBuilder do
     context 'with attributes as partials' do
       it 'with Regexp' do
         selector = {name: /user/}
-        built = {xpath: ".//*[contains(@name, 'user')]"}
+        built = {xpath: './/*[contains(@name, "user")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'with multiple Regexp attributes separated by and' do
         selector = {readonly: /read/, id: /good/}
-        built = {xpath: ".//*[contains(@readonly, 'read') and contains(@id, 'good')]"}
+        built = {xpath: './/*[contains(@readonly, "read") and contains(@id, "good")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -274,7 +277,7 @@ describe Watir::Locators::Element::SelectorBuilder do
 
       it 'Regexp uses contains normalize space' do
         selector = {text: /Add/}
-        built = {xpath: ".//*[contains(normalize-space(), 'Add')]"}
+        built = {xpath: './/*[contains(normalize-space(), "Add")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -327,8 +330,8 @@ describe Watir::Locators::Element::SelectorBuilder do
     context 'with labels' do
       it 'locates the element associated with the label element located by the text of the provided label key' do
         selector = {label: 'Cars'}
-        built = {xpath: ".//*[@id=//label[normalize-space()='Cars']/@for "\
-"or parent::label[normalize-space()='Cars']]"}
+        built = {xpath: ".//*[@id=//label[normalize-space()='Cars']/@for " \
+                        "or parent::label[normalize-space()='Cars']]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -390,8 +393,8 @@ describe Watir::Locators::Element::SelectorBuilder do
 
         it 'with multiple locators' do
           selector = {adjacent: :ancestor, id: true, tag_name: 'div', class: 'ancestor', index: 1}
-          built = {xpath: "./ancestor::*[local-name()='div']"\
-"[contains(concat(' ', @class, ' '), ' ancestor ')][@id][2]"}
+          built = {xpath: "./ancestor::*[local-name()='div']" \
+                          "[contains(concat(' ', normalize-space(@class), ' '), ' ancestor ')][@id][2]"}
 
           expect(selector_builder.build(selector)).to eq built
         end
@@ -421,8 +424,8 @@ describe Watir::Locators::Element::SelectorBuilder do
 
         it 'with multiple locators' do
           selector = {adjacent: :following, tag_name: 'div', class: 'b', index: 0, id: true}
-          built = {xpath: "./following-sibling::*[local-name()='div']"\
-"[contains(concat(' ', @class, ' '), ' b ')][@id][1]"}
+          built = {xpath: "./following-sibling::*[local-name()='div']" \
+                          "[contains(concat(' ', normalize-space(@class), ' '), ' b ')][@id][1]"}
 
           expect(selector_builder.build(selector)).to eq built
         end
@@ -452,8 +455,8 @@ describe Watir::Locators::Element::SelectorBuilder do
 
         it 'with multiple locators' do
           selector = {adjacent: :preceding, tag_name: 'div', class: 'b', id: true, index: 0}
-          built = {xpath: "./preceding-sibling::*[local-name()='div']"\
-"[contains(concat(' ', @class, ' '), ' b ')][@id][1]"}
+          built = {xpath: "./preceding-sibling::*[local-name()='div']" \
+                          "[contains(concat(' ', normalize-space(@class), ' '), ' b ')][@id][1]"}
 
           expect(selector_builder.build(selector)).to eq built
         end
@@ -483,8 +486,8 @@ describe Watir::Locators::Element::SelectorBuilder do
 
         it 'with multiple locators' do
           selector = {adjacent: :child, tag_name: 'div', class: 'b', id: true, index: 0}
-          built = {xpath: "./child::*[local-name()='div']"\
-"[contains(concat(' ', @class, ' '), ' b ')][@id][1]"}
+          built = {xpath: "./child::*[local-name()='div']" \
+                          "[contains(concat(' ', normalize-space(@class), ' '), ' b ')][@id][1]"}
 
           expect(selector_builder.build(selector)).to eq built
         end
@@ -501,8 +504,8 @@ describe Watir::Locators::Element::SelectorBuilder do
     context 'with multiple locators' do
       it 'locates using tag name, class, attributes and text' do
         selector = {tag_name: 'div', class: 'content', contenteditable: 'true', text: 'Foo'}
-        built = {xpath: ".//*[local-name()='div'][contains(concat(' ', @class, ' '), ' content ')]" \
-"[normalize-space()='Foo'][@contenteditable='true']"}
+        built = {xpath: ".//*[local-name()='div'][contains(concat(' ', normalize-space(@class), ' '), ' content ')]" \
+                        "[normalize-space()='Foo'][@contenteditable='true']"}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -511,14 +514,14 @@ describe Watir::Locators::Element::SelectorBuilder do
     context 'with simple Regexp' do
       it 'handles spaces' do
         selector = {title: /od Lu/}
-        built = {xpath: ".//*[contains(@title, 'od Lu')]"}
+        built = {xpath: './/*[contains(@title, "od Lu")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'handles escaped characters' do
         selector = {src: %r{ages/but}}
-        built = {xpath: ".//*[contains(@src, 'ages/but')]"}
+        built = {xpath: './/*[contains(@src, "ages/but")]'}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -527,21 +530,21 @@ describe Watir::Locators::Element::SelectorBuilder do
     context 'with complex Regexp' do
       it 'handles wildcards' do
         selector = {src: /ages.*but/}
-        built = {xpath: ".//*[contains(@src, 'ages') and contains(@src, 'but')]", src: /ages.*but/}
+        built = {xpath: './/*[contains(@src, "ages") and contains(@src, "but")]', src: /ages.*but/}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'handles optional characters' do
         selector = {src: /ages ?but/}
-        built = {xpath: ".//*[contains(@src, 'ages') and contains(@src, 'but')]", src: /ages ?but/}
+        built = {xpath: './/*[contains(@src, "ages") and contains(@src, "but")]', src: /ages ?but/}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'handles anchors' do
         selector = {name: /^new_user_image$/}
-        built = {xpath: ".//*[contains(@name, 'new_user_image')]", name: /^new_user_image$/}
+        built = {xpath: './/*[contains(@name, "new_user_image")]', name: /^new_user_image$/}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -555,7 +558,7 @@ describe Watir::Locators::Element::SelectorBuilder do
 
       it 'does not use starts-with if visible locator used' do
         selector = {id: /^vis/, visible_text: 'shown div'}
-        built = {xpath: ".//*[contains(@id, 'vis')]", id: /^vis/, visible_text: 'shown div'}
+        built = {xpath: './/*[contains(@id, "vis")]', id: /^vis/, visible_text: 'shown div'}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -563,11 +566,11 @@ describe Watir::Locators::Element::SelectorBuilder do
       it 'handles case insensitive' do
         selector = {action: /ME/i}
         built = {xpath: './/*[contains(translate(@action,' \
-"'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
-"'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ'), " \
-"translate('me'," \
-"'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
-"'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ'))]"}
+                        "'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
+                        "'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ'), " \
+                        'translate("me",' \
+                        "'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
+                        "'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ'))]"}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -606,14 +609,14 @@ describe Watir::Locators::Element::SelectorBuilder do
     context 'returns locators that can not be directly translated' do
       it 'attribute with complicated Regexp at end' do
         selector = {action: /me$/}
-        built = {xpath: ".//*[contains(@action, 'me')]", action: /me$/}
+        built = {xpath: './/*[contains(@action, "me")]', action: /me$/}
 
         expect(selector_builder.build(selector)).to eq built
       end
 
       it 'class with complicated Regexp' do
         selector = {class: /he?r/}
-        built = {xpath: ".//*[contains(@class, 'h') and contains(@class, 'r')]", class: [/he?r/]}
+        built = {xpath: './/*[contains(@class, "h") and contains(@class, "r")]', class: [/he?r/]}
 
         expect(selector_builder.build(selector)).to eq built
       end
@@ -738,6 +741,7 @@ describe Watir::Locators::Element::SelectorBuilder do
         allow(query_scope).to receive(:selector_builder).and_return(selector_builder)
         allow(query_scope).to receive(:browser).and_return(browser)
         allow(query_scope).to receive(:is_a?).with(Watir::Browser).and_return(false)
+        allow(query_scope).to receive(:is_a?).with(Watir::ShadowRoot).and_return(false)
         allow(query_scope).to receive(:is_a?).with(Watir::IFrame).and_return(true)
 
         selector = {tag_name: 'div'}
@@ -750,48 +754,45 @@ describe Watir::Locators::Element::SelectorBuilder do
     end
 
     context 'with case-insensitive attributes' do
+      let(:upper) { "'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'" }
+      let(:lower) { "'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ'" }
+
       it 'respects case when locating uknown element with uknown attribute' do
         expect(selector_builder.build(hreflang: 'en')).to eq(xpath: ".//*[@hreflang='en']")
-        expect(selector_builder.build(hreflang: /en/)).to eq(xpath: ".//*[contains(@hreflang, 'en')]")
+        expect(selector_builder.build(hreflang: /en/)).to eq(xpath: './/*[contains(@hreflang, "en")]')
       end
 
       it 'ignores case when locating uknown element with defined attribute' do
-        lhs = 'translate(@lang,' \
-  "'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
-  "'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ')"
-        rhs = "translate('en'," \
-  "'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
-  "'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ')"
+        lhs = "translate(@lang,#{upper},#{lower})"
+        rhs = "translate('en',#{upper},#{lower})"
+        rhs_regex = "translate(\"en\",#{upper},#{lower})"
         expect(selector_builder.build(lang: 'en')).to eq(xpath: ".//*[#{lhs}=#{rhs}]")
-        expect(selector_builder.build(lang: /en/)).to eq(xpath: ".//*[contains(#{lhs}, #{rhs})]")
+        expect(selector_builder.build(lang: /en/)).to eq(xpath: ".//*[contains(#{lhs}, #{rhs_regex})]")
         expect(selector_builder.build(tag_name: /a/, lang: 'en'))
-          .to eq(xpath: ".//*[contains(local-name(), 'a')][#{lhs}=#{rhs}]")
+          .to eq(xpath: ".//*[contains(local-name(), \"a\")][#{lhs}=#{rhs}]")
         expect(selector_builder.build(tag_name: /a/, lang: /en/))
-          .to eq(xpath: ".//*[contains(local-name(), 'a')][contains(#{lhs}, #{rhs})]")
+          .to eq(xpath: ".//*[contains(local-name(), \"a\")][contains(#{lhs}, #{rhs_regex})]")
       end
 
       it 'ignores case when attribute is defined for element' do
-        lhs = 'translate(@hreflang,' \
-  "'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
-  "'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ')"
-        rhs = "translate('en'," \
-  "'ABCDEFGHIJKLMNOPQRSTUVWXYZÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞŸŽŠŒ'," \
-  "'abcdefghijklmnopqrstuvwxyzàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿžšœ')"
+        lhs = "translate(@hreflang,#{upper},#{lower})"
+        rhs = "translate('en',#{upper},#{lower})"
+        rhs_regex = "translate(\"en\",#{upper},#{lower})"
         expect(selector_builder.build(tag_name: 'a', hreflang: 'en'))
           .to eq(xpath: ".//*[local-name()='a'][#{lhs}=#{rhs}]")
         expect(selector_builder.build(tag_name: 'a', hreflang: /en/))
-          .to eq(xpath: ".//*[local-name()='a'][contains(#{lhs}, #{rhs})]")
+          .to eq(xpath: ".//*[local-name()='a'][contains(#{lhs}, #{rhs_regex})]")
       end
 
       it 'respects case when attribute is not defined for element' do
         expect(selector_builder.build(tag_name: 'table', hreflang: 'en'))
           .to eq(xpath: ".//*[local-name()='table'][@hreflang='en']")
         expect(selector_builder.build(tag_name: 'table', hreflang: /en/))
-          .to eq(xpath: ".//*[local-name()='table'][contains(@hreflang, 'en')]")
+          .to eq(xpath: ".//*[local-name()='table'][contains(@hreflang, \"en\")]")
         expect(selector_builder.build(tag_name: /a/, hreflang: 'en'))
-          .to eq(xpath: ".//*[contains(local-name(), 'a')][@hreflang='en']")
+          .to eq(xpath: ".//*[contains(local-name(), \"a\")][@hreflang='en']")
         expect(selector_builder.build(tag_name: /a/, hreflang: /en/))
-          .to eq(xpath: ".//*[contains(local-name(), 'a')][contains(@hreflang, 'en')]")
+          .to eq(xpath: './/*[contains(local-name(), "a")][contains(@hreflang, "en")]')
       end
     end
   end

data/spec/unit/selector_builder/row_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../unit_helper'
 
 describe Watir::Locators::Row::SelectorBuilder do
@@ -13,7 +15,8 @@ describe Watir::Locators::Row::SelectorBuilder do
         @query_scope = element(tag_name: 'table')
 
         built = {xpath: "./*[local-name()='tr'] | ./*[local-name()='tbody']/*[local-name()='tr'] | " \
-"./*[local-name()='thead']/*[local-name()='tr'] | ./*[local-name()='tfoot']/*[local-name()='tr']"}
+                        "./*[local-name()='thead']/*[local-name()='tr'] | " \
+                        "./*[local-name()='tfoot']/*[local-name()='tr']"}
 
         expect(selector_built).to eq built
       end
@@ -49,7 +52,8 @@ describe Watir::Locators::Row::SelectorBuilder do
         it 'positive' do
           @selector = {index: 1}
           built = {xpath: "(./*[local-name()='tr'] | ./*[local-name()='tbody']/*[local-name()='tr'] | " \
-"./*[local-name()='thead']/*[local-name()='tr'] | ./*[local-name()='tfoot']/*[local-name()='tr'])[2]"}
+                          "./*[local-name()='thead']/*[local-name()='tr'] | " \
+                          "./*[local-name()='tfoot']/*[local-name()='tr'])[2]"}
 
           expect(selector_built).to eq built
         end
@@ -57,7 +61,8 @@ describe Watir::Locators::Row::SelectorBuilder do
         it 'negative' do
           @selector = {index: -3}
           built = {xpath: "(./*[local-name()='tr'] | ./*[local-name()='tbody']/*[local-name()='tr'] | " \
-"./*[local-name()='thead']/*[local-name()='tr'] | ./*[local-name()='tfoot']/*[local-name()='tr'])[last()-2]"}
+                          "./*[local-name()='thead']/*[local-name()='tr'] | " \
+                          "./*[local-name()='tfoot']/*[local-name()='tr'])[last()-2]"}
 
           expect(selector_built).to eq built
         end
@@ -65,7 +70,8 @@ describe Watir::Locators::Row::SelectorBuilder do
         it 'last' do
           @selector = {index: -1}
           built = {xpath: "(./*[local-name()='tr'] | ./*[local-name()='tbody']/*[local-name()='tr'] | " \
-"./*[local-name()='thead']/*[local-name()='tr'] | ./*[local-name()='tfoot']/*[local-name()='tr'])[last()]"}
+                          "./*[local-name()='thead']/*[local-name()='tr'] | " \
+                          "./*[local-name()='tfoot']/*[local-name()='tr'])[last()]"}
 
           expect(selector_built).to eq built
         end
@@ -73,7 +79,8 @@ describe Watir::Locators::Row::SelectorBuilder do
         it 'does not return index if it is zero' do
           @selector = {index: 0}
           built = {xpath: "./*[local-name()='tr'] | ./*[local-name()='tbody']/*[local-name()='tr'] | " \
-"./*[local-name()='thead']/*[local-name()='tr'] | ./*[local-name()='tfoot']/*[local-name()='tr']"}
+                          "./*[local-name()='thead']/*[local-name()='tr'] | " \
+                          "./*[local-name()='tfoot']/*[local-name()='tr']"}
 
           expect(selector_built).to eq built
         end
@@ -88,10 +95,13 @@ describe Watir::Locators::Row::SelectorBuilder do
       context 'with multiple locators' do
         it 'attribute and class' do
           @selector = {id: 'gregory', class: /brick/}
-          built = {xpath: "./*[local-name()='tr'][contains(@class, 'brick')][@id='gregory'] | " \
-"./*[local-name()='tbody']/*[local-name()='tr'][contains(@class, 'brick')][@id='gregory'] | " \
-"./*[local-name()='thead']/*[local-name()='tr'][contains(@class, 'brick')][@id='gregory'] | " \
-"./*[local-name()='tfoot']/*[local-name()='tr'][contains(@class, 'brick')][@id='gregory']"}
+          built = {xpath: "./*[local-name()='tr'][contains(@class, \"brick\")][@id='gregory'] | " \
+                          "./*[local-name()='tbody']/*[local-name()='tr']" \
+                          "[contains(@class, \"brick\")][@id='gregory'] | " \
+                          "./*[local-name()='thead']/*[local-name()='tr']" \
+                          "[contains(@class, \"brick\")][@id='gregory'] | " \
+                          "./*[local-name()='tfoot']/*[local-name()='tr']" \
+                          "[contains(@class, \"brick\")][@id='gregory']"}
 
           expect(selector_built).to eq built
         end
@@ -101,7 +111,8 @@ describe Watir::Locators::Row::SelectorBuilder do
         it 'any text value' do
           @selector = {text: 'Gregory'}
           built = {xpath: "./*[local-name()='tr'] | ./*[local-name()='tbody']/*[local-name()='tr'] | " \
-"./*[local-name()='thead']/*[local-name()='tr'] | ./*[local-name()='tfoot']/*[local-name()='tr']", text: 'Gregory'}
+                          "./*[local-name()='thead']/*[local-name()='tr'] | " \
+                          "./*[local-name()='tfoot']/*[local-name()='tr']", text: 'Gregory'}
 
           expect(selector_built).to eq built
         end