Fixed regexp escaping and added tests

Author: rslifka

spec/unit/client_spec.rb

@@ -232,19 +232,34 @@ module Proxy
         client.remap_dns_hosts(DOMAIN => '1.2.3.4')
       end
 
-      it 'sets a rewrite rule' do
-        resource['rewrite'].should_receive(:put).
-          with(:matchRegex => 'old.com', :replace => 'new.com')
+      describe 'rewrite rules' do
 
-        client.rewrite(%r[old.com], 'new.com')
-      end
+        context 'when using a regular expression' do
+          it 'sets a rewrite rule' do
+            resource['rewrite'].should_receive(:put).
+              with(:matchRegex => 'old\.com', :replace => 'new.com')
+
+            client.rewrite('old\.com', 'new.com')
+          end
+        end
+
+        context 'when using a string' do
+          it 'sets a rewrite rule' do
+            resource['rewrite'].should_receive(:put).
+              with(:matchRegex => 'old\.com', :replace => 'new.com')
 
-      it 'clears the rewrite rules' do
-        resource['rewrite'].should_receive(:delete)
+            client.rewrite(%r{old\.com}, 'new.com')
+          end
+        end
+
+        it 'clears the rewrite rules' do
+          resource['rewrite'].should_receive(:delete)
 
-        client.clear_rewrites
+          client.clear_rewrites
+        end
       end
 
+
       context "#selenium_proxy" do
         it "defaults to HTTP proxy only" do
           proxy = client.selenium_proxy