gcloud 0.6.1 → 0.6.2

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ODE4NzljMmM3NWM4ZTAzNjExY2E2YWEwMWQ5MjY3YzdiZjAwOGJmOQ==
+    MDViNDRlOTQwOWM0MzQyM2IwNDJhODBkOWU1NzU3N2MxZjg4M2NiYw==
   data.tar.gz: !binary |-
-    MDdlNWYxMjBiMjk1Mjc1ZTNjNjRkZmU0Y2M0NmZiMjVmOWVmODI4NA==
+    YmEyNDM2ZTM1N2RmNWM0MzliYzU1NTAwMmYwZDdiYTAzN2M4OTM3OA==
 SHA512:
   metadata.gz: !binary |-
-    ZDRmODU3NjQ0MWIxYjU3NWVlMGU1YTZjMjQ5ZDQ1ZmVlZjQ0ZDUwODVkNjA0
-    NjcyNGJlMTFkOGVjNmJiZjkwZGMzMzlhNmMzYjI1ZGVlZmE5ZDhmYzU3OGVi
-    ODBmNDM1ZmEzYjAzNDkwZjBmMzhiYmE2MjZlMjM5MzgyM2MxMmY=
+    OWEzOTIyMjBmZjZkZDNkMjcwMjgzOTNmY2I2NzNiN2I0ZGFmYTllNjQyNDQy
+    NjY0Yzk4OTRjN2MzZGQyYzEyNWJmMzg4NWZmMDNkZTQxNDJhODU1ZTExZWUx
+    YmYyMzVkNTQ5NWZiMTllMDc0Y2IxZWY1Y2RmMmQyOWYzZjc3YWE=
   data.tar.gz: !binary |-
-    YWNmNzNiNzBlOTA1NjZjOGIwYzlhYzdiYTg3YTU1MWM4NTVlZjU4MjJkNDU2
-    YmUwNTdiZWEyMzJhYmNiZTNmNzhhZTdhY2EzMTkyOTU0Y2EyZWQyNWQ0YmUz
-    OTkwNGJiMGVhZjIzNDY0MmUyOWY0NzI0ZjhkZDBhOGRmZDYzNjU=
+    ZGYzNzFlMjMyOWY5YWM2MWYxYmJkN2NmYjE0Yzc3ZWI2ZjRjODI4MGEyZGY1
+    OTdjZWQ2MzE3ZDI4YTA5ZTUzNjkxMTUyOGJjMTQ0ZGY5ZDgyOGFhZmJiNWFh
+    ZWNmNzgyNjRkNTUyZDg0M2YzNDlhMzA4NmZmOGYyYTI1MzY0NGE=

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Release History
 
+### 0.6.2 / 2016-03-02
+
+#### Changes
+
+* BigQuery
+  * Fix error accessing a job's data before the job is complete (yhirano55)
+  * Fix undefined local variable in Dataset's access rules (yhirano55)
+  * Optionally specify location when creating a Dataset (gramos74)
+* Datastore
+  * Fix bug in calculating an Entity's `exclude_from_indexes` (bmclean)
+* Pub/Sub
+  * Correctly raise error when accessing a policy without permissions
+  * Update policy permissions documentation
+
 ### 0.6.1 / 2015-12-29
 
 #### Changes

data/lib/gcloud/bigquery/connection.rb

@@ -355,7 +355,8 @@ module Gcloud
           "friendlyName" => options[:name],
           "description" => options[:description],
           "defaultTableExpirationMs" => options[:expiration],
-          "access" => options[:access]
+          "access" => options[:access],
+          "location" => options[:location]
         }.delete_if { |_, v| v.nil? }
       end
 

data/lib/gcloud/bigquery/data.rb

@@ -93,8 +93,8 @@ module Gcloud
       # Disabled rubocop because this implementation will not last.
 
       def self.format_rows rows, fields
-        headers = fields.map { |f| f["name"] }
-        field_types = fields.map { |f| f["type"] }
+        headers = Array(fields).map { |f| f["name"] }
+        field_types = Array(fields).map { |f| f["type"] }
 
         Array(rows).map do |row|
           values = row["f"].map { |f| f["v"] }

data/lib/gcloud/bigquery/dataset/access.rb

@@ -426,7 +426,7 @@ module Gcloud
         def validate_special_group value #:nodoc:
           good_value = GROUPS[value.to_s]
           return good_value unless good_value.nil?
-          scope
+          value
         end
 
         def validate_view view #:nodoc:

data/lib/gcloud/bigquery/load_job.rb

@@ -50,18 +50,14 @@ module Gcloud
       # The delimiter used between fields in the source data. The default is a
       # comma (,).
       def delimiter
-        val = config["load"]["fieldDelimiter"]
-        val = "," if val.nil?
-        val
+        config["load"]["fieldDelimiter"] || ","
       end
 
       ##
       # The number of header rows at the top of a CSV file to skip. The default
       # value is +0+.
       def skip_leading_rows
-        val = config["load"]["skipLeadingRows"]
-        val = 0 if val.nil?
-        val
+        config["load"]["skipLeadingRows"] || 0
       end
 
       ##

data/lib/gcloud/bigquery/project.rb

@@ -295,6 +295,9 @@ module Gcloud
       #   data structure of an array of hashes. See {BigQuery Access
       #   Control}[https://cloud.google.com/bigquery/access-control] for more
       #   information. (+Array of Hashes+)
+      # +location+::
+      #   The geographic location where the dataset should reside. Possible
+      #   values include +EU+ and +US+. The default value is +US+.
       #
       # === Returns
       #
@@ -343,7 +346,7 @@ module Gcloud
       #   end
       #
       def create_dataset dataset_id, name: nil, description: nil,
-                         expiration: nil, access: nil
+                         expiration: nil, access: nil, location: nil
         if block_given?
           access_builder = Dataset::Access.new connection.default_access_rules,
                                                "projectId" => project
@@ -353,7 +356,7 @@ module Gcloud
 
         ensure_connection!
         options = { name: name, description: description,
-                    expiration: expiration, access: access }
+                    expiration: expiration, access: access, location: location }
         resp = connection.insert_dataset dataset_id, options
         return Dataset.from_gapi(resp.data, connection) if resp.success?
         fail ApiError.from_response(resp)

data/lib/gcloud/bigquery/query_data.rb

@@ -106,8 +106,12 @@ module Gcloud
       ##
       # New Data from a response object.
       def self.from_gapi gapi, connection #:nodoc:
-        formatted_rows = format_rows gapi["rows"],
-                                     gapi["schema"]["fields"]
+        if gapi["schema"].nil?
+          formatted_rows = []
+        else
+          formatted_rows = format_rows gapi["rows"],
+                                       gapi["schema"]["fields"]
+        end
 
         data = new formatted_rows
         data.gapi = gapi

data/lib/gcloud/datastore/entity.rb

@@ -314,9 +314,9 @@ module Gcloud
       def update_exclude_indexes! entity #:nodoc:
         @_exclude_indexes = {}
         Array(entity.property).each do |property|
-          @_exclude_indexes[property.name] = property.value.indexed
+          @_exclude_indexes[property.name] = !property.value.indexed
           unless property.value.list_value.nil?
-            exclude = Array(property.value.list_value).map(&:indexed)
+            exclude = Array(property.value.list_value).map{|v| !v.indexed}
             @_exclude_indexes[property.name] = exclude
           end
         end

data/lib/gcloud/pubsub/subscription.rb

@@ -498,6 +498,7 @@ module Gcloud
         @policy ||= begin
           ensure_connection!
           resp = connection.get_subscription_policy name
+          fail ApiError.from_response(resp) unless resp.success?
           policy = resp.data
           policy = policy.to_hash if policy.respond_to? :to_hash
           policy
