Skip to content

Commit

Permalink
chore: add Warn to generated client (#1564)
Browse files Browse the repository at this point in the history
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/70f72a53-eb64-4c25-a7ef-dbae8cf796ea/targets

- [ ] To automatically regenerate this PR, check this box. (May take up to 24 hours.)

PiperOrigin-RevId: 385101839
Source-Link: googleapis/googleapis@80f4042
  • Loading branch information
yoshi-automation authored Jul 27, 2021
1 parent 446abb0 commit 8b38f32
Show file tree
Hide file tree
Showing 7 changed files with 39 additions and 6 deletions.
11 changes: 10 additions & 1 deletion dev/src/v1/firestore_admin_client.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ const version = require('../../../package.json').version;
export class FirestoreAdminClient {
private _terminated = false;
private _opts: ClientOptions;
private _providedCustomServicePath: boolean;
private _gaxModule: typeof gax | typeof gax.fallback;
private _gaxGrpc: gax.GrpcClient | gax.fallback.GrpcClient;
private _protos: {};
Expand All @@ -61,6 +62,7 @@ export class FirestoreAdminClient {
longrunning: {},
batching: {},
};
warn: (code: string, message: string, warnType?: string) => void;
innerApiCalls: {[name: string]: Function};
pathTemplates: {[name: string]: gax.PathTemplate};
operationsClient: gax.OperationsClient;
Expand Down Expand Up @@ -105,6 +107,9 @@ export class FirestoreAdminClient {
const staticMembers = this.constructor as typeof FirestoreAdminClient;
const servicePath =
opts?.servicePath || opts?.apiEndpoint || staticMembers.servicePath;
this._providedCustomServicePath = !!(
opts?.servicePath || opts?.apiEndpoint
);
const port = opts?.port || staticMembers.port;
const clientConfig = opts?.clientConfig ?? {};
const fallback =
Expand Down Expand Up @@ -258,6 +263,9 @@ export class FirestoreAdminClient {
// of calling the API is handled in `google-gax`, with this code
// merely providing the destination and request information.
this.innerApiCalls = {};

// Add a warn function to the client constructor so it can be easily tested.
this.warn = gax.warn;
}

/**
Expand Down Expand Up @@ -286,7 +294,8 @@ export class FirestoreAdminClient {
)
: // eslint-disable-next-line @typescript-eslint/no-explicit-any
(this._protos as any).google.firestore.admin.v1.FirestoreAdmin,
this._opts
this._opts,
this._providedCustomServicePath
) as Promise<{[method: string]: Function}>;

// Iterate over each of the methods that the service provides
Expand Down
11 changes: 10 additions & 1 deletion dev/src/v1/firestore_client.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ const version = require('../../../package.json').version;
export class FirestoreClient {
private _terminated = false;
private _opts: ClientOptions;
private _providedCustomServicePath: boolean;
private _gaxModule: typeof gax | typeof gax.fallback;
private _gaxGrpc: gax.GrpcClient | gax.fallback.GrpcClient;
private _protos: {};
Expand All @@ -66,6 +67,7 @@ export class FirestoreClient {
longrunning: {},
batching: {},
};
warn: (code: string, message: string, warnType?: string) => void;
innerApiCalls: {[name: string]: Function};
firestoreStub?: Promise<{[name: string]: Function}>;

Expand Down Expand Up @@ -108,6 +110,9 @@ export class FirestoreClient {
const staticMembers = this.constructor as typeof FirestoreClient;
const servicePath =
opts?.servicePath || opts?.apiEndpoint || staticMembers.servicePath;
this._providedCustomServicePath = !!(
opts?.servicePath || opts?.apiEndpoint
);
const port = opts?.port || staticMembers.port;
const clientConfig = opts?.clientConfig ?? {};
const fallback =
Expand Down Expand Up @@ -205,6 +210,9 @@ export class FirestoreClient {
// of calling the API is handled in `google-gax`, with this code
// merely providing the destination and request information.
this.innerApiCalls = {};

// Add a warn function to the client constructor so it can be easily tested.
this.warn = gax.warn;
}

/**
Expand Down Expand Up @@ -233,7 +241,8 @@ export class FirestoreClient {
)
: // eslint-disable-next-line @typescript-eslint/no-explicit-any
(this._protos as any).google.firestore.v1.Firestore,
this._opts
this._opts,
this._providedCustomServicePath
) as Promise<{[method: string]: Function}>;

// Iterate over each of the methods that the service provides
Expand Down
11 changes: 10 additions & 1 deletion dev/src/v1beta1/firestore_client.ts
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@ const version = require('../../../package.json').version;
export class FirestoreClient {
private _terminated = false;
private _opts: ClientOptions;
private _providedCustomServicePath: boolean;
private _gaxModule: typeof gax | typeof gax.fallback;
private _gaxGrpc: gax.GrpcClient | gax.fallback.GrpcClient;
private _protos: {};
Expand All @@ -69,6 +70,7 @@ export class FirestoreClient {
longrunning: {},
batching: {},
};
warn: (code: string, message: string, warnType?: string) => void;
innerApiCalls: {[name: string]: Function};
firestoreStub?: Promise<{[name: string]: Function}>;

Expand Down Expand Up @@ -111,6 +113,9 @@ export class FirestoreClient {
const staticMembers = this.constructor as typeof FirestoreClient;
const servicePath =
opts?.servicePath || opts?.apiEndpoint || staticMembers.servicePath;
this._providedCustomServicePath = !!(
opts?.servicePath || opts?.apiEndpoint
);
const port = opts?.port || staticMembers.port;
const clientConfig = opts?.clientConfig ?? {};
const fallback =
Expand Down Expand Up @@ -208,6 +213,9 @@ export class FirestoreClient {
// of calling the API is handled in `google-gax`, with this code
// merely providing the destination and request information.
this.innerApiCalls = {};

// Add a warn function to the client constructor so it can be easily tested.
this.warn = gax.warn;
}

/**
Expand Down Expand Up @@ -236,7 +244,8 @@ export class FirestoreClient {
)
: // eslint-disable-next-line @typescript-eslint/no-explicit-any
(this._protos as any).google.firestore.v1beta1.Firestore,
this._opts
this._opts,
this._providedCustomServicePath
) as Promise<{[method: string]: Function}>;

// Iterate over each of the methods that the service provides
Expand Down
6 changes: 3 additions & 3 deletions synth.metadata
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,15 @@
"git": {
"name": ".",
"remote": "https://github.com/googleapis/nodejs-firestore.git",
"sha": "5e384157efda51dc91eeb314a7ac53466764af3d"
"sha": "8d9c50381eedf6ee8043eed681d03b44262b9820"
}
},
{
"git": {
"name": "googleapis",
"remote": "https://github.com/googleapis/googleapis.git",
"sha": "076f7e9f0b258bdb54338895d7251b202e8f0de3",
"internalRef": "380641501"
"sha": "80f404215a9346259db760d80d0671f28c433453",
"internalRef": "385101839"
}
},
{
Expand Down
2 changes: 2 additions & 0 deletions types/v1/firestore_admin_client.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,14 @@ import * as protos from '../protos/firestore_admin_v1_proto_api';
export declare class FirestoreAdminClient {
private _terminated;
private _opts;
private _providedCustomServicePath;
private _gaxModule;
private _gaxGrpc;
private _protos;
private _defaults;
auth: gax.GoogleAuth;
descriptors: Descriptors;
warn: (code: string, message: string, warnType?: string) => void;
innerApiCalls: {
[name: string]: Function;
};
Expand Down
2 changes: 2 additions & 0 deletions types/v1/firestore_client.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,14 @@ import * as protos from '../protos/firestore_v1_proto_api';
export declare class FirestoreClient {
private _terminated;
private _opts;
private _providedCustomServicePath;
private _gaxModule;
private _gaxGrpc;
private _protos;
private _defaults;
auth: gax.GoogleAuth;
descriptors: Descriptors;
warn: (code: string, message: string, warnType?: string) => void;
innerApiCalls: {
[name: string]: Function;
};
Expand Down
2 changes: 2 additions & 0 deletions types/v1beta1/firestore_client.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -41,12 +41,14 @@ import * as protos from '../protos/firestore_v1beta1_proto_api';
export declare class FirestoreClient {
private _terminated;
private _opts;
private _providedCustomServicePath;
private _gaxModule;
private _gaxGrpc;
private _protos;
private _defaults;
auth: gax.GoogleAuth;
descriptors: Descriptors;
warn: (code: string, message: string, warnType?: string) => void;
innerApiCalls: {
[name: string]: Function;
};
Expand Down

0 comments on commit 8b38f32

Please sign in to comment.