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

test: label v3 batch_translate test as flaky #363

Merged
merged 4 commits into from
Oct 29, 2019
Merged

test: label v3 batch_translate test as flaky #363

merged 4 commits into from
Oct 29, 2019

Conversation

bcoe
Copy link
Contributor

@bcoe bcoe commented Oct 28, 2019

This test has been flapping since added, labeling as flaky.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 28, 2019
@@ -71,7 +71,7 @@ describe(REGION_TAG, () => {
await operation.promise();
});

it('should batch translate the input text with a glossary', async () => {
it.skip('should batch translate the input text with a glossary [FLAKY]', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Just an idea. It could turn out useful to leave a code comment that links to the issue #364. That way, if the future fixer comes straight from the code, as opposed to finding the issue first, we won’t have a stale issue hanging around.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 good idea.

@codecov
Copy link

codecov bot commented Oct 28, 2019

Codecov Report

Merging #363 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #363   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          85     85           
=====================================
  Hits           85     85

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8ea6de...addf176. Read the comment docs.

@bcoe bcoe merged commit 1238a02 into master Oct 29, 2019
@bcoe bcoe deleted the label-flaky-test branch October 29, 2019 00:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants