From 9fea062d33b82b83db6dd7d9118328a285f9e5b5 Mon Sep 17 00:00:00 2001
From: "Benjamin E. Coe" <bencoe@google.com>
Date: Fri, 3 Apr 2020 12:48:37 -0700
Subject: [PATCH] test: configure more tests with retries (#407)

---
 monitoring/snippets/test/metrics.test.js | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/monitoring/snippets/test/metrics.test.js b/monitoring/snippets/test/metrics.test.js
index fb30ab5a56..8f36c14b92 100644
--- a/monitoring/snippets/test/metrics.test.js
+++ b/monitoring/snippets/test/metrics.test.js
@@ -36,14 +36,16 @@ const delay = async test => {
   const retries = test.currentRetry();
   if (retries === 0) return; // no retry on the first failure.
   // see: https://cloud.google.com/storage/docs/exponential-backoff:
-  const ms = Math.pow(2, retries) * 250 + Math.random() * 1000;
+  const ms = Math.pow(2, retries) * 500 + Math.random() * 1000;
   return new Promise(done => {
     console.info(`retrying "${test.title}" in ${ms}ms`);
     setTimeout(done, ms);
   });
 };
 describe('metrics', async () => {
-  it('should create a metric descriptors', () => {
+  it('should create a metric descriptors', async function() {
+    this.retries(8);
+    await delay(this.test);
     const output = execSync(`${cmd} create`);
     assert.include(output, 'Created custom Metric');
     assert.include(output, `Type: ${customMetricId}`);
@@ -51,7 +53,7 @@ describe('metrics', async () => {
 
   it('should list metric descriptors, including the new custom one', async function() {
     this.retries(8);
-    await delay(this.test); // delay the start of the test, if this is a retry.
+    await delay(this.test);
     const output = execSync(`${cmd} list`);
     assert.include(output, customMetricId);
     assert.include(output, computeMetricId);
@@ -62,12 +64,16 @@ describe('metrics', async () => {
     assert.include(output, `Type: ${customMetricId}`);
   });
 
-  it('should write time series data', () => {
+  it('should write time series data', async function() {
+    this.retries(5);
+    await delay(this.test);
     const output = execSync(`${cmd} write`);
     assert.include(output, 'Done writing time series data.');
   });
 
-  it('should delete a metric descriptor', () => {
+  it('should delete a metric descriptor', async function() {
+    this.retries(5);
+    await delay(this.test);
     const output = execSync(`${cmd} delete ${customMetricId}`);
     assert.include(output, `Deleted ${customMetricId}`);
   });
@@ -135,7 +141,7 @@ describe('metrics', async () => {
 
   it('should read time series data aggregated', async function() {
     this.retries(5);
-    await delay(this.test); // delay the start of the test, if this is a retry.
+    await delay(this.test);
     const [timeSeries] = await client.listTimeSeries({
       name: client.projectPath(projectId),
       filter: filter,