fix: Fix flakey test by extending timeout (#1350)

* fix: Fix flakey test by extending timeout Add timeout to the read syncronously test as it is blocking a few tests from being merged. * Increase the timeout to unblock the CI pipeline Tests are failing for non-windows cases due to timeouts. Add this line of code for timeouts. * run linter

Author: danieljbruce

system-test/bigtable.ts

@@ -155,9 +155,8 @@ describe('Bigtable', () => {
 
  it('should test Iam permissions for the instance', async () => {
  const permissions = ['bigtable.tables.get', 'bigtable.tables.readRows'];
- const [grantedPermissions] = await INSTANCE.testIamPermissions(
- permissions
- );
+ const [grantedPermissions] =
+ await INSTANCE.testIamPermissions(permissions);
  assert.strictEqual(grantedPermissions.length, permissions.length);
  permissions.forEach(permission => {
  assert.strictEqual(grantedPermissions.includes(permission), true);

test/readrows.ts

@@ -42,7 +42,9 @@ describe('Bigtable/ReadRows', () => {
  service = new BigtableClientMockService(server);
  });
 
- it('should create read stream and read synchronously', done => {
+ it('should create read stream and read synchronously', function (done) {
+ this.timeout(60000);
+
  // 1000 rows must be enough to reproduce issues with losing the data and to create backpressure
  const keyFrom = 0;
  const keyTo = 1000;