google-gax 0.8.5 → 0.8.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 798b058e269c1439eae63e4902b7558b5be005c0
-  data.tar.gz: 3ee51d01d9addb3d2edb034efb196478fcf0c4ae
+  metadata.gz: f71e32189b1fb624d4eea6c700022151e4e94134
+  data.tar.gz: 5885fdf6ce17d16932c891b3677cf60a6b30f574
 SHA512:
-  metadata.gz: 84f4d99c7830db006033237c1f92f3de3b7d8dc4a43e4357b182f0cfabc52df76e6c7d3f0863ce6b89307221c31c3a50cf39573d116e8b4e78d28e42e4a11fb3
-  data.tar.gz: 95ab0e0670ba6eeb51cc6a2a98ba93f4beda3246be5ac43a9e6b3b344aedb9a369d6d01ccb32710f56ac34797a1f22d2d7ecb28f9beb221753f2af8d636c3835
+  metadata.gz: 251e5b49190d51b23dca1350d45300687ec8b41c632df5f62198385223f25f7391f828cc5475d7ccd15847b0245128cab5b919494bb9fc6a5cbfc6671eb63a16
+  data.tar.gz: f8a733a59af40210710c1e1661fc5ef88692178799b37fe5d8f404e120159a461e8cf68a28fb69877dced9190a4cd8c07e965b11707083530574563bbcf0c063

data/lib/google/gax/util.rb

@@ -90,7 +90,7 @@ module Google
     #
     # @return [Object] The coerced version of the given value.
     def coerce_submessage(val, field_descriptor)
-      if field_descriptor.label == :repeated
+      if (field_descriptor.label == :repeated) && !(map_field? field_descriptor)
         coerce_array(val, field_descriptor)
       else
         coerce(val, field_descriptor)
@@ -108,12 +108,24 @@ module Google
     #
     # @return [Array<Object>] The coerced version of the given values.
     def coerce_array(array, field_descriptor)
-      raise ArgumentError unless array.is_a? Array
+      unless array.is_a? Array
+        raise ArgumentError, 'Value ' + array.to_s + ' must be an array'
+      end
       array.map do |val|
         coerce(val, field_descriptor)
       end
     end
 
+    # Hack to determine if field_descriptor is for a map.
+    #
+    # TODO(geigerj): Remove this once protobuf Ruby supports an official way
+    # to determine if a FieldDescriptor represents a map.
+    # See: https://github.com/google/protobuf/issues/3425
+    def map_field?(field_descriptor)
+      (field_descriptor.label == :repeated) &&
+        (field_descriptor.subtype.name.include? '_MapEntry_')
+    end
+
     # Coerces the value of a field to be acceptable by the instantiation method
     # of the wrapping message.
     #
@@ -125,13 +137,13 @@ module Google
     #
     # @return [Object] The coerced version of the given value.
     def coerce(val, field_descriptor)
-      return val unless val.is_a? Hash
+      return val unless (val.is_a? Hash) && !(map_field? field_descriptor)
       to_proto(val, field_descriptor.subtype.msgclass)
     end
 
     module_function :to_proto, :coerce_submessages, :coerce_submessage,
-                    :coerce_array, :coerce
+                    :coerce_array, :coerce, :map_field?
     private_class_method :coerce_submessages, :coerce_submessage, :coerce_array,
-                         :coerce
+                         :coerce, :map_field?
   end
 end

data/lib/google/gax/version.rb

@@ -29,6 +29,6 @@
 
 module Google
   module Gax
-    VERSION = '0.8.5'.freeze
+    VERSION = '0.8.6'.freeze
   end
 end

data/lib/google/longrunning/operations_client.rb

@@ -48,7 +48,7 @@ module Google
     # Manages long-running operations with an API service.
     #
     # When an API method normally takes long time to complete, it can be designed
-    # to return Operation to the client, and the client can use this
+    # to return {Google::Longrunning::Operation Operation} to the client, and the client can use this
     # interface to receive the real response asynchronously by polling the
     # operation resource, or pass the operation resource to another API (such as
     # Google Cloud Pub/Sub API) to receive the response.  Any API service that
@@ -86,13 +86,21 @@ module Google
       #   The domain name of the API remote host.
       # @param port [Integer]
       #   The port on which to connect to the remote host.
