Accept repeated parameters
[arvados.git] / spec / google / api_client / discovery_spec.rb
index bff4685ce9564fa30240ec4c3aa1666d4208eca7..7714e25c6052e9784e9803511a4a07a6249f51ba 100644 (file)
@@ -14,6 +14,7 @@
 
 require 'spec_helper'
 
+require 'json'
 require 'signet/oauth_1/client'
 require 'httpadapter/adapters/net_http'
 
@@ -66,6 +67,9 @@ describe Google::APIClient do
   describe 'with the prediction API' do
     before do
       @client.authorization = nil
+      # The prediction API no longer exposes a v1, so we have to be
+      # careful about looking up the wrong API version.
+      @prediction = @client.discovered_api('prediction', 'v1.2')
     end
 
     it 'should correctly determine the discovery URI' do
@@ -73,46 +77,83 @@ describe Google::APIClient do
         'https://www.googleapis.com/discovery/v1/apis/prediction/v1/rest'
     end
 
+    it 'should correctly determine the discovery URI if :user_ip is set' do
+      @client.user_ip = '127.0.0.1'
+      request = @client.generate_request(
+        :http_method => 'GET',
+        :uri => @client.discovery_uri('prediction', 'v1.2'),
+        :authenticated => false
+      )
+      http_method, uri, headers, body = request
+      uri.should === (
+        'https://www.googleapis.com/discovery/v1/apis/prediction/v1.2/rest' +
+        '?userIp=127.0.0.1'
+      )
+    end
+
+    it 'should correctly determine the discovery URI if :key is set' do
+      @client.key = 'qwerty'
+      request = @client.generate_request(
+        :http_method => 'GET',
+        :uri => @client.discovery_uri('prediction', 'v1.2'),
+        :authenticated => false
+      )
+      http_method, uri, headers, body = request
+      uri.should === (
+        'https://www.googleapis.com/discovery/v1/apis/prediction/v1.2/rest' +
+        '?key=qwerty'
+      )
+    end
+
+    it 'should correctly determine the discovery URI if both are set' do
+      @client.key = 'qwerty'
+      @client.user_ip = '127.0.0.1'
+      request = @client.generate_request(
+        :http_method => 'GET',
+        :uri => @client.discovery_uri('prediction', 'v1.2'),
+        :authenticated => false
+      )
+      http_method, uri, headers, body = request
+      uri.should === (
+        'https://www.googleapis.com/discovery/v1/apis/prediction/v1.2/rest' +
+        '?key=qwerty&userIp=127.0.0.1'
+      )
+    end
+
     it 'should correctly generate API objects' do
-      @client.discovered_api('prediction').name.should == 'prediction'
-      @client.discovered_api('prediction').version.should == 'v1'
-      @client.discovered_api(:prediction).name.should == 'prediction'
-      @client.discovered_api(:prediction).version.should == 'v1'
+      @client.discovered_api('prediction', 'v1.2').name.should == 'prediction'
+      @client.discovered_api('prediction', 'v1.2').version.should == 'v1.2'
+      @client.discovered_api(:prediction, 'v1.2').name.should == 'prediction'
+      @client.discovered_api(:prediction, 'v1.2').version.should == 'v1.2'
     end
 
     it 'should discover methods' do
       @client.discovered_method(
-        'prediction.training.insert', 'prediction'
+        'prediction.training.insert', 'prediction', 'v1.2'
       ).name.should == 'insert'
       @client.discovered_method(
-        :'prediction.training.insert', :prediction
+        :'prediction.training.insert', :prediction, 'v1.2'
       ).name.should == 'insert'
-    end
-
-    it 'should discover methods' do
       @client.discovered_method(
-        'prediction.training.delete', 'prediction', 'v1.1'
+        'prediction.training.delete', 'prediction', 'v1.2'
       ).name.should == 'delete'
     end
 
     it 'should not find methods that are not in the discovery document' do
       @client.discovered_method(
-        'prediction.training.delete', 'prediction', 'v1'
-      ).should == nil
-      @client.discovered_method(
-        'prediction.bogus', 'prediction', 'v1'
+        'prediction.bogus', 'prediction', 'v1.2'
       ).should == nil
     end
 
     it 'should raise an error for bogus methods' do
       (lambda do
-        @client.discovered_method(42, 'prediction', 'v1')
+        @client.discovered_method(42, 'prediction', 'v1.2')
       end).should raise_error(TypeError)
     end
 
     it 'should raise an error for bogus methods' do
       (lambda do
-        @client.generate_request(@client.discovered_api('prediction'))
+        @client.generate_request(@client.discovered_api('prediction', 'v1.2'))
       end).should raise_error(TypeError)
     end
 
@@ -123,49 +164,50 @@ describe Google::APIClient do
 
     it 'should generate valid requests' do
       request = @client.generate_request(
-        'prediction.training.insert',
-        {'data' => '12345', }
+        :api_method => @prediction.training.insert,
+        :parameters => {'data' => '12345', }
       )
       method, uri, headers, body = request
       method.should == 'POST'
       uri.should ==
