Restart delete resumable upload v4 test1

.toys/batch-review-legacy.rb

@@ -41,7 +41,7 @@
       /\.release-please-manifest\.json$/,
       /\/CHANGELOG\.md$/,
       /\/version\.rb$/,
-      /\/snippets\/snippet_metadata_[\w\.]+\.json$/,
+      /\/snippets\/snippet_metadata_[\w.]+\.json$/,
     ],
   },
   "releases-wrappers" => {
@@ -62,7 +62,7 @@
       /\.release-please-manifest\.json$/,
       /\/CHANGELOG\.md$/,
       /\/version\.rb$/,
-      /\/snippets\/snippet_metadata_[\w\.]+\.json$/,
+      /\/snippets\/snippet_metadata_[\w.]+\.json$/,
     ],
   },
 }.freeze

.toys/batch-review.rb

@@ -31,7 +31,7 @@
 batch_reviewer.define_preset "releases", based_on: :basic_releases do |preset|
   preset.message << :pr_title_number
   preset.diff_expectations.expect name: "snippet metadata" do |expect|
-    expect.change_type(:changed).path_pattern(/\/snippets\/snippet_metadata_[\w\.]+\.json$/)
+    expect.change_type(:changed).path_pattern(/\/snippets\/snippet_metadata_[\w.]+\.json$/)
   end
   preset.diff_expectations.expect name: "repo metadata" do |expect|
     expect.change_type(:changed).path_pattern(/\/\.repo-metadata\.json$/)

google-cloud-storage/google-cloud-storage.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |gem|
   gem.add_dependency "google-cloud-core", "~> 1.6"
   gem.add_dependency "google-apis-core", ">= 0.18", "< 2"
   gem.add_dependency "google-apis-iamcredentials_v1", "~> 0.18"
-  gem.add_dependency "google-apis-storage_v1", ">= 0.42"
+  gem.add_dependency "google-apis-storage_v1", ">= 0.56"
   gem.add_dependency "googleauth", "~> 1.9"
   gem.add_dependency "digest-crc", "~> 0.4"
   gem.add_dependency "addressable", "~> 2.8"

google-cloud-storage/lib/google/cloud/storage/bucket.rb

@@ -717,6 +717,30 @@ def default_kms_key= new_default_kms_key
           patch_gapi! :encryption
         end
 
+        ##
+        # Restart resumable upload
+        # @param [String, ::File] file Path of the file on the filesystem to
+        #   upload. Can be an File object, or File-like object such as StringIO.
+        # @param [String] upload_id Unique Id of a resumable upload
+        #
+        # @return [Google::Apis::StorageV1::Object, Integer]
+        #   The object metadata on success, or 0 request is not completed.
+        #
+        # @example
+        #   require "google/cloud/storage"
+        #
+        #   storage = Google::Cloud::Storage.new
+        #
+        #   bucket = storage.bucket "my-bucket"
+        #   bucket.restart_resumable_upload file, upload_id
+
+        def restart_resumable_upload file, upload_id
+          ensure_service!
+          ensure_io_or_file_exists! file
+          raise ArgumentError, "Upload Id missing" unless upload_id
+          service.restart_resumable_upload name, file, upload_id
+        end
+
         ##
         # The period of time (in seconds) that files in the bucket must be
         # retained, and cannot be deleted, overwritten, or archived.
@@ -1410,6 +1434,26 @@ def delete if_metageneration_match: nil, if_metageneration_not_match: nil
                                 user_project: user_project
         end
 
+        ##
+        # Delete resumable upload
+        # @param [String] upload_id Unique Id of a resumable upload
+        #
+        # @return [Boolean,nil] Returns `true` if the resumable upload was deleted, 'nil' if the request is not completed.
+        #
+        # @example
+        #   require "google/cloud/storage"
+        #
+        #   storage = Google::Cloud::Storage.new
+        #
+        #   bucket = storage.bucket "my-bucket"
+        #   bucket.delete_resumable_upload upload_id
+
+        def delete_resumable_upload upload_id
+          ensure_service!
+          raise ArgumentError, "Upload Id missing" unless upload_id
+          service.delete_resumable_upload name, upload_id
+        end
+
         ##
         # Retrieves a list of files matching the criteria.
         #

google-cloud-storage/lib/google/cloud/storage/service.rb

