Skip to content
This repository has been archived by the owner on Jul 20, 2023. It is now read-only.

Commit

Permalink
fix(issue-417): adding additional asserts to track flaky error (#421)
Browse files Browse the repository at this point in the history
* refactor: adding additional asserts to track flaky error

* refactor: remove .only from test

* refactor: fix lint

* refactor: update assert.fail msg

Co-authored-by: Averi Kitsch <akitsch@google.com>
  • Loading branch information
pattishin and averikitsch committed Jun 23, 2022
1 parent 6e40ebe commit 23cd446
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions samples/test/containerAnalysis.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -405,10 +405,19 @@ describe('pubsub', () => {
this.retries(3);
await delay(this.test);

await client
.getGrafeasClient()
.deleteNote({name: `${formattedNoteName}-pubsub`});
await pubsub.subscription(subscriptionId).delete();
try {
await client
.getGrafeasClient()
.deleteNote({name: `${formattedNoteName}-pubsub`});
} catch (err) {
assert.fail(err);
}

try {
await pubsub.subscription(subscriptionId).delete();
} catch (err) {
assert.fail(err);
}
});
});
});

0 comments on commit 23cd446

Please sign in to comment.