-        'https://www.googleapis.com/prediction/v1/training?data=12345'
+        'https://www.googleapis.com/prediction/v1.2/training?data=12345'
       (headers.inject({}) { |h,(k,v)| h[k]=v; h }).should == {}
       body.should respond_to(:each)
     end
 
     it 'should generate requests against the correct URIs' do
       request = @client.generate_request(
-        :'prediction.training.insert',
-        {'data' => '12345'}
+        :api_method => @prediction.training.insert,
+        :parameters => {'data' => '12345'}
       )
       method, uri, headers, body = request
       uri.should ==
-        'https://www.googleapis.com/prediction/v1/training?data=12345'
+        'https://www.googleapis.com/prediction/v1.2/training?data=12345'
     end
 
     it 'should generate requests against the correct URIs' do
-      prediction = @client.discovered_api('prediction', 'v1')
       request = @client.generate_request(
-        prediction.training.insert,
-        {'data' => '12345'}
+        :api_method => @prediction.training.insert,
+        :parameters => {'data' => '12345'}
       )
       method, uri, headers, body = request
       uri.should ==
-        'https://www.googleapis.com/prediction/v1/training?data=12345'
+        'https://www.googleapis.com/prediction/v1.2/training?data=12345'
     end
 
     it 'should allow modification to the base URIs for testing purposes' do
-      prediction = @client.discovered_api('prediction', 'v1')
+      prediction = @client.discovered_api('prediction', 'v1.2')
       prediction.method_base =
-        'https://testing-domain.googleapis.com/prediction/v1/'
+        'https://testing-domain.googleapis.com/prediction/v1.2/'
       request = @client.generate_request(
-        prediction.training.insert,
-        {'data' => '123'}
+        :api_method => prediction.training.insert,
+        :parameters => {'data' => '123'}
       )
       method, uri, headers, body = request
-      uri.should ==
-        'https://testing-domain.googleapis.com/prediction/v1/training?data=123'
+      uri.should == (
+        'https://testing-domain.googleapis.com/' +
+        'prediction/v1.2/training?data=123'
+      )
     end
 
     it 'should generate OAuth 1 requests' do
@@ -173,8 +215,8 @@ describe Google::APIClient do
       @client.authorization.token_credential_key = '12345'
       @client.authorization.token_credential_secret = '12345'
       request = @client.generate_request(
-        'prediction.training.insert',
-        {'data' => '12345'}
+        :api_method => @prediction.training.insert,
+        :parameters => {'data' => '12345'}
       )
       method, uri, headers, body = request
       headers = headers.inject({}) { |h,(k,v)| h[k]=v; h }
@@ -186,8 +228,8 @@ describe Google::APIClient do
       @client.authorization = :oauth_2
       @client.authorization.access_token = '12345'
       request = @client.generate_request(
-        'prediction.training.insert',
-        {'data' => '12345'}
+        :api_method => @prediction.training.insert,
+        :parameters => {'data' => '12345'}
       )
       method, uri, headers, body = request
       headers = headers.inject({}) { |h,(k,v)| h[k]=v; h }
@@ -199,24 +241,60 @@ describe Google::APIClient do
       @client.authorization = :oauth_1
       @client.authorization.token_credential_key = '12345'
       @client.authorization.token_credential_secret = '12345'
-      response = @client.execute(
-        'prediction.training.insert',
+      result = @client.execute(
+        @prediction.training.insert,
         {'data' => '12345'}
       )
-      status, headers, body = response
+      status, headers, body = result.response
       status.should == 401
     end
 
     it 'should not be able to execute improperly authorized requests' do
       @client.authorization = :oauth_2
       @client.authorization.access_token = '12345'
-      response = @client.execute(
-        'prediction.training.insert',
+      result = @client.execute(
+        @prediction.training.insert,
         {'data' => '12345'}
       )
-      status, headers, body = response
+      status, headers, body = result.response
       status.should == 401
     end
+
+    it 'should not be able to execute improperly authorized requests' do
+      (lambda do
+        @client.authorization = :oauth_1
+        @client.authorization.token_credential_key = '12345'
+        @client.authorization.token_credential_secret = '12345'
+        result = @client.execute!(
+          @prediction.training.insert,
+          {'data' => '12345'}
+        )
+      end).should raise_error(Google::APIClient::ClientError)
+    end
+
+    it 'should not be able to execute improperly authorized requests' do
+      (lambda do
+        @client.authorization = :oauth_2
+        @client.authorization.access_token = '12345'
+        result = @client.execute!(
+          @prediction.training.insert,
+          {'data' => '12345'}
+        )
+      end).should raise_error(Google::APIClient::ClientError)
+    end
+
+    it 'should correctly handle unnamed parameters' do
+      @client.authorization = :oauth_2
+      @client.authorization.access_token = '12345'
+      result = @client.execute(
+        @prediction.training.insert,
+        {},
+        JSON.generate({"id" => "bucket/object"}),
+        {'Content-Type' => 'application/json'}
+      )
+      method, uri, headers, body = result.request
+      Hash[headers]['Content-Type'].should == 'application/json'
+    end
   end
 
   describe 'with the buzz API' do
@@ -251,22 +329,18 @@ describe Google::APIClient do
     it 'should fail for string RPC names that do not match API name' do
       (lambda do
         @client.generate_request(
-          'chili.activities.list',
-          {'alt' => 'json'},
-          '',
-          [],
-          {:signed => false}
+          :api_method => 'chili.activities.list',
+          :parameters => {'alt' => 'json'},
+          :authenticated => false
         )
       end).should raise_error(Google::APIClient::TransmissionError)
     end
 
     it 'should generate requests against the correct URIs' do
       request = @client.generate_request(
-        @buzz.activities.list,
-        {'userId' => 'hikingfan', 'scope' => '@public'},
-        '',
-        [],
-        {:signed => false}
+        :api_method => @buzz.activities.list,
+        :parameters => {'userId' => 'hikingfan', 'scope' => '@public'},
+        :authenticated => false
       )
       method, uri, headers, body = request
       uri.should ==
@@ -276,11 +350,9 @@ describe Google::APIClient do
     it 'should correctly validate parameters' do
       (lambda do
         @client.generate_request(
-          @buzz.activities.list,
-          {'alt' => 'json'},
-          '',
-          [],
-          {:signed => false}
+          :api_method => @buzz.activities.list,
+          :parameters => {'alt' => 'json'},
+          :authenticated => false
         )
       end).should raise_error(ArgumentError)
     end
@@ -288,26 +360,33 @@ describe Google::APIClient do
     it 'should correctly validate parameters' do
       (lambda do
         @client.generate_request(
-          @buzz.activities.list,
-          {'userId' => 'hikingfan', 'scope' => '@bogus'},
-          '',
-          [],
-          {:signed => false}
+          :api_method => @buzz.activities.list,
+          :parameters => {'userId' => 'hikingfan', 'scope' => '@bogus'},
+          :authenticated => false
         )
       end).should raise_error(ArgumentError)
     end
 
     it 'should be able to execute requests without authorization' do
-      response = @client.execute(
+      result = @client.execute(
         @buzz.activities.list,
         {'alt' => 'json', 'userId' => 'hikingfan', 'scope' => '@public'},
         '',
         [],
-        {:signed => false}
+        :authenticated => false
       )
-      status, headers, body = response
+      status, headers, body = result.response
       status.should == 200
     end
+
+    it 'should not be able to execute requests without authorization' do
+      result = @client.execute(
+        @buzz.activities.list,
+        'alt' => 'json', 'userId' => '@me', 'scope' => '@self'
+      )
+      status, headers, body = result.response
+      status.should == 401
+    end
   end
 
   describe 'with the latitude API' do
@@ -338,11 +417,8 @@ describe Google::APIClient do
 
     it 'should generate requests against the correct URIs' do
       request = @client.generate_request(
-        'latitude.currentLocation.get',
-        {},
-        '',
-        [],
-        {:signed => false}
+        :api_method => 'latitude.currentLocation.get',
+        :authenticated => false
       )
       method, uri, headers, body = request
       uri.should ==
@@ -351,11 +427,8 @@ describe Google::APIClient do
 
     it 'should generate requests against the correct URIs' do
       request = @client.generate_request(
-        @latitude.current_location.get,
-        {},
-        '',
-        [],
-        {:signed => false}
+        :api_method => @latitude.current_location.get,
+        :authenticated => false
       )
       method, uri, headers, body = request
       uri.should ==
@@ -363,14 +436,11 @@ describe Google::APIClient do
     end
 
     it 'should not be able to execute requests without authorization' do
-      response = @client.execute(
-        'latitude.currentLocation.get',
-        {},
-        '',
-        [],
-        {:signed => false}
+      result = @client.execute(
+        :api_method => 'latitude.currentLocation.get',
+        :authenticated => false
       )
-      status, headers, body = response
+      status, headers, body = result.response
       status.should == 401
     end
   end
@@ -403,11 +473,8 @@ describe Google::APIClient do
 
     it 'should generate requests against the correct URIs' do
       request = @client.generate_request(
-        'moderator.profiles.get',
-        {},
-        '',
-        [],
-        {:signed => false}
+        :api_method => 'moderator.profiles.get',
+        :authenticated => false
       )
       method, uri, headers, body = request
       uri.should ==
@@ -416,11 +483,8 @@ describe Google::APIClient do
 
     it 'should generate requests against the correct URIs' do
       request = @client.generate_request(
-        @moderator.profiles.get,
-        {},
-        '',
-        [],
-        {:signed => false}
+        :api_method => @moderator.profiles.get,
+        :authenticated => false
       )
       method, uri, headers, body = request
       uri.should ==
@@ -428,14 +492,14 @@ describe Google::APIClient do
     end
 
     it 'should not be able to execute requests without authorization' do
-      response = @client.execute(
+      result = @client.execute(
         'moderator.profiles.get',
         {},
         '',
         [],
-        {:signed => false}
+        {:authenticated => false}
       )
-      status, headers, body = response
+      status, headers, body = result.response
       status.should == 401
     end
   end