feat!: drop node8 support, support for async iterators (#335)

BREAKING CHANGE: The library now supports Node.js v10+. The last version to support Node.js v8 is tagged legacy-8 on NPM.

New feature: methods with pagination now support async iteration.

.eslintrc.json

@@ -0,0 +1,3 @@
+{
+  "extends": "./node_modules/gts"
+}

.github/workflows/ci.yaml

@@ -9,7 +9,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [8, 10, 12, 13]
+        node: [10, 12, 13]
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1

.prettierrc.js

@@ -0,0 +1,17 @@
+// 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
+//
+//      http://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.
+
+module.exports = {
+  ...require('gts/.prettierrc.json')
+}

package.json

@@ -5,7 +5,7 @@
   "license": "Apache-2.0",
   "author": "Google Inc",
   "engines": {
-    "node": ">=8.10.0"
+    "node": ">=10"
   },
   "repository": "googleapis/nodejs-bigquery-data-transfer",
   "main": "build/src/index.js",
@@ -42,17 +42,18 @@
     "prelint": "cd samples; npm link ../; npm i"
   },
   "dependencies": {
-    "google-gax": "^1.9.0"
+    "google-gax": "^2.0.1"
   },
   "devDependencies": {
     "@types/mocha": "^5.2.5",
     "@types/node": "^12.0.0",
+    "@types/sinon": "^7.5.2",
     "c8": "^7.0.0",
     "eslint": "^6.0.0",
     "eslint-config-prettier": "^6.0.0",
     "eslint-plugin-node": "^11.0.0",
     "eslint-plugin-prettier": "^3.1.0",
-    "gts": "^1.0.0",
+    "gts": "2.0.0-alpha.9",
     "jsdoc": "^3.6.2",
     "jsdoc-fresh": "^1.0.1",
     "jsdoc-region-tag": "^1.0.2",
@@ -61,8 +62,9 @@
     "null-loader": "^3.0.0",
     "pack-n-play": "^1.0.0-2",
     "prettier": "^1.11.1",
+    "sinon": "^9.0.1",
     "ts-loader": "^6.2.1",
-    "typescript": "^3.7.0",
+    "typescript": "^3.8.3",
     "webpack": "^4.41.2",
     "webpack-cli": "^3.3.10"
   }

smoke-test/data_transfer_service_smoke_test.js

@@ -23,6 +23,7 @@ describe('DataTransferServiceSmokeTest', () => {
   const projectId = process.env.GCLOUD_PROJECT;
 
   it('successfully makes a call to the service using promises', done => {
+    // eslint-disable-next-line node/no-missing-require
     const bigqueryDataTransfer = require('../src');
 
     const client = new bigqueryDataTransfer.v1.DataTransferServiceClient({
@@ -45,6 +46,7 @@ describe('DataTransferServiceSmokeTest', () => {
   });
 
   it('successfully makes a call to the service using callbacks', done => {
+    // eslint-disable-next-line node/no-missing-require
     const bigqueryDataTransfer = require('../src');
 
     const client = new bigqueryDataTransfer.v1.DataTransferServiceClient({
@@ -78,6 +80,7 @@ describe('DataTransferServiceSmokeTest', () => {
   });
 
   it('successfully makes a call to the service using streaming', done => {
+    // eslint-disable-next-line node/no-missing-require
     const bigqueryDataTransfer = require('../src');
 
     const client = new bigqueryDataTransfer.v1.DataTransferServiceClient({