From 72f9bbbd76e788e62bb399110efd4fc3eaa6893d Mon Sep 17 00:00:00 2001 From: Yoshi Automation Bot Date: Wed, 5 Jun 2019 09:56:54 -0700 Subject: [PATCH] feat: support apiEndpoint override in client constructor (#219) --- .../src/v1/data_transfer_service_client.js | 14 ++++++++++- .../synth.metadata | 10 ++++---- .../test/gapic-v1.js | 23 +++++++++++++++++++ 3 files changed, 41 insertions(+), 6 deletions(-) diff --git a/packages/google-cloud-bigquery-datatransfer/src/v1/data_transfer_service_client.js b/packages/google-cloud-bigquery-datatransfer/src/v1/data_transfer_service_client.js index bb46dc3455b..51e13dc7a0d 100644 --- a/packages/google-cloud-bigquery-datatransfer/src/v1/data_transfer_service_client.js +++ b/packages/google-cloud-bigquery-datatransfer/src/v1/data_transfer_service_client.js @@ -58,14 +58,18 @@ class DataTransferServiceClient { * API remote host. */ constructor(opts) { + opts = opts || {}; this._descriptors = {}; + const servicePath = + opts.servicePath || opts.apiEndpoint || this.constructor.servicePath; + // Ensure that options include the service address and port. opts = Object.assign( { clientConfig: {}, port: this.constructor.port, - servicePath: this.constructor.servicePath, + servicePath, }, opts ); @@ -200,6 +204,14 @@ class DataTransferServiceClient { return 'bigquerydatatransfer.googleapis.com'; } + /** + * The DNS address for this API service - same as servicePath(), + * exists for compatibility reasons. + */ + static get apiEndpoint() { + return 'bigquerydatatransfer.googleapis.com'; + } + /** * The port for this API service. */ diff --git a/packages/google-cloud-bigquery-datatransfer/synth.metadata b/packages/google-cloud-bigquery-datatransfer/synth.metadata index a8631ab8b6e..c4a4cd07156 100644 --- a/packages/google-cloud-bigquery-datatransfer/synth.metadata +++ b/packages/google-cloud-bigquery-datatransfer/synth.metadata @@ -1,19 +1,19 @@ { - "updateTime": "2019-05-21T11:08:45.880007Z", + "updateTime": "2019-06-05T14:13:40.674076Z", "sources": [ { "generator": { "name": "artman", - "version": "0.20.0", - "dockerImage": "googleapis/artman@sha256:3246adac900f4bdbd62920e80de2e5877380e44036b3feae13667ec255ebf5ec" + "version": "0.23.1", + "dockerImage": "googleapis/artman@sha256:9d5cae1454da64ac3a87028f8ef486b04889e351c83bb95e83b8fab3959faed0" } }, { "git": { "name": "googleapis", "remote": "https://github.com/googleapis/googleapis.git", - "sha": "32a10f69e2c9ce15bba13ab1ff928bacebb25160", - "internalRef": "249058354" + "sha": "47c142a7cecc6efc9f6f8af804b8be55392b795b", + "internalRef": "251635729" } }, { diff --git a/packages/google-cloud-bigquery-datatransfer/test/gapic-v1.js b/packages/google-cloud-bigquery-datatransfer/test/gapic-v1.js index a38135b2485..4c6f53ed994 100644 --- a/packages/google-cloud-bigquery-datatransfer/test/gapic-v1.js +++ b/packages/google-cloud-bigquery-datatransfer/test/gapic-v1.js @@ -23,6 +23,29 @@ const error = new Error(); error.code = FAKE_STATUS_CODE; describe('DataTransferServiceClient', () => { + it('has servicePath', () => { + const servicePath = + bigqueryDataTransferModule.v1.DataTransferServiceClient.servicePath; + assert(servicePath); + }); + + it('has apiEndpoint', () => { + const apiEndpoint = + bigqueryDataTransferModule.v1.DataTransferServiceClient.apiEndpoint; + assert(apiEndpoint); + }); + + it('has port', () => { + const port = bigqueryDataTransferModule.v1.DataTransferServiceClient.port; + assert(port); + assert(typeof port === 'number'); + }); + + it('should create a client with no options', () => { + const client = new bigqueryDataTransferModule.v1.DataTransferServiceClient(); + assert(client); + }); + describe('getDataSource', () => { it('invokes getDataSource without error', done => { const client = new bigqueryDataTransferModule.v1.DataTransferServiceClient(