fix escaping bug when dealing with embedded regexes

Author: Frederik Fix

lib/selenium/client/javascript_expression_builder.rb

@@ -109,7 +109,7 @@ def window_script(expression)
       end
 
       def quote_escaped(a_string)
-        a_string.gsub(/'/, %q<\\\'>)
+        a_string.gsub(/\\/, "\\\\\\").gsub(/'/, %q<\\\'>)
       end
     end
   end

test/unit/selenium/client/javascript_expression_builder_tests.rb

@@ -36,7 +36,13 @@
     assert_equal "//div[@id=\\'demo-effect-appear\\']", 
                  builder.quote_escaped("//div[@id='demo-effect-appear']")
   end
-  
+
+  test "quote_escaped escapes backslashes" do
+    builder = Selenium::Client::JavascriptExpressionBuilder.new
+    assert_equal "webratlink=evalregex:/Pastry Lovers \\\\(Organizer\\\\)/", 
+                 builder.quote_escaped("webratlink=evalregex:/Pastry Lovers \\(Organizer\\)/")
+  end
+
   test "text_match matches on entire string when pattern is a string" do
     builder = Selenium::Client::JavascriptExpressionBuilder.new
     assert_equal "element.innerHTML == 'some text'", 
@@ -60,7 +66,13 @@
     assert_match(/element\s+=\s+selenium.browserbot.findElement\('a_locator'\);/m, 
                  builder.find_element('a_locator').script)
   end
-  
+
+  test "find_element should handle embedded evalregex locators" do
+    builder = Selenium::Client::JavascriptExpressionBuilder.new
+    assert_match(/element\s+=\s+selenium.browserbot.findElement\('webratlink=evalregex:\/Pastry Lovers \\\\\(Organizer\\\\\)\/'\);/m, 
+                 builder.find_element("webratlink=evalregex:/Pastry Lovers \\(Organizer\\)/").script)
+  end
+
   test "javascript_framework_for :prototype returns JavascriptFrameworks::Prototype" do
     builder = Selenium::Client::JavascriptExpressionBuilder.new
     assert_equal Selenium::Client::JavascriptFrameworks::Prototype,