google-gax 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 87b2e27818073928a99f7edc7a49ab31a1ff05a6
-  data.tar.gz: bbe86ff3f6584127ace5d22509c59bb1be0b7e60
+  metadata.gz: 8cd0a10641a7e1420289be043083df9265950f90
+  data.tar.gz: 59814b61241779e9dad2d71dbca9ddaa3f303518
 SHA512:
-  metadata.gz: 108eb512cbf772cb7b0fd1196b9676ef594bb7ecbdab2038f4e307e9e1d20289b7a108d4031a28545c8110468adc23c2ffbccdafa57e311a00f152a6afe32db6
-  data.tar.gz: 37fb30807ae24551812160303d18ead970587dacb30c53e1a01708eee42ac77e6b4ff5c68b322470c34f0e8a0f04ac98505c4ecb1aab00632bdd31a274cbdd08
+  metadata.gz: c7f3f07f30bf2382ff4bf38c2a6c2142dc7a72070ccbe14071b9193ebe53ba281b9fac525669179f5dd3fbc845a1b781c7659768d968b549398cbdf2948033ac
+  data.tar.gz: 683d0a62441eabaf25a7c1c87962ffcc86bc8ee5ff3936daa3a8cc78ec20b6c4daadfb5e85d546854676604e8879eb27a4d9f359a327a3b9c9cc2844d7aeafa1

data/lib/google/gax/settings.rb

@@ -388,8 +388,19 @@ module Google
       RetryOptions.new(codes, backoff_settings)
     end
 
-    def upper_camel_to_lower_underscore(string)
-      string.scan(/[[:upper:]][^[:upper:]]*/).map(&:downcase).join('_')
+    # Port of GRPC::GenericService.underscore that works on frozen strings.
+    # Note that this function often is used on strings inside Hashes, which
+    # are frozen by default, so the GRPC implementation cannot be used directly.
+    #
+    # TODO(geigerj): Consider whether this logic can be factored out into
+    # a shared location that both gRPC and GAX can depend on in order to remove
+    # the additional dependency on gRPC this introduces.
+    def upper_camel_to_lower_underscore(s)
+      s = s.gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
+      s = s.gsub(/([a-z\d])([A-Z])/, '\1_\2')
+      s = s.tr('-', '_')
+      s = s.downcase
+      s
     end
 
     # Constructs a dictionary mapping method names to CallSettings.

data/lib/google/gax/version.rb

@@ -29,6 +29,6 @@
 
 module Google
   module Gax
-    VERSION = '1.0.0'.freeze
+    VERSION = '1.0.1'.freeze
   end
 end

data/spec/google/gax/settings_spec.rb

@@ -60,7 +60,7 @@ A_CONFIG = {
             'element_count_limit' => 10
           }
         },
-        'PageStreamingMethod' => {
+        'SomeHTTPSPageStreamingMethod' => {
           'retry_codes_name' => 'bar_retry',
           'retry_params_name' => 'default'
         }
@@ -70,7 +70,7 @@ A_CONFIG = {
 }.freeze
 
 PAGE_DESCRIPTORS = {
-  'page_streaming_method' => Google::Gax::PageDescriptor.new(
+  'some_https_page_streaming_method' => Google::Gax::PageDescriptor.new(
     'page_token', 'next_page_token', 'page_streams'
   )
 }.freeze
@@ -107,7 +107,7 @@ describe Google::Gax do
     expect(settings.kwargs).to match('key' => 'value')
     expect(settings.errors).to match_array([StandardError])
 
-    settings = defaults['page_streaming_method']
+    settings = defaults['some_https_page_streaming_method']
     expect(settings.timeout).to be(30)
     expect(settings.bundler).to be_nil
     expect(settings.bundle_descriptor).to be_nil
@@ -126,7 +126,7 @@ describe Google::Gax do
       'interfaces' => {
         SERVICE_NAME => {
           'methods' => {
-            'PageStreamingMethod' => nil,
+            'SomeHTTPSPageStreamingMethod' => nil,
             'BundlingMethod' => {
               'bundling' => nil
             }
@@ -145,7 +145,7 @@ describe Google::Gax do
     expect(settings.bundler).to be_nil
     expect(settings.page_descriptor).to be_nil
 
-    settings = defaults['page_streaming_method']
+    settings = defaults['some_https_page_streaming_method']
     expect(settings.timeout).to be(30)
     expect(settings.page_descriptor).to be_a(Google::Gax::PageDescriptor)
     expect(settings.retry_options).to be_nil
@@ -194,10 +194,10 @@ describe Google::Gax do
     expect(settings.bundler).to be_nil
     expect(settings.bundle_descriptor).to be_a(Google::Gax::BundleDescriptor)
 
-    # page_streaming_method is unaffected because it's not specified in
-    # overrides. 'bar_retry' or 'default' definitions in overrides should
+    # some_https_page_streaming_method is unaffected because it's not specified
+    # in overrides. 'bar_retry' or 'default' definitions in overrides should
     # not affect the methods which are not in the overrides.
-    settings = defaults['page_streaming_method']
+    settings = defaults['some_https_page_streaming_method']
     backoff = settings.retry_options.backoff_settings
     expect(backoff.initial_retry_delay_millis).to be(100)
     expect(backoff.retry_delay_multiplier).to be(1.2)

metadata

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