fix: type widening to match gapic, and two typing fixes

Author: feywind

src/publisher/index.ts

@@ -191,8 +191,10 @@ export class Publisher {
     // We must have at least one of:
     //   - `data` as a Buffer
     //   - `attributes` that are not empty
-    if (data && !(data instanceof Buffer)) {
-      throw new TypeError('Data must be in the form of a Buffer.');
+    if (data && !(data instanceof Uint8Array)) {
+      throw new TypeError(
+        'Data must be in the form of a Buffer or Uint8Array.'
+      );
     }
 
     const keys = Object.keys(attributes!);

test/publisher/index.ts

@@ -237,7 +237,7 @@ describe('Publisher', () => {
       const badData = {} as Buffer;
       assert.throws(
         () => publisher.publishMessage({data: badData}, spy),
-        /Data must be in the form of a Buffer\./
+        /Data must be in the form of a Buffer or Uint8Array\./
       );
     });
 

test/pubsub.ts

@@ -689,7 +689,10 @@ describe('PubSub', () => {
       const apiResponse = {
         name: 'new-topic',
       };
-      let requestStub: sinon.SinonStub<unknown[], unknown>;
+
+      // Types changed, so this is needed instead of `unknown`.
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      let requestStub: sinon.SinonStub<any[], any>;
 
       beforeEach(() => {
         requestStub = sandbox