fog-google 1.7.1 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4ef0c77cc212c88e05b3b2ba96a93a50b4e7ec6a
-  data.tar.gz: 3d052f7c4ae45331d3284fe06ab4b5c6387a2505
+  metadata.gz: c56136ffcc21e614018a5157077bcaf6315c740f
+  data.tar.gz: b9da25d90e07d29a3b1645d98776e7897206155a
 SHA512:
-  metadata.gz: b769db3c40da12eb211331bf3c00f73b7a938aeea5f09a1672cc142c68044b05303e79919b4dc611bce66684b6bf8b4255d3532e615c412357e58b80ca95fc9f
-  data.tar.gz: 775be1007e7030acbb089441297345e78d9afa106659be6d47d4017ad86f3db36c4ca9ab96eec0bb4a181d1bb8aebaef431cd730a0e6ccdee9b3ec8c90c7d546
+  metadata.gz: 855a091e79a21628ce05c763310ea92b2bf1eb281316fedca9cb6cd6a166097064d2c9d7795f04950201e0c6478d7bd0ba4461215770b72b4e786e0fef380370
+  data.tar.gz: 95925fcea76bdec54634581318827e87c7d03b98acea750bedfed767437dbd0a03e5f520ac7e635177fcc9ff7d286a2e8443d8578646e94583dcf840d3976915

data/.rubocop.yml

@@ -6,6 +6,9 @@ Style/ConditionalAssignment:
   SingleLineConditionsOnly: true
   EnforcedStyle: assign_inside_condition
 
+Style/RedundantReturn:
+  Enabled: false
+
 Style/FormatString:
   Enabled: false
 

data/.travis.yml

@@ -12,9 +12,6 @@ matrix:
     - rvm: 2.5
     - rvm: jruby-head
   allow_failures:
-  # allow 2.5 to fail until Travis CI is fixed.
-  # See travis-ci/travis-ci#8969
-    - rvm: 2.5
     - rvm: jruby-head
 notifications:
   email: false

data/CHANGELOG.md

@@ -1,6 +1,50 @@
 # Changelog
 All notable changes to this project will be documented in this file.
-The format is loosely based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
+The format is loosely based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
+
+This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
+
+## 1.8.0
+
+### User-facing
+
+#### Added
+- \#418 Reintroduce client options for proxy support, etc. [AlexanderZagaynov]
+
+#### Fixed
+
+- \#419 Locked down fog upstream dependencies to alleviate deprecation warnings
+  until they can be properly dealt with. [temikus]
+- \#400 Small `%Collection%.get` and `%Collection%.all` behaviour fixes [temikus]
+  - `Fog::Google::SQL::Instances.get(nil)` no longer returns an invalid 
+    `sql#instancesList` object.
+  - `Fog::Compute::Google::InstanceGroups.get` and `.all` methods now support 
+    more than just `:filter` option, fixed `.all` output without `zone` option.
+  - Fix a typo causing `Operations.get(region:REGION)` to fail.
+  - `Fog::Compute::Google::Images.get(IMAGE, PROJECT)`, now returns `nil` if 
+    image is not found rather than throwing `Google::Apis::ClientError`.
+
+### Development changes
+
+#### Added
+
+- \#400 Additional test coverage [temikus]
+  - Expanded tests for `%Collection%.get` behavior - scoped requests 
+    (e.g. `get(zone:ZONE)`) and their corresponding code paths are now also 
+    properly tested.
+  - Increase `Fog::Compute::Google::Images` integration test coverage.
+  - Unit tests now work without a `~/.fog` config file set up.
+  - Expanded unit test coverage.
+- \#424 Add simple integration tests to check client proxy options being 
+  applied.
+
+#### Changed
+
+- \#400 Refactored most compute `get()` and `all()` methods to common format. [temikus]
+
+#### Fixed
+
+- \#400 Removed the Travis Ruby 2.5 workaround. [temikus]
 
 ## 1.7.1
 

data/CONTRIBUTING.md

@@ -101,20 +101,26 @@ test:
 Then you can run all the live tests:
 
 ```shell
-$ rake test
+$ bundle exec rake test
 ```
 
 or just one API:
 
 ```
-$ rake test:compute
+$ bundle exec rake test:compute
 ```
 (See `rake -T` for all available tasks)
 
-or just one file:
+or just one test:
 
 ```shell
-$ rake test TEST=test/integration/compute/test_servers.rb TESTOPTS="--name=TestServers#test_bootstrap_ssh_destroy"
+$ bundle exec rake test TESTOPTS="--name=TestServers#test_bootstrap_ssh_destroy"
+```
+
+or a series of tests by name:
+
+```
+$ bundle exec rake test TESTOPTS="--name=/test_nil_get/"
 ```
 
 #### Unit tests
@@ -133,7 +139,7 @@ We're in progress of extending the library with more unit tests and contribution
 #### The transition from `shindo` to Minitest
 
 Previously, [shindo](https://github.com/geemus/shindo) was the primary testing framework. 
-We've started moving away from it, and to Minitest, but some artifacts may remain.
+We've moved away from it, and to Minitest, but some artifacts may remain.
 
 For more information on transition, read [#50](https://github.com/fog/fog-google/issues/50).
 

data/README.md

@@ -42,8 +42,6 @@ We are always looking for people to improve our code and test coverage, so pleas
 
 ## Pubsub
 
-Note: You **must** have a version of google-api-client > 0.8.5 to use the Pub/Sub API; previous versions will not work.
-
 Fog mostly implements [v1](https://cloud.google.com/pubsub/docs/reference/rest/) of the Google Cloud Pub/Sub API; however some less common API methods are missing. Pull requests for additions would be greatly appreciated.
 
 ## Installation

data/fog-google.gemspec

@@ -20,15 +20,14 @@ Gem::Specification.new do |spec|
   # As of 0.1.1
   spec.required_ruby_version = "~> 2.0"
 
-  spec.add_dependency "fog-core"
-  spec.add_dependency "fog-json"
-  spec.add_dependency "fog-xml"
+  # Locked until https://github.com/fog/fog-google/issues/417 is resolved
+  spec.add_dependency "fog-core", ">= 2.0", "<= 2.1.0"
+  spec.add_dependency "fog-json", "~> 1.2.0"
+  spec.add_dependency "fog-xml", "~> 0.1.0"
 
   # Hard Requirement as of 1.0
   spec.add_dependency "google-api-client", "~> 0.23.0"
   
-  spec.add_development_dependency "mime-types"
-
   # Debugger
   spec.add_development_dependency "pry"
   spec.add_development_dependency "pry-byebug"

data/lib/fog/bin/google.rb

@@ -15,6 +15,8 @@ module Google
         Fog::Storage::Google
       when :sql
         Fog::Google::SQL
+      when :pubsub
+        Fog::Google::Pubsub
       else
         raise ArgumentError, "Unsupported #{self} service: #{key}"
       end
@@ -33,6 +35,8 @@ module Google
                       Fog::Google::Monitoring.new
                     when :sql
                       Fog::Google::SQL.new
+                    when :pubsub
+                      Fog::Google::Pubsub.new
                     when :storage
                       Fog::Logger.warning("Google[:storage] is not recommended, use Storage[:google] for portability")
                       Fog::Storage.new(:provider => "Google")

data/lib/fog/compute/google/models/addresses.rb

@@ -23,9 +23,15 @@ module Fog
           load(data.map(&:to_h))
         end
 
-        def get(identity, region)
-          if address = service.get_address(identity, region).to_h
-            new(address)
+        def get(identity, region = nil)
+          if region
+            address = service.get_address(identity, region).to_h
+            return new(address)
+          elsif identity
+            response = all(:filter => "name eq #{identity}",
+                           :max_results => 1)
+            address = response.first unless response.empty?
+            return address
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404

data/lib/fog/compute/google/models/backend_services.rb

@@ -10,8 +10,9 @@ module Fog
         end
 
         def get(identity)
-          if backend_service = service.get_backend_service(identity)
-            new(backend_service.to_h)
+          if identity
+            backend_service = service.get_backend_service(identity).to_h
+            return new(backend_service)
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404

data/lib/fog/compute/google/models/disk_types.rb

@@ -24,15 +24,14 @@ module Fog
         end
 
         def get(identity, zone = nil)
-          response = nil
           if zone
-            response = service.get_disk_type(identity, zone).to_h
+            disk_type = service.get_disk_type(identity, zone).to_h
+            return new(disk_type)
           else
-            disk_types = all(:filter => "name eq .*#{identity}")
-            response = disk_types.first.attributes unless disk_types.empty?
+            response = all(:filter => "name eq .*#{identity}")
+            disk_type = response.first unless response.empty?
+            return disk_type
           end
-          return nil if response.nil?
-          new(response)
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404
           nil

data/lib/fog/compute/google/models/disks.rb

@@ -23,10 +23,16 @@ module Fog
           load(data.map(&:to_h))
         end
 
-        def get(identity, zone)
-          response = service.get_disk(identity, zone)
-          return nil if response.nil?
-          new(response.to_h)
+        def get(identity, zone = nil)
+          if zone
+            disk = service.get_disk(identity, zone).to_h
+            return new(disk)
+          elsif identity
+            response = all(:filter => "name eq #{identity}",
+                           :max_results => 1)
+            disk = response.first unless response.empty?
+            return disk
+          end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404
           nil

data/lib/fog/compute/google/models/firewalls.rb

@@ -10,8 +10,9 @@ module Fog
         end
 
         def get(identity)
-          if firewall = service.get_firewall(identity)
-            new(firewall.to_h)
+          if identity
+            firewall = service.get_firewall(identity).to_h
+            return new(firewall)
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404

data/lib/fog/compute/google/models/forwarding_rules.rb

@@ -4,9 +4,7 @@ module Fog
       class ForwardingRules < Fog::Collection
         model Fog::Compute::Google::ForwardingRule
 
-        def all(region: nil, filter: nil, max_results: nil,
-                order_by: nil, page_token: nil)
-
+        def all(region: nil, filter: nil, max_results: nil, order_by: nil, page_token: nil)
           opts = {
             :filter => filter,
             :max_results => max_results,
@@ -15,7 +13,7 @@ module Fog
           }
 
           if region
-            data = service.list_forwarding_rules(region, opts).items
+            data = service.list_forwarding_rules(region, opts).items || []
           else
             data = []
             service.list_aggregated_forwarding_rules(opts).items
@@ -29,18 +27,16 @@ module Fog
         end
 
         def get(identity, region = nil)
-          response = nil
           if region
-            response = service.get_forwarding_rule(identity, region).to_h
-          else
+            forwarding_rule = service.get_forwarding_rule(identity, region).to_h
+            return new(forwarding_rule)
+          elsif identity
             response = all(
               :filter => "name eq #{identity}", :max_results => 1
-            ).first
-            response = response.attributes unless response.nil?
+            )
+            forwarding_rule = response.first unless response.empty?
+            return forwarding_rule
           end
-
-          return nil if response.nil?
-          new(response)
         end
       end
     end

data/lib/fog/compute/google/models/global_addresses.rb

@@ -10,8 +10,9 @@ module Fog
         end
 
         def get(identity)
-          if address = service.get_global_address(identity).to_h
-            new(address)
+          if identity
+            address = service.get_global_address(identity).to_h
+            return new(address)
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404

data/lib/fog/compute/google/models/global_forwarding_rules.rb

@@ -10,8 +10,9 @@ module Fog
         end
 
         def get(identity)
-          if rule = service.get_global_forwarding_rule(identity).to_h
-            new(rule)
+          if identity
+            rule = service.get_global_forwarding_rule(identity).to_h
+            return new(rule)
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404

data/lib/fog/compute/google/models/http_health_checks.rb

@@ -10,9 +10,10 @@ module Fog
         end
 
         def get(identity)
-          response = service.get_http_health_check(identity)
-          return nil if response.nil?
-          new(response.to_h)
+          if identity
+            http_health_check = service.get_http_health_check(identity).to_h
+            return new(http_health_check)
+          end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404
           nil

data/lib/fog/compute/google/models/images.rb

@@ -46,22 +46,33 @@ module Fog
         end
 
         def get(identity, project = nil)
-          project.nil? ? projects = all_projects : projects = [project]
-          data = nil
-          projects.each do |proj|
+          if project
             begin
-              data = service.get_image(identity, proj).to_h
-              data[:project] = proj
+              image = service.get_image(identity, project).to_h
+              # TODO: Remove response modification - see #405
+              image[:project] = project
+              return new(image)
             rescue ::Google::Apis::ClientError => e
-              next if e.status_code == 404
-              break
-            else
-              break
+              raise e unless e.status_code == 404
+              nil
+            end
+          elsif identity
+            projects = all_projects
+            projects.each do |proj|
+              begin
+                response = service.get_image(identity, proj).to_h
+                # TODO: Remove response modification - see #405
+                response[:project] = proj
+                image = response
+                return new(image)
+              rescue ::Google::Apis::ClientError => e
+                next if e.status_code == 404
+                break
+              end
             end
+            # If nothing is found - return nil
+            nil
           end
-
-          return nil if data.nil?
-          new(data)
         end
 
         def get_from_family(family, project = nil)

data/lib/fog/compute/google/models/instance_group_managers.rb

@@ -12,10 +12,11 @@ module Fog
             :order_by => order_by,
             :page_token => page_token
           }
-          data = []
+
           if zone
-            data += service.list_instance_group_managers(zone, opts).items || []
+            data = service.list_instance_group_managers(zone, opts).items || []
           else
+            data = []
             service.list_aggregated_instance_group_managers(opts).items.each_value do |group|
               data.concat(group.instance_group_managers) if group.instance_group_managers
             end
@@ -26,11 +27,13 @@ module Fog
 
         def get(identity, zone = nil)
           if zone
-            if instance_group_manager = service.get_instance_group_manager(identity, zone)
-              new(instance_group_manager.to_h)
-            end
-          else
-            all(:filter => "name eq .*#{identity}").first
+            instance_group_manager = service.get_instance_group_manager(identity, zone).to_h
+            return new(instance_group_manager)
+          elsif identity
+            response = all(:filter => "name eq .*#{identity}",
+                           :max_results => 1)
+            instance_group_manager = response.first unless response.empty?
+            return instance_group_manager
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404

data/lib/fog/compute/google/models/instance_groups.rb

@@ -4,13 +4,20 @@ module Fog
       class InstanceGroups < Fog::Collection
         model Fog::Compute::Google::InstanceGroup
 
-        def all(filters = {})
-          if filters[:zone]
-            data = Array(service.list_instance_groups(filters[:zone]))
+        def all(zone: nil, filter: nil, max_results: nil, order_by: nil, page_token: nil)
+          opts = {
+            :filter => filter,
+            :max_results => max_results,
+            :order_by => order_by,
+            :page_token => page_token
+          }
+
+          if zone
+            data = service.list_instance_groups(zone).items || []
           else
             data = []
-            service.list_aggregated_instance_groups.items.each_value do |group|
-              data.concat(group.instance_groups) if group.instance_groups
+            service.list_aggregated_instance_groups(opts).items.each_value do |group|
+              data.concat(group.instance_groups) if group && group.instance_groups
             end
           end
 
@@ -18,19 +25,18 @@ module Fog
         end
 
         def get(identity, zone = nil)
-          if zone.nil?
-            zones = service.list_aggregated_instance_groups(:filter => "name eq .*#{identity}").items
-            instance_groups = zones.each_value.map(&:instance_groups).compact.first
-            if instance_groups
-              zone = instance_groups.first.zone.split("/")[-1]
-            end
-          end
-
-          if instance_group = service.get_instance_group(identity, zone)
-            new(instance_group.to_h)
+          if zone
+            instance_group = service.get_instance_group(identity, zone).to_h
+            new(instance_group)
+          elsif identity
+            response = all(:filter => "name eq #{identity}",
+                           :max_results => 1)
+            instance_group = response.first unless response.empty?
+            return instance_group
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404
+
           nil
         end