-      # @param channel [Channel]
-      #   A Channel object through which to make calls.
-      # @param chan_creds [Grpc::ChannelCredentials]
-      #   A ChannelCredentials for the setting up the RPC client.
-      # @param updater_proc [Proc]
-      #   A function that transforms the metadata for requests, e.g., to give
-      #   OAuth credentials.
+      # @param credentials
+      #   [Google::Gax::Credentials, String, Hash, GRPC::Core::Channel, GRPC::Core::ChannelCredentials, Proc]
+      #   Provides the means for authenticating requests made by the client. This parameter can
+      #   be many types.
+      #   A `Google::Gax::Credentials` uses a the properties of its represented keyfile for
+      #   authenticating requests made by this client.
+      #   A `String` will be treated as the path to the keyfile to be used for the construction of
+      #   credentials for this client.
+      #   A `Hash` will be treated as the contents of a keyfile to be used for the construction of
+      #   credentials for this client.
+      #   A `GRPC::Core::Channel` will be used to make calls through.
+      #   A `GRPC::Core::ChannelCredentials` for the setting up the RPC client. The channel credentials
+      #   should already be composed with a `GRPC::Core::CallCredentials` object.
+      #   A `Proc` will be used as an updater_proc for the Grpc channel. The proc transforms the
+      #   metadata for requests, generally, to give OAuth credentials.
       # @param scopes [Array<String>]
       #   The OAuth scopes for this service. This parameter is ignored if
       #   an updater_proc is supplied.
@@ -109,6 +117,7 @@ module Google
           channel: nil,
           chan_creds: nil,
           updater_proc: nil,
+          credentials: nil,
           scopes: ALL_SCOPES,
           client_config: {},
           timeout: DEFAULT_TIMEOUT,
@@ -122,11 +131,35 @@ module Google
         require "google/gax/grpc"
         require "google/longrunning/operations_services_pb"
 
-
+        if channel || chan_creds || updater_proc
+          warn "The `channel`, `chan_creds`, and `updater_proc` parameters will be removed " \
+            "on 2017/09/08"
+          credentials ||= channel
+          credentials ||= chan_creds
+          credentials ||= updater_proc
+        end
         if app_name || app_version
           warn "`app_name` and `app_version` are no longer being used in the request headers."
         end
 
+        credentials ||= Google::Gax::Credentials.default
+
+        if credentials.is_a?(String) || credentials.is_a?(Hash)
+          updater_proc = Google::Gax::Credentials.new(credentials).updater_proc
+        end
+        if credentials.is_a?(GRPC::Core::Channel)
+          channel = credentials
+        end
+        if credentials.is_a?(GRPC::Core::ChannelCredentials)
+          chan_creds = credentials
+        end
+        if credentials.is_a?(Proc)
+          updater_proc = credentials
+        end
+        if credentials.is_a?(Google::Gax::Credentials)
+          updater_proc = credentials.updater_proc
+        end
+
         google_api_client = "gl-ruby/#{RUBY_VERSION}"
         google_api_client << " #{lib_name}/#{lib_version}" if lib_name
         google_api_client << " gapic/0.6.8 gax/#{Google::Gax::VERSION}"
@@ -193,16 +226,17 @@ module Google
       # @example
       #   require "google/longrunning"
       #
-      #   operations_client = Google::Longrunning::OperationsClient.new
+      #   operations_client = Google::Longrunning.new
       #   name = ''
       #   response = operations_client.get_operation(name)
 
       def get_operation \
           name,
           options: nil
-        req = Google::Longrunning::GetOperationRequest.new({
+        req = {
           name: name
-        }.delete_if { |_, v| v.nil? })
+        }.delete_if { |_, v| v.nil? }
+        req = Google::Gax::to_proto(req, Google::Longrunning::GetOperationRequest)
         @get_operation.call(req, options)
       end
 
@@ -234,7 +268,7 @@ module Google
       # @example
       #   require "google/longrunning"
       #
-      #   operations_client = Google::Longrunning::OperationsClient.new
+      #   operations_client = Google::Longrunning.new
       #   name = ''
       #   filter = ''
       #
@@ -256,11 +290,12 @@ module Google
           filter,
           page_size: nil,
           options: nil
-        req = Google::Longrunning::ListOperationsRequest.new({
+        req = {
           name: name,
           filter: filter,
           page_size: page_size
-        }.delete_if { |_, v| v.nil? })
+        }.delete_if { |_, v| v.nil? }
+        req = Google::Gax::to_proto(req, Google::Longrunning::ListOperationsRequest)
         @list_operations.call(req, options)
       end
 
@@ -268,11 +303,11 @@ module Google
       # makes a best effort to cancel the operation, but success is not
       # guaranteed.  If the server doesn't support this method, it returns
       # +google.rpc.Code.UNIMPLEMENTED+.  Clients can use
