Skip to content

Commit

Permalink
feat: support apiEndpoint override in client constructor (#173)
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 3cfeaf2 commit 67ff0dc
Show file tree
Hide file tree
Showing 3 changed files with 39 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -65,14 +65,18 @@ class KeyManagementServiceClient {
* 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 @@ -245,6 +249,14 @@ class KeyManagementServiceClient {
return 'cloudkms.googleapis.com';
}

/**
* The DNS address for this API service - same as servicePath(),
* exists for compatibility reasons.
*/
static get apiEndpoint() {
return 'cloudkms.googleapis.com';
}

/**
* The port for this API service.
*/
Expand Down
10 changes: 5 additions & 5 deletions packages/google-cloud-kms/synth.metadata
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
{
"updateTime": "2019-05-21T11:17:12.709646Z",
"updateTime": "2019-06-05T14:19:24.011735Z",
"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
21 changes: 21 additions & 0 deletions packages/google-cloud-kms/test/gapic-v1.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,27 @@ const error = new Error();
error.code = FAKE_STATUS_CODE;

describe('KeyManagementServiceClient', () => {
it('has servicePath', () => {
const servicePath = kmsModule.v1.KeyManagementServiceClient.servicePath;
assert(servicePath);
});

it('has apiEndpoint', () => {
const apiEndpoint = kmsModule.v1.KeyManagementServiceClient.apiEndpoint;
assert(apiEndpoint);
});

it('has port', () => {
const port = kmsModule.v1.KeyManagementServiceClient.port;
assert(port);
assert(typeof port === 'number');
});

it('should create a client with no options', () => {
const client = new kmsModule.v1.KeyManagementServiceClient();
assert(client);
});

describe('listKeyRings', () => {
it('invokes listKeyRings without error', done => {
const client = new kmsModule.v1.KeyManagementServiceClient({
Expand Down

0 comments on commit 67ff0dc

Please sign in to comment.