google-cloud-firestore 1.4.4 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 767d0de57b4cb69eb636c2dd85e95da99bab28356514daeae0cb29e87aac8989
-  data.tar.gz: 9a2c8ebd59ef8694fcfe1cd78c027acca7442ae638c90b125dbde4f7adc4a8e0
+  metadata.gz: e83411cc86b2d8999f2ea7771cc243f032b9119c03885427aa57b007c3695406
+  data.tar.gz: 2ba1894414f95466c521ac6159dcfeedfb713727049171226a0eaecb2b041a71
 SHA512:
-  metadata.gz: 93d63f0d8d61ff6c32c2b94b0d38e8b1a1deadfdbe3bd104558ffc6fa55f265701dc262fd4f1200ea9d1802d26f5e5f79cd9859452d6128d3b3e49e36eae044c
-  data.tar.gz: d43a9f1f2e7b0aeb0dbfd56f10a41790036f29199d1dec6862f4e97e14fd057baf72ed66a1d414073e5a590cecfbdbb0182524f486e63e4951ffa8aa11d8dfd0
+  metadata.gz: ee5186f01ad3ec6adf7508a043793306c622f77b5a4552f91a787d52860e57e8e65c99561d8995cd7dd3a424801ac7e56b36174e6a9139179bed3ac708391966
+  data.tar.gz: 0c2037885a4d5b12151f346f181de0f632e6c2c399e095899cd9567ad484ac954463c757740c0306639473edf69115a1e8fc37e1914727cbef11d90241aaef9c

data/AUTHENTICATION.md

@@ -76,7 +76,7 @@ The environment variables that google-cloud-firestore checks for project ID are:
 1. `FIRESTORE_PROJECT`
 2. `GOOGLE_CLOUD_PROJECT`
 
-The environment variables that google-cloud-firestore checks for credentials are configured on {Google::Cloud::Firestore::V1::Credentials}:
+The environment variables that google-cloud-firestore checks for credentials are configured on {Google::Cloud::Firestore::Credentials}:
 
 1. `FIRESTORE_CREDENTIALS` - Path to JSON file, or JSON contents
 2. `FIRESTORE_KEYFILE` - Path to JSON file, or JSON contents

data/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Release History
 
+### 2.0.0 / 2020-08-06
+
+This is a major update that removes the "low-level" client interface code, and
+instead adds the new `google-cloud-firestore-v1` gem as a dependency.
+The new dependency is a rewritten low-level client, produced by a next-
+generation client code generator, with improved performance and stability.
+
+This change should have no effect on the high-level interface that most users
+will use. The one exception is that the (mostly undocumented) `client_config`
+argument, for adjusting low-level parameters such as RPC retry settings on
+client objects, has been removed. If you need to adjust these parameters, use
+the configuration interface in `google-cloud-firestore-v1`.
+
+Substantial changes have been made in the low-level interfaces, however. If you
+are using the low-level classes under the `Google::Cloud::Firestore::V1` module,
+please review the docs for the new `google-cloud-firestore-v1` gem. In
+particular:
+
+* Some classes have been renamed, notably the client class itself.
+* The client constructor takes a configuration block instead of configuration
+  keyword arguments.
+* All RPC method arguments are now keyword arguments.
+
 ### 1.4.4 / 2020-05-28
 
 #### Documentation

data/lib/google-cloud-firestore.rb

@@ -42,8 +42,6 @@ module Google
     #
     #   * `https://www.googleapis.com/auth/datastore`
     # @param [Integer] timeout Default timeout to use in requests. Optional.
-    # @param [Hash] client_config A hash of values to override the default
-    #   behavior of the API client. Optional.
     #
     # @return [Google::Cloud::Firestore::Client]
     #
@@ -60,10 +58,8 @@ module Google
     #   platform_scope = "https://www.googleapis.com/auth/cloud-platform"
     #   firestore = gcloud.firestore scope: platform_scope
     #
-    def firestore scope: nil, timeout: nil, client_config: nil
-      Google::Cloud.firestore @project, @keyfile,
-                              scope: scope, timeout: (timeout || @timeout),
-                              client_config: client_config
+    def firestore scope: nil, timeout: nil
+      Google::Cloud.firestore @project, @keyfile, scope: scope, timeout: (timeout || @timeout)
     end
 
     ##
@@ -87,8 +83,6 @@ module Google
     #
     #   * `https://www.googleapis.com/auth/datastore`
     # @param [Integer] timeout Default timeout to use in requests. Optional.