-      # Operations::GetOperation or
+      # {Google::Longrunning::Operations::GetOperation Operations::GetOperation} or
       # other methods to check whether the cancellation succeeded or whether the
       # operation completed despite cancellation. On successful cancellation,
       # the operation is not deleted; instead, it becomes an operation with
-      # an Operation#error value with a Google::Rpc::Status#code of 1,
+      # an {Google::Longrunning::Operation#error Operation#error} value with a {Google::Rpc::Status#code} of 1,
       # corresponding to +Code.CANCELLED+.
       #
       # @param name [String]
@@ -284,16 +319,17 @@ module Google
       # @example
       #   require "google/longrunning"
       #
-      #   operations_client = Google::Longrunning::OperationsClient.new
+      #   operations_client = Google::Longrunning.new
       #   name = ''
       #   operations_client.cancel_operation(name)
 
       def cancel_operation \
           name,
           options: nil
-        req = Google::Longrunning::CancelOperationRequest.new({
+        req = {
           name: name
-        }.delete_if { |_, v| v.nil? })
+        }.delete_if { |_, v| v.nil? }
+        req = Google::Gax::to_proto(req, Google::Longrunning::CancelOperationRequest)
         @cancel_operation.call(req, options)
         nil
       end
@@ -312,16 +348,17 @@ module Google
       # @example
       #   require "google/longrunning"
       #
-      #   operations_client = Google::Longrunning::OperationsClient.new
+      #   operations_client = Google::Longrunning.new
       #   name = ''
       #   operations_client.delete_operation(name)
 
       def delete_operation \
           name,
           options: nil
-        req = Google::Longrunning::DeleteOperationRequest.new({
+        req = {
           name: name
-        }.delete_if { |_, v| v.nil? })
+        }.delete_if { |_, v| v.nil? }
+        req = Google::Gax::to_proto(req, Google::Longrunning::DeleteOperationRequest)
         @delete_operation.call(req, options)
         nil
       end

data/lib/google/longrunning/operations_client_config.json

@@ -6,9 +6,7 @@
           "DEADLINE_EXCEEDED",
           "UNAVAILABLE"
         ],
-        "non_idempotent": [
-          "UNAVAILABLE"
-        ]
+        "non_idempotent": []
       },
       "retry_params": {
         "default": {

data/spec/fixtures/fixture.proto

@@ -28,6 +28,7 @@ message User {
   string name = 1;
   UserType type = 2;
   repeated Post posts = 3;
+  map<string, string> map_field = 4;
 }
 
 message Post {

data/spec/fixtures/fixture_pb.rb

@@ -12,6 +12,7 @@ Google::Protobuf::DescriptorPool.generated_pool.build do
     optional :name, :string, 1
     optional :type, :enum, 2, "google.protobuf.User.UserType"
     repeated :posts, :message, 3, "google.protobuf.Post"
+    map :map_field, :string, :string, 4
   end
   add_enum "google.protobuf.User.UserType" do
     value :UNSPECIFIED, 0

data/spec/google/gax/util_spec.rb

@@ -36,6 +36,10 @@ describe Google::Gax do
     USER_NAME = 'Ernest'.freeze
     USER_TYPE = :ADMINISTRATOR
     POST_TEXT = 'This is a test post.'.freeze
+    MAP = {
+      'key1' => 'val1',
+      'key2' => 'val2'
+    }.freeze
 
     it 'creates a protobuf message from a simple hash' do
       hash = { name: USER_NAME, type: USER_TYPE }
@@ -91,6 +95,20 @@ describe Google::Gax do
       end
     end
 
+    it 'handles maps' do
+      request_hash = {
+        name: USER_NAME,
+        map_field: MAP
+      }
+      user = Google::Gax.to_proto(request_hash, Google::Protobuf::User)
+      expect(user).to be_an_instance_of(Google::Protobuf::User)
+      expect(user.name).to eq(USER_NAME)
+      expect(user.map_field).to be_an_instance_of(Google::Protobuf::Map)
+      user.map_field.each do |k, v|
+        expect(MAP[k]).to eq v
+      end
+    end
+
     it 'fails if a key does not exist in the target message type' do
       user_hash = {
         name: USER_NAME,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: google-gax
 version: !ruby/object:Gem::Version
-  version: 0.8.5
+  version: 0.8.6
 platform: ruby
 authors:
 - Google API Authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-21 00:00:00.000000000 Z
+date: 2017-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: googleauth