@@ -559,6 +560,15 @@ module Gcloud
       #   (such as +*+ or +storage.*+) are not allowed.
       #   (String or Array of Strings)
       #
+      #   The permissions that can be checked on a subscription are:
+      #
+      #   * pubsub.subscriptions.consume
+      #   * pubsub.subscriptions.get
+      #   * pubsub.subscriptions.delete
+      #   * pubsub.subscriptions.update
+      #   * pubsub.subscriptions.getIamPolicy
+      #   * pubsub.subscriptions.setIamPolicy
+      #
       # === Returns
       #
       # The permissions that have access. (Array of Strings)
@@ -570,10 +580,10 @@ module Gcloud
       #   gcloud = Gcloud.new
       #   pubsub = gcloud.pubsub
       #   sub = pubsub.subscription "my-subscription"
-      #   perms = sub.test_permissions "projects.subscriptions.list",
-      #                                "projects.subscriptions.pull"
-      #   perms.include? "projects.subscriptions.list" #=> true
-      #   perms.include? "projects.subscriptions.pull" #=> false
+      #   perms = sub.test_permissions "pubsub.subscriptions.get",
+      #                                "pubsub.subscriptions.consume"
+      #   perms.include? "pubsub.subscriptions.get" #=> true
+      #   perms.include? "pubsub.subscriptions.consume" #=> false
       #
       def test_permissions *permissions
         permissions = Array(permissions).flatten

data/lib/gcloud/pubsub/topic.rb

@@ -395,6 +395,7 @@ module Gcloud
         @policy ||= begin
           ensure_connection!
           resp = connection.get_topic_policy name
+          fail ApiError.from_response(resp) unless resp.success?
           policy = resp.data
           policy = policy.to_hash if policy.respond_to? :to_hash
           policy
@@ -456,6 +457,16 @@ module Gcloud
       #   (such as +*+ or +storage.*+) are not allowed.
       #   (String or Array of Strings)
       #
+      #   The permissions that can be checked on a topic are:
+      #
+      #   * pubsub.topics.publish
+      #   * pubsub.topics.attachSubscription
+      #   * pubsub.topics.get
+      #   * pubsub.topics.delete
+      #   * pubsub.topics.update
+      #   * pubsub.topics.getIamPolicy
+      #   * pubsub.topics.setIamPolicy
+      #
       # === Returns
       #
       # The permissions that have access. (Array of Strings)
@@ -467,10 +478,10 @@ module Gcloud
       #   gcloud = Gcloud.new
       #   pubsub = gcloud.pubsub
       #   topic = pubsub.topic "my-topic"
-      #   perms = topic.test_permissions "projects.topic.list",
-      #                                  "projects.topic.publish"
-      #   perms.include? "projects.topic.list" #=> true
-      #   perms.include? "projects.topic.publish" #=> false
+      #   perms = topic.test_permissions "pubsub.topics.get",
+      #                                  "pubsub.topics.publish"
+      #   perms.include? "pubsub.topics.get" #=> true
+      #   perms.include? "pubsub.topics.publish" #=> false
       #
       def test_permissions *permissions
         permissions = Array(permissions).flatten

data/lib/gcloud/version.rb

@@ -16,5 +16,5 @@
 #--
 # Google Cloud Version
 module Gcloud
-  VERSION = "0.6.1"
+  VERSION = "0.6.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gcloud
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - Silvano Luciani
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-30 00:00:00.000000000 Z
+date: 2016-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: beefcake
@@ -170,16 +170,16 @@ dependencies:
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - <=
       - !ruby/object:Gem::Version
-        version: '0.27'
+        version: 0.35.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - <=
       - !ruby/object:Gem::Version
-        version: '0.27'
+        version: 0.35.1
 - !ruby/object:Gem::Dependency
   name: httpclient
   requirement: !ruby/object:Gem::Requirement