Skip to content

Commit

Permalink
feat: support apiEndpoint override in client constructor (#219)
Browse files Browse the repository at this point in the history
  • Loading branch information
yoshi-automation authored and JustinBeckwith committed Jun 5, 2019
1 parent 14b1761 commit a079e35
Show file tree
Hide file tree
Showing 3 changed files with 41 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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
);
Expand Down Expand Up @@ -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.
*/
Expand Down
10 changes: 5 additions & 5 deletions packages/google-cloud-bigquery-datatransfer/synth.metadata
Original file line number Diff line number Diff line change
@@ -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"
}
},
{
Expand Down
23 changes: 23 additions & 0 deletions packages/google-cloud-bigquery-datatransfer/test/gapic-v1.js
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down

0 comments on commit a079e35

Please sign in to comment.