feat: update client libraries to support Database operations (#1676)

summer-ji-eng · web-flow · commit 533aade3272f · 2022-02-07T09:21:18.000-07:00
diff --git a/dev/protos/admin_v1.json b/dev/protos/admin_v1.json
diff --git a/dev/protos/firestore_admin_v1_proto_api.d.ts b/dev/protos/firestore_admin_v1_proto_api.d.ts
@@ -1,5 +1,5 @@
 /*!
- * Copyright 2021 Google LLC
+ * Copyright 2022 Google LLC
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/dev/protos/firestore_admin_v1_proto_api.js b/dev/protos/firestore_admin_v1_proto_api.js
@@ -1,5 +1,5 @@
 /*!
- * Copyright 2021 Google LLC
+ * Copyright 2022 Google LLC
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/dev/protos/firestore_v1_proto_api.d.ts b/dev/protos/firestore_v1_proto_api.d.ts
@@ -1,5 +1,5 @@
 /*!
- * Copyright 2021 Google LLC
+ * Copyright 2022 Google LLC
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/dev/protos/firestore_v1_proto_api.js b/dev/protos/firestore_v1_proto_api.js
@@ -1,5 +1,5 @@
 /*!
- * Copyright 2021 Google LLC
+ * Copyright 2022 Google LLC
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/dev/protos/firestore_v1beta1_proto_api.d.ts b/dev/protos/firestore_v1beta1_proto_api.d.ts
@@ -1,5 +1,5 @@
 /*!
- * Copyright 2021 Google LLC
+ * Copyright 2022 Google LLC
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/dev/protos/firestore_v1beta1_proto_api.js b/dev/protos/firestore_v1beta1_proto_api.js
@@ -1,5 +1,5 @@
 /*!
- * Copyright 2021 Google LLC
+ * Copyright 2022 Google LLC
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/dev/src/v1/firestore_admin_client.ts b/dev/src/v1/firestore_admin_client.ts
@@ -31,7 +31,7 @@ import {
 import {Transform} from 'stream';
 import {RequestType} from 'google-gax/build/src/apitypes';
 import * as protos from '../../protos/firestore_admin_v1_proto_api';
-import jsonProtos = require('../../protos/v1_admin.json');
+import jsonProtos = require('../../protos/admin_v1.json');
 /**
  * Client JSON configuration object, loaded from
  * `src/v1/firestore_admin_client_config.json`.
diff --git a/dev/src/v1/firestore_client.ts b/dev/src/v1/firestore_client.ts
@@ -1226,7 +1226,7 @@ export class FirestoreClient {
    */
   write(options?: CallOptions): gax.CancellableStream {
     this.initialize();
-    return this.innerApiCalls.write({}, options);
+    return this.innerApiCalls.write(null, options);
   }
 
   /**
@@ -1246,7 +1246,7 @@ export class FirestoreClient {
    */
   listen(options?: CallOptions): gax.CancellableStream {
     this.initialize();
-    return this.innerApiCalls.listen({}, options);
+    return this.innerApiCalls.listen(null, options);
   }
 
   /**
diff --git a/dev/src/v1beta1/firestore_client.ts b/dev/src/v1beta1/firestore_client.ts
@@ -1243,7 +1243,7 @@ export class FirestoreClient {
    */
   write(options?: CallOptions): gax.CancellableStream {
     this.initialize();
-    return this.innerApiCalls.write({}, options);
+    return this.innerApiCalls.write(null, options);
   }
 
   /**
@@ -1263,7 +1263,7 @@ export class FirestoreClient {
    */
   listen(options?: CallOptions): gax.CancellableStream {
     this.initialize();
-    return this.innerApiCalls.listen({}, options);
+    return this.innerApiCalls.listen(null, options);
   }
 
   /**
diff --git a/dev/test/gapic_firestore_v1.ts b/dev/test/gapic_firestore_v1.ts
@@ -1321,9 +1321,7 @@ describe('v1.FirestoreClient', () => {
       const response = await promise;
       assert.deepStrictEqual(response, expectedResponse);
       assert(
-        (client.innerApiCalls.write as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.write as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
@@ -1364,9 +1362,7 @@ describe('v1.FirestoreClient', () => {
       });
       await assert.rejects(promise, expectedError);
       assert(
-        (client.innerApiCalls.write as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.write as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
@@ -1407,9 +1403,7 @@ describe('v1.FirestoreClient', () => {
       const response = await promise;
       assert.deepStrictEqual(response, expectedResponse);
       assert(
-        (client.innerApiCalls.listen as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.listen as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
@@ -1450,9 +1444,7 @@ describe('v1.FirestoreClient', () => {
       });
       await assert.rejects(promise, expectedError);
       assert(
-        (client.innerApiCalls.listen as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.listen as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
diff --git a/dev/test/gapic_firestore_v1beta1.ts b/dev/test/gapic_firestore_v1beta1.ts
@@ -1325,9 +1325,7 @@ describe('v1beta1.FirestoreClient', () => {
       const response = await promise;
       assert.deepStrictEqual(response, expectedResponse);
       assert(
-        (client.innerApiCalls.write as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.write as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
@@ -1368,9 +1366,7 @@ describe('v1beta1.FirestoreClient', () => {
       });
       await assert.rejects(promise, expectedError);
       assert(
-        (client.innerApiCalls.write as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.write as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
@@ -1411,9 +1407,7 @@ describe('v1beta1.FirestoreClient', () => {
       const response = await promise;
       assert.deepStrictEqual(response, expectedResponse);
       assert(
-        (client.innerApiCalls.listen as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.listen as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
@@ -1454,9 +1448,7 @@ describe('v1beta1.FirestoreClient', () => {
       });
       await assert.rejects(promise, expectedError);
       assert(
-        (client.innerApiCalls.listen as SinonStub)
-          .getCall(0)
-          .calledWithExactly({}, undefined)
+        (client.innerApiCalls.listen as SinonStub).getCall(0).calledWith(null)
       );
       assert.deepStrictEqual(
         ((stream as unknown as PassThrough)._transform as SinonStub).getCall(0)
diff --git a/owlbot.py b/owlbot.py
@@ -170,6 +170,7 @@
     os.chdir("protos")
     os.unlink('protos.js')
     os.unlink('protos.d.ts')
+    os.unlink('protos.json')
     subprocess.run('./update.sh', shell=True)
     os.chdir("../../")
 
diff --git a/types/protos/firestore_admin_v1_proto_api.d.ts b/types/protos/firestore_admin_v1_proto_api.d.ts
diff --git a/types/protos/firestore_v1_proto_api.d.ts b/types/protos/firestore_v1_proto_api.d.ts
diff --git a/types/protos/firestore_v1beta1_proto_api.d.ts b/types/protos/firestore_v1beta1_proto_api.d.ts
diff --git a/types/v1/firestore_admin_client.d.ts b/types/v1/firestore_admin_client.d.ts

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*!`
`2`		`- * Copyright 2021 Google LLC`
	`2`	`+ * Copyright 2022 Google LLC`
`3`	`3`	`*`
`4`	`4`	`* Licensed under the Apache License, Version 2.0 (the "License");`
`5`	`5`	`* you may not use this file except in compliance with the License.`