-
Notifications
You must be signed in to change notification settings - Fork 97
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
tests: avoid require in non-initial go routines
docs: > FailNow must be called from the goroutine running the test or benchmark function, > not from other goroutines created during the test This would cause workflow to hang if it was waiting on goroutine, so introduce helper functions which: 1. Check assertion 2. Log failure & mark test failed 3. Cancel workflow 4. Abort goroutine A followup PR will fix calls to compareTableContents after #967 merges too
- Loading branch information
Showing
7 changed files
with
201 additions
and
186 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.