chor(storage): Restart & delete resumable upload part 2

[shubhangi-google]

Feature to support Restart and Delete resumable upload functionality

@example   

storage = Google::Cloud::Storage.new
bucket = storage.bucket bucket_name
## sample upload_id=  "AKDAyIs3wDvKFtl2e4PopZi-YtGZDxFr2vlR4Z58T6hL9Seypi5W2s7vrRe6-Qw5Fur79g11bT7MtcsSu-vYRkm5nVUKPpm_ciLPn65GQDqNMQ"
#Initiating Resumable upload 
bucket.create_file file, file_name
#Restarting  Resumable upload 
bucket.restart_resumable_upload file, upload_id
#Deleting Resumable upload 
bucket.delete_resumable_upload upload_id

storage library Implementation linked with- googleapis/google-cloud-ruby#29215
Core library Implementation(merged): #21896

[bajajneha27]

Shouldn't we have to update the tests accordingly?

[shubhangi-google]

Shouldn't we have to update the tests accordingly?

updating the test case is not required since we are setting the value of command.delete_upload here
https://github.com/googleapis/google-api-ruby-client/pull/21896/files#diff-064196c57cc405543bd5ca673ab208b2faf5b7526017522e502e87119961b6b0R379

[bajajneha27]

Shouldn't we have to update the tests accordingly?

updating the test case is not required since we are setting the value of command.delete_upload here https://github.com/googleapis/google-api-ruby-client/pull/21896/files#diff-064196c57cc405543bd5ca673ab208b2faf5b7526017522e502e87119961b6b0R379

I can see one place for sure https://github.com/googleapis/google-api-ruby-client/pull/21896/files#diff-760f5664ec67b31a778b73e99c7976822f091ef3ad45eb654a3704c72fb7f555R299

[shubhangi-google]

Shouldn't we have to update the tests accordingly?

updating the test case is not required since we are setting the value of command.delete_upload here https://github.com/googleapis/google-api-ruby-client/pull/21896/files#diff-064196c57cc405543bd5ca673ab208b2faf5b7526017522e502e87119961b6b0R379

I can see one place for sure https://github.com/googleapis/google-api-ruby-client/pull/21896/files#diff-760f5664ec67b31a778b73e99c7976822f091ef3ad45eb654a3704c72fb7f555R299

missed it as this was not affecting the test outcome updated it now

[bajajneha27]

@dazuma I'm not sure about the new check multi-approver. I'm not able to merge the PR with just my approval.