-    # @param [Hash] client_config A hash of values to override the default
-    #   behavior of the API client. Optional.
     #
     # @return [Google::Cloud::Firestore::Client]
     #
@@ -97,13 +91,12 @@ module Google
     #
     #   firestore = Google::Cloud.firestore
     #
-    def self.firestore project_id = nil, credentials = nil, scope: nil,
-                       timeout: nil, client_config: nil
+    def self.firestore project_id = nil, credentials = nil, scope: nil, timeout: nil
       require "google/cloud/firestore"
-      Google::Cloud::Firestore.new project_id: project_id,
+      Google::Cloud::Firestore.new project_id:  project_id,
                                    credentials: credentials,
-                                   scope: scope, timeout: timeout,
-                                   client_config: client_config
+                                   scope:       scope,
+                                   timeout:     timeout
     end
   end
 end
@@ -122,17 +115,17 @@ Google::Cloud.configure.add_config! :firestore do |config|
   default_emulator = Google::Cloud::Config.deferred do
     ENV["FIRESTORE_EMULATOR_HOST"]
   end
+  default_scopes = [
+    "https://www.googleapis.com/auth/cloud-platform",
+    "https://www.googleapis.com/auth/datastore"
+  ]
 
   config.add_field! :project_id, default_project, match: String, allow_nil: true
   config.add_alias! :project, :project_id
-  config.add_field! :credentials, default_creds,
-                    match:     [String, Hash, Google::Auth::Credentials],
-                    allow_nil: true
+  config.add_field! :credentials, default_creds, match: [String, Hash, Google::Auth::Credentials], allow_nil: true
   config.add_alias! :keyfile, :credentials
-  config.add_field! :scope, nil, match: [String, Array]
+  config.add_field! :scope, default_scopes, match: [String, Array]
   config.add_field! :timeout, nil, match: Integer
-  config.add_field! :client_config, nil, match: Hash
-  config.add_field! :emulator_host, default_emulator,
-                    match: String, allow_nil: true
-  config.add_field! :endpoint, nil, match: String
+  config.add_field! :emulator_host, default_emulator, match: String, allow_nil: true
+  config.add_field! :endpoint, "firestore.googleapis.com", match: String
 end

data/lib/google/cloud/firestore.rb

@@ -33,7 +33,6 @@ module Google
     #
     module Firestore
       # rubocop:disable Metrics/AbcSize
-      # rubocop:disable Metrics/MethodLength
 
       ##
       # Creates a new object for connecting to the Firestore service.
@@ -56,8 +55,6 @@ module Google
       #
       #   * `https://www.googleapis.com/auth/datastore`
       # @param [Integer] timeout Default timeout to use in requests. Optional.
-      # @param [Hash] client_config A hash of values to override the default
-      #   behavior of the API client. Optional.
       # @param [String] endpoint Override of the endpoint host name. Optional.
       #   If the param is nil, uses the default endpoint.
       # @param [String] emulator_host Firestore emulator host. Optional.
@@ -73,13 +70,17 @@ module Google
       #
       #   firestore = Google::Cloud::Firestore.new
       #
-      def self.new project_id: nil, credentials: nil, scope: nil, timeout: nil,
-                   client_config: nil, endpoint: nil, emulator_host: nil,
-                   project: nil, keyfile: nil
+      def self.new project_id: nil,
+                   credentials: nil,
+                   scope: nil,
+                   timeout: nil,
+                   endpoint: nil,
+                   emulator_host: nil,
+                   project: nil,
+                   keyfile: nil
         project_id    ||= (project || default_project_id)
         scope         ||= configure.scope
         timeout       ||= configure.timeout
-        client_config ||= configure.client_config
         endpoint      ||= configure.endpoint
         emulator_host ||= configure.emulator_host
 
@@ -87,13 +88,8 @@ module Google
           project_id = project_id.to_s
           raise ArgumentError, "project_id is missing" if project_id.empty?
 
-          return Firestore::Client.new(
-            Firestore::Service.new(
-              project_id, :this_channel_is_insecure,
-              host: emulator_host, timeout: timeout,
-              client_config: client_config
-            )
-          )
+          service = Firestore::Service.new project_id, :this_channel_is_insecure, host: emulator_host, timeout: timeout
+          return Firestore::Client.new service
         end
 
         credentials ||= (keyfile || default_credentials(scope: scope))
@@ -107,16 +103,11 @@ module Google
         project_id = project_id.to_s
         raise ArgumentError, "project_id is missing" if project_id.empty?
 
-        Firestore::Client.new(
-          Firestore::Service.new(
-            project_id, credentials,
-            host: endpoint, timeout: timeout, client_config: client_config
-          )
-        )
+        service = Firestore::Service.new project_id, credentials, host: endpoint, timeout: timeout
+        Firestore::Client.new service
       end
 
       # rubocop:enable Metrics/AbcSize
-      # rubocop:enable Metrics/MethodLength
 
       ##
       # Configure the Google Cloud Firestore library.
@@ -132,8 +123,6 @@ module Google
       # * `scope` - (String, Array<String>) The OAuth 2.0 scopes controlling
       #   the set of resources and operations that the connection can access.
       # * `timeout` - (Integer) Default timeout to use in requests.
-      # * `client_config` - (Hash) A hash of values to override the default
-      #   behavior of the API client.
       # * `endpoint` - (String) Override of the endpoint host name, or `nil`
       #   to use the default endpoint.
       # * `emulator_host` - (String) Host name of the emulator. Defaults to

data/lib/google/cloud/firestore/client.rb

@@ -82,12 +82,13 @@ module Google
         # @!group Access
 
         ##
-        # Retrieves a list of collections.
+        # Retrieves an enumerator for the root collections.
         #
         # @yield [collections] The block for accessing the collections.
-        # @yieldparam [CollectionReference] collection A collection.
+        # @yieldparam [CollectionReference] collection A collection reference object.
         #
-        # @return [Enumerator<CollectionReference>] collection list.
+        # @return [Enumerator<CollectionReference>] An enumerator of collection references. If a block is provided,
+        #  this is the same enumerator that is accessed through the block.
         #
         # @example
         #   require "google/cloud/firestore"
@@ -101,11 +102,10 @@ module Google
         #
         def cols
           ensure_service!
-
-          return enum_for :cols unless block_given?
-
-          collection_ids = service.list_collections "#{path}/documents"
-          collection_ids.each { |collection_id| yield col collection_id }
+          grpc = service.list_collections "#{path}/documents"
+          cols_enum = CollectionReferenceList.from_grpc(grpc, self, "#{path}/documents").all
+          cols_enum.each { |c| yield c } if block_given?
+          cols_enum
         end
         alias collections cols
         alias list_collections cols
@@ -134,8 +134,7 @@ module Google
             raise ArgumentError, "collection_path must refer to a collection."
           end
 
-          CollectionReference.from_path \
-            "#{path}/documents/#{collection_path}", self
+          CollectionReference.from_path "#{path}/documents/#{collection_path}", self
         end
         alias collection col
 
@@ -167,9 +166,9 @@ module Google
             raise ArgumentError, "Invalid collection_id: '#{collection_id}', " \
               "must not contain '/'."
           end
-          query = Google::Firestore::V1::StructuredQuery.new(
+          query = Google::Cloud::Firestore::V1::StructuredQuery.new(
             from: [
-              Google::Firestore::V1::StructuredQuery::CollectionSelector.new(
+              Google::Cloud::Firestore::V1::StructuredQuery::CollectionSelector.new(
                 collection_id: collection_id, all_descendants: true
               )
             ]
@@ -670,8 +669,7 @@ module Google
         # @private
         def list_documents parent, collection_id, token: nil, max: nil
           ensure_service!
-          grpc = service.list_documents \
-            parent, collection_id, token: token, max: max
+          grpc = service.list_documents parent, collection_id, token: token, max: max
           DocumentReference::List.from_grpc grpc, self, parent, collection_id
         end
 

data/lib/google/cloud/firestore/collection_reference.rb

@@ -18,6 +18,7 @@ require "google/cloud/firestore/document_reference"
 require "google/cloud/firestore/document_snapshot"
 require "google/cloud/firestore/query"
 require "google/cloud/firestore/generate"
+require "google/cloud/firestore/collection_reference_list"
 
 module Google
   module Cloud

data/lib/google/cloud/firestore/collection_reference_list.rb

@@ -0,0 +1,137 @@
+# Copyright 2020 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+
+require "delegate"
+
+module Google
+  module Cloud
+    module Firestore
+      ##
+      # @private
+      #
+      # An Array delegate for pagination. Private class exposing only an Enumerator to clients.
+      #
+      class CollectionReferenceList < DelegateClass(::Array)
+        ##
+        # If not empty, indicates that there are more records that match the request and this value should be passed
+        # to continue.
+        attr_accessor :token
+
+        ##
+        # @private Create a new CollectionReference::List with an array of CollectionReference instances.
+        def initialize arr = []
+          super arr
+        end
+
+        ##
+        # Whether there is a next page of collection references.
+        #
+        # @return [Boolean]
+        #
+        def next?
+          !token.nil?
+        end
+
+        ##
+        # Retrieve the next page of collection references.
+        #
+        # @return [CollectionReference::List] The list of collection references.
+        #
+        def next
+          return nil unless next?
+          ensure_service!
+          grpc = @client.service.list_collections @parent, token: token, max: @max
+          self.class.from_grpc grpc, @client, @parent, max: @max
+        end
+
+        ##
+        # Retrieves remaining results by repeatedly invoking {#next} until {#next?} returns `false`. Calls the given
+        # block once for each result, which is passed as the argument to the block.
+        #
+        # An Enumerator is returned if no block is given.
+        #
+        # This method will make repeated API calls until all remaining results are retrieved. (Unlike `#each`, for
+        # example, which merely iterates over the results returned by a single API call.) Use with caution.
+        #
+        # @param [Integer] request_limit The upper limit of API requests to make to load all collection references.
+        #   Default is no limit.
+        # @yield [collection_reference] The block for accessing each collection_reference.
+        # @yieldparam [CollectionReference] collection_reference The collection reference object.
+        #
+        # @return [Enumerator]
+        #
+        # @example Iterating each collection reference by passing a block:
+        #   require "google/cloud/firestore"
+        #
+        #   firestore = Google::Cloud::Firestore.new
+        #
+        #   firestore.cols do |collection_reference|
+        #     puts collection_reference.collection_id
+        #   end
+        #
+        # @example Using the enumerator by not passing a block:
+        #   require "google/cloud/firestore"
+        #
+        #   firestore = Google::Cloud::Firestore.new
+        #
+        #   all_collection_ids = firestore.cols.map do |collection_reference|
+        #     collection_reference.collection_id
+        #   end
+        #
+        def all request_limit: nil
+          request_limit = request_limit.to_i if request_limit
+          unless block_given?
+            return enum_for :all, request_limit: request_limit
+          end
+          results = self
+          loop do
+            results.each { |r| yield r }
+            if request_limit
+              request_limit -= 1
+              break if request_limit < 0
+            end
+            break unless results.next?
+            results = results.next
+          end
+        end
+
+        ##
+        # @private New CollectionReference::List from a `Google::Cloud::Firestore::V1::ListCollectionIdsResponse`
+        # object.
+        def self.from_grpc grpc, client, parent, max: nil
+          raise ArgumentError, "parent is required" unless parent
+          cols = CollectionReferenceList.new(Array(grpc.collection_ids).map do |collection_id|
+            CollectionReference.from_path "#{parent}/#{collection_id}", client
+          end)
+          token = grpc.next_page_token
+          token = nil if token == ""
+          cols.instance_variable_set :@token, token
+          cols.instance_variable_set :@client, client
+          cols.instance_variable_set :@parent, parent
+          cols.instance_variable_set :@max, max
+          cols
+        end
+
+        protected
+
+        ##
+        # Raise an error unless an active service is available.
+        def ensure_service!
+          raise "Must have active connection" unless @client.service
+        end
+      end
+    end
+  end
+end

data/lib/google/cloud/firestore/convert.rb

@@ -85,40 +85,40 @@ module Google
 
           def raw_to_value obj
             if NilClass === obj
-              Google::Firestore::V1::Value.new null_value: :NULL_VALUE
+              Google::Cloud::Firestore::V1::Value.new null_value: :NULL_VALUE
             elsif TrueClass === obj || FalseClass === obj
-              Google::Firestore::V1::Value.new boolean_value: obj
+              Google::Cloud::Firestore::V1::Value.new boolean_value: obj
             elsif Integer === obj
-              Google::Firestore::V1::Value.new integer_value: obj
+              Google::Cloud::Firestore::V1::Value.new integer_value: obj
             elsif Numeric === obj # Any number not an integer is a double
-              Google::Firestore::V1::Value.new double_value: obj.to_f
+              Google::Cloud::Firestore::V1::Value.new double_value: obj.to_f
             elsif Time === obj || DateTime === obj || Date === obj
-              Google::Firestore::V1::Value.new \
+              Google::Cloud::Firestore::V1::Value.new \
                 timestamp_value: time_to_timestamp(obj.to_time)
             elsif String === obj || Symbol === obj
-              Google::Firestore::V1::Value.new string_value: obj.to_s
+              Google::Cloud::Firestore::V1::Value.new string_value: obj.to_s
             elsif Google::Cloud::Firestore::DocumentReference === obj
-              Google::Firestore::V1::Value.new reference_value: obj.path
+              Google::Cloud::Firestore::V1::Value.new reference_value: obj.path
             elsif Array === obj
               values = obj.map { |o| raw_to_value(o) }
-              Google::Firestore::V1::Value.new(array_value:
-                Google::Firestore::V1::ArrayValue.new(values: values))
+              Google::Cloud::Firestore::V1::Value.new(array_value:
+                Google::Cloud::Firestore::V1::ArrayValue.new(values: values))
             elsif Hash === obj
               # keys have been changed to strings before the hash gets here
               geo_pairs = hash_is_geo_point? obj
               if geo_pairs
-                Google::Firestore::V1::Value.new(
+                Google::Cloud::Firestore::V1::Value.new(
                   geo_point_value: hash_to_geo_point(obj, geo_pairs)
                 )
               else
                 fields = hash_to_fields obj
-                Google::Firestore::V1::Value.new(map_value:
-                  Google::Firestore::V1::MapValue.new(fields: fields))
+                Google::Cloud::Firestore::V1::Value.new(map_value:
+                  Google::Cloud::Firestore::V1::MapValue.new(fields: fields))
               end
             elsif obj.respond_to?(:read) && obj.respond_to?(:rewind)
               obj.rewind
               content = obj.read.force_encoding "ASCII-8BIT"
-              Google::Firestore::V1::Value.new bytes_value: content
+              Google::Cloud::Firestore::V1::Value.new bytes_value: content
             else
               raise ArgumentError,
                     "A value of type #{obj.class} is not supported."
@@ -129,7 +129,7 @@ module Google
             return false unless hash.keys.count == 2
 
             pairs = hash.map { |k, v| [String(k), v] }.sort
-            if pairs.map(&:first) == ["latitude".freeze, "longitude".freeze]
+            if pairs.map(&:first) == ["latitude", "longitude"]
               pairs
             end
           end
@@ -156,11 +156,11 @@ module Google
             data, field_paths_and_values = remove_field_value_from data
 
             if data.any? || field_paths_and_values.empty?
-              write = Google::Firestore::V1::Write.new(
-                update: Google::Firestore::V1::Document.new(
+              write = Google::Cloud::Firestore::V1::Write.new(
+                update: Google::Cloud::Firestore::V1::Document.new(
                   name: doc_path,
                   fields: hash_to_fields(data)),
-                current_document: Google::Firestore::V1::Precondition.new(
+                current_document: Google::Cloud::Firestore::V1::Precondition.new(
                   exists: false)
               )
               writes << write
@@ -171,7 +171,7 @@ module Google
 
               if data.empty?
                 transform_write.current_document = \
-                  Google::Firestore::V1::Precondition.new(exists: false)
+                  Google::Cloud::Firestore::V1::Precondition.new(exists: false)
               end
 
               writes << transform_write
@@ -207,8 +207,8 @@ module Google
 
             data, field_paths_and_values = remove_field_value_from data
 
-            writes << Google::Firestore::V1::Write.new(
-              update: Google::Firestore::V1::Document.new(
+            writes << Google::Cloud::Firestore::V1::Write.new(
+              update: Google::Cloud::Firestore::V1::Document.new(
                 name: doc_path,
                 fields: hash_to_fields(data))
             )
@@ -278,11 +278,11 @@ module Google
             end
 
             if data.any? || field_paths.any? || (allow_empty && field_paths_and_values.empty?)
-              writes << Google::Firestore::V1::Write.new(
-                update: Google::Firestore::V1::Document.new(
+              writes << Google::Cloud::Firestore::V1::Write.new(
+                update: Google::Cloud::Firestore::V1::Document.new(
                   name: doc_path,
                   fields: hash_to_fields(data)),
-                update_mask: Google::Firestore::V1::DocumentMask.new(
+                update_mask: Google::Cloud::Firestore::V1::DocumentMask.new(
                   field_paths: field_paths.map(&:formatted_string).sort)
               )
             end
@@ -339,18 +339,18 @@ module Google
             end
 
             if data.any? || delete_paths.any?
-              write = Google::Firestore::V1::Write.new(
-                update: Google::Firestore::V1::Document.new(
+              write = Google::Cloud::Firestore::V1::Write.new(
+                update: Google::Cloud::Firestore::V1::Document.new(
                   name: doc_path,
                   fields: hash_to_fields(data)),
-                update_mask: Google::Firestore::V1::DocumentMask.new(
+                update_mask: Google::Cloud::Firestore::V1::DocumentMask.new(
                   field_paths: (field_paths).map(&:formatted_string).sort),
-                current_document: Google::Firestore::V1::Precondition.new(
+                current_document: Google::Cloud::Firestore::V1::Precondition.new(
                   exists: true)
               )
               if update_time
                 write.current_document = \
-                  Google::Firestore::V1::Precondition.new(
+                  Google::Cloud::Firestore::V1::Precondition.new(
                     update_time: time_to_timestamp(update_time))
               end
               writes << write
@@ -360,7 +360,7 @@ module Google
               transform_write = transform_write doc_path, field_paths_and_values
               if data.empty?
                 transform_write.current_document = \
-                  Google::Firestore::V1::Precondition.new(exists: true)
+                  Google::Cloud::Firestore::V1::Precondition.new(exists: true)
               end
               writes << transform_write
             end
@@ -373,13 +373,13 @@ module Google
               raise ArgumentError, "cannot specify both exists and update_time"
             end
 
-            write = Google::Firestore::V1::Write.new(
+            write = Google::Cloud::Firestore::V1::Write.new(
               delete: doc_path
             )
 
             unless exists.nil? && update_time.nil?
               write.current_document = \
-                Google::Firestore::V1::Precondition.new({
+                Google::Cloud::Firestore::V1::Precondition.new({
                   exists: exists, update_time: time_to_timestamp(update_time)
                 }.delete_if { |_, v| v.nil? })
             end
@@ -584,8 +584,8 @@ module Google
               to_field_transform field_path, field_value
             end
 
-            Google::Firestore::V1::Write.new(
-              transform: Google::Firestore::V1::DocumentTransform.new(
+            Google::Cloud::Firestore::V1::Write.new(
+              transform: Google::Cloud::Firestore::V1::DocumentTransform.new(
                 document: doc_path,
                 field_transforms: field_transforms
               )
@@ -594,32 +594,32 @@ module Google
 
           def to_field_transform field_path, field_value
             if field_value.type == :server_time
-              Google::Firestore::V1::DocumentTransform::FieldTransform.new(
+              Google::Cloud::Firestore::V1::DocumentTransform::FieldTransform.new(
                 field_path: field_path.formatted_string,
                 set_to_server_value: :REQUEST_TIME
               )
             elsif field_value.type == :array_union
-              Google::Firestore::V1::DocumentTransform::FieldTransform.new(
+              Google::Cloud::Firestore::V1::DocumentTransform::FieldTransform.new(
                 field_path: field_path.formatted_string,
                 append_missing_elements: raw_to_value(Array(field_value.value)).array_value
               )
             elsif field_value.type == :array_delete
-              Google::Firestore::V1::DocumentTransform::FieldTransform.new(
+              Google::Cloud::Firestore::V1::DocumentTransform::FieldTransform.new(
                 field_path: field_path.formatted_string,
                 remove_all_from_array: raw_to_value(Array(field_value.value)).array_value
               )
             elsif field_value.type == :increment
-              Google::Firestore::V1::DocumentTransform::FieldTransform.new(
+              Google::Cloud::Firestore::V1::DocumentTransform::FieldTransform.new(
                 field_path: field_path.formatted_string,
                 increment: raw_to_value(field_value.value)
               )
             elsif field_value.type == :maximum
-              Google::Firestore::V1::DocumentTransform::FieldTransform.new(
+              Google::Cloud::Firestore::V1::DocumentTransform::FieldTransform.new(
                 field_path: field_path.formatted_string,
                 maximum: raw_to_value(field_value.value)
               )
             elsif field_value.type == :minimum
-              Google::Firestore::V1::DocumentTransform::FieldTransform.new(
+              Google::Cloud::Firestore::V1::DocumentTransform::FieldTransform.new(
                 field_path: field_path.formatted_string,
                 minimum: raw_to_value(field_value.value)
               )