Merge pull request #119 from authzed/re-enable-tests

re-enable tests

Author: vroldanbet

.github/workflows/authzed-node.yaml

@@ -28,7 +28,7 @@ jobs:
               - package.json
               - js-dist/package.json
   test:
-    name: Lint
+    name: Lint and Test
     runs-on: "buildjet-2vcpu-ubuntu-2204"
     strategy:
       matrix:
@@ -51,6 +51,9 @@ jobs:
       - name: Run lint
         run: CI=true yarn lint
         working-directory: ./
+      - name: Run Yarn tests
+        run: CI=true yarn only-run-tests
+        working-directory: ./
   publish-npm:
     name: Publish to NPM
     needs:

src/v1-promise.test.ts

@@ -476,11 +476,10 @@ describe('Experimental Service', () => {
     });
 
     await client.writeSchema(request);
-
     client.close();
   });
 
-  it('can bulk import relationships', async (done) => {
+  it('can bulk import relationships', () => {
     const { promises: client } = NewClient(
       token,
       'localhost:50051',
@@ -489,11 +488,14 @@ describe('Experimental Service', () => {
 
     const writeStream = client.bulkImportRelationships((err, value) => {
       if (err) {
-        done.fail(err);
+        fail(err);
       }
 
       expect(value?.numLoaded).toEqual('2');
-      done();
+    });
+
+    writeStream.on('error', (e) => {
+      fail(e);
     });
 
     const resource = ObjectReference.create({