fix(deps): update dependency gcs-resumable-upload to v1

[renovate]

This PR contains the following updates:

Package	Type	Update	Change	References
gcs-resumable-upload	dependencies	major	^0.14.1 -> ^1.0.0	source

---

Release Notes

googleapis/gcs-resumable-upload

v1.0.0

Compare Source

02-28-2019 06:27 PST

This release has breaking changes. The underlying transport library was changed from request to gaxios. Any response objects returned via the API will now return a GaxiosResponse object.

Old Code

.on('response', function (resp, metadata) {
  console.log(resp.statusCode);
})

New Code

.on('response', function (resp) {
  console.log(resp.status);
});

Implementation Changes

• fix: replace request with gaxios (#​174)

Documentation

• docs: update links in contrib guide (#​184)
• docs: add lint/fix example to contributing guide (#​177)

Internal / Testing Changes

• chore(deps): update dependency mocha to v6 (#​185)
• build: use linkinator for docs test (#​183)
• build: create docs test npm scripts (#​182)
• build: test using @​grpc/grpc-js in CI (#​181)
• chore: move CONTRIBUTING.md to root (#​179)
• chore(deps): update dependency typescript to ~3.3.0 (#​176)

---

Renovate configuration

📅 Schedule: "after 9am and before 3pm" (UTC).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR has been generated by Renovate Bot. View repository job log here.

[JustinBeckwith]

@stephenplusplus can you figure out what we need to do to land this one?

[codecov]

Codecov Report

Merging #619 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #619   +/-   ##
=======================================
  Coverage   97.94%   97.94%           
=======================================
  Files           9        9           
  Lines         874      874           
  Branches       99       99           
=======================================
  Hits          856      856           
  Misses          9        9           
  Partials        9        9

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23a3cf0...38b1e8d. Read the comment docs.

[stephenplusplus]

I was looking into this and it actually is broken on Node 8.

[JustinBeckwith]

Oooooops. But it looked like the tests were all passing after the gaxios change?

[stephenplusplus]

The gcs-resumable-upload tests pass, but something isn't working with these tests :\

Locally, too:

  1) storage
       write, read, and remove files
         stream write
           should resume an upload after an interruption:
     Uncaught Upload Client Broken Connection

[JustinBeckwith]

I think I see what happened. We landed the PR to run system tests on node 10 yesterday. On re-run - all of the tests passed 🤦‍♂️

[stephenplusplus]

Yeah :( I gave this issue a shout-out over here: googleapis/google-cloud-node#2905 in case we want to think about a way around this kinda thing bumping into us again in the future.