@@ -696,6 +696,18 @@ def delete_file bucket_name,
           end
         end
 
+        def restart_resumable_upload bucket_name, source, upload_id, options: {}
+          execute do
+            service.restart_resumable_upload bucket_name, source, upload_id, options: options
+          end
+        end
+
+        def delete_resumable_upload bucket_name, upload_id, options: {}
+          execute do
+            service.delete_resumable_upload bucket_name, upload_id, options: options
+          end
+        end
+
         ##
         # Restore soft deleted bucket
         def restore_bucket bucket_name,

google-cloud-storage/test/google/cloud/storage/bucket_test.rb

@@ -1408,6 +1408,84 @@
     end
   end
 
+  it "restarts a resumable upload with upload_id" do
+    new_file_name = random_file_path
+    upload_id = "TEST_ID"
+
+    Tempfile.open ["google-cloud", ".txt"] do |tmpfile|
+      tmpfile.write "Hello world"
+      tmpfile.rewind
+      mock = Minitest::Mock.new
+      expected_return_value = create_file_gapi(bucket.name, new_file_name)
+      mock.expect :restart_resumable_upload, expected_return_value,
+        [bucket.name, tmpfile, upload_id],
+        **resumable_upload_args(options: {})
+      bucket.service.mocked_service = mock
+      returned_value= bucket.restart_resumable_upload tmpfile, upload_id
+      assert_equal expected_return_value, returned_value
+      mock.verify
+    end
+  end
+
+  it "raises ArgumentError if anyone or both arguments are not provided to restart_resumable_upload" do
+    new_file_name = random_file_path
+    upload_id = "TEST_ID"
+
+    Tempfile.open ["google-cloud", ".txt"] do |tmpfile|
+      tmpfile.write "Hello world"
+      tmpfile.rewind
+      error = _ { bucket.restart_resumable_upload  }.must_raise ArgumentError
+      assert_match "wrong number of arguments", error.message
+
+      error = _ { bucket.restart_resumable_upload upload_id }.must_raise ArgumentError
+      assert_match "wrong number of arguments", error.message
+
+      error = _ { bucket.restart_resumable_upload tmpfile }.must_raise ArgumentError
+      assert_match "wrong number of arguments", error.message
+    end
+  end
+
+  it "raises ArgumentError if upload_id as nil is provided to restart_resumable_upload" do
+    new_file_name = random_file_path
+    upload_id = nil
+
+    Tempfile.open ["google-cloud", ".txt"] do |tmpfile|
+      tmpfile.write "Hello world"
+      tmpfile.rewind
+      error = _ { bucket.restart_resumable_upload tmpfile, upload_id }.must_raise ArgumentError
+      assert_match "Upload Id missing", error.message
+    end
+  end
+
+
+  it "deletes a resumable upload with upload_id" do
+    upload_id = "TEST_ID"
+
+    mock = Minitest::Mock.new
+    expected_return_value = true
+    mock.expect :delete_resumable_upload, expected_return_value,
+      [bucket.name, upload_id],
+      **resumable_upload_args(options: {})
+    bucket.service.mocked_service = mock
+    returned_value = bucket.delete_resumable_upload upload_id
+    assert_equal expected_return_value, returned_value
+    mock.verify
+  end
+
+  it "raises ArgumentError if upload_id is not provided to delete_resumable_upload" do
+    new_file_name = random_file_path
+    upload_id = "TEST_ID"
+    error = _ { bucket.delete_resumable_upload  }.must_raise ArgumentError
+    assert_match "wrong number of arguments", error.message
+  end
+
+  it "raises ArgumentError if upload_id as nil is provided to delete_resumable_upload" do
+    new_file_name = random_file_path
+    upload_id = nil
+    error = _ { bucket.delete_resumable_upload upload_id }.must_raise ArgumentError
+    assert_match "Upload Id missing", error.message
+  end
+
   def create_file_gapi bucket=nil, name = nil
     Google::Apis::StorageV1::Object.from_json random_file_hash(bucket, name).to_json
   end

google-cloud-storage/test/helper.rb

@@ -391,6 +391,12 @@ def insert_object_args name: nil,
     }
   end
 
+  def resumable_upload_args options: {}
+    {
+      options: options
+    }
+  end
+
   def get_object_args generation: nil,
                       if_generation_match: nil,
                       if_generation_not_match: nil,