Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

test: Move obtain test instance#1225

Merged
danieljbruce merged 10 commits intogoogleapis:mainfrom
danieljbruce:move-obtain-test-instance
Jan 30, 2023
Merged

test: Move obtain test instance#1225
danieljbruce merged 10 commits intogoogleapis:mainfrom
danieljbruce:move-obtain-test-instance

Conversation

@danieljbruce
Copy link
Copy Markdown
Contributor

This PR will reduce the number of failures due to flakey tests by fixing the flakey tests.

@danieljbruce danieljbruce requested review from a team January 30, 2023 16:55
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: bigtable Issues related to the googleapis/nodejs-bigtable API. labels Jan 30, 2023
@danieljbruce danieljbruce changed the title Move obtain test instance test: Move obtain test instance Jan 30, 2023
Comment thread samples/test/reads.js
TIMESTAMP.setUTCHours(0);

before(async () => {
const instance = await obtainTestInstance();
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change makes more sense to me. It's a good cautionary tale that we should avoid await outside the before, after, beforeEach, afterEach blocks.

@danieljbruce danieljbruce added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 30, 2023
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 30, 2023
@danieljbruce danieljbruce added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 30, 2023
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 30, 2023
@danieljbruce danieljbruce merged commit 9e1ab05 into googleapis:main Jan 30, 2023
@danieljbruce danieljbruce mentioned this pull request Feb 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigtable Issues related to the googleapis/nodejs-bigtable API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants