Throw a real error when uploading non-file things in Ruby.

Signed-off-by: Jari Bakken <jari.bakken@gmail.com>

Author: DylanLacey

rb/lib/selenium/webdriver/remote/bridge.rb

@@ -413,7 +413,7 @@ def sendKeysToElement(element, keys)
 
         def upload(local_file)
           unless File.file?(local_file)
-            raise WebDriverError::Error, "you may only upload files: #{local_file.inspect}"
+            raise Error::WebDriverError, "you may only upload files: #{local_file.inspect}"
           end
 
           execute :uploadFile, {}, :file => Zipper.zip_file(local_file)

rb/spec/unit/selenium/webdriver/remote/bridge_spec.rb

@@ -8,6 +8,16 @@ module Remote
         it "raises ArgumentError if passed invalid options" do
           lambda { Bridge.new(:foo => 'bar') }.should raise_error(ArgumentError)
         end
+
+        it "raises WebDriverError if uploading non-files" do
+          request_body = {:sessionId => '11123', :value => {}}.to_json
+          headers = {'Content-Type' => 'application/json'}
+          stub_request(:post, "http://127.0.0.1:4444/wd/hub/session").to_return(
+            :status => 200, :body => request_body, :headers => headers)
+
+          bridge = Bridge.new
+          lambda { bridge.upload("NotAFile")}.should raise_error(Error::WebDriverError)
+        end
       end
 
     end # Remote