test: add retries to cloud run tests

Author: kweinmeister

run/helloworld/test/index.test.js

@@ -25,13 +25,13 @@ describe('Unit Tests', () => {
 
   it('Service uses the NAME override', async () => {
     process.env.NAME = 'Cloud';
-    const response = await request.get('/').expect(200);
+    const response = await request.get('/').retry(3).expect(200);
     assert.equal(response.text, 'Hello Cloud!');
   });
 
   it('Service uses the NAME default', async () => {
     process.env.NAME = '';
-    const response = await request.get('/').expect(200);
+    const response = await request.get('/').retry(3).expect(200);
     assert.equal(response.text, 'Hello World!');
   });
 });

run/idp-sql/test/app.test.js

@@ -28,13 +28,14 @@ describe('Unit Tests', () => {
   });
 
   it('should reject request without JWT token', async () => {
-    await request.post('/').expect(401);
+    await request.post('/').retry(3).expect(401);
   });
 
   it('should reject request with invalid JWT token', async () => {
     await request
       .post('/')
       .set('Authorization', 'Bearer iam-a-token')
+      .retry(3)
       .expect(403);
   });
 

run/image-processing/test/app.test.js

@@ -19,6 +19,7 @@ describe('Unit Tests', () => {
         .post('/')
         .type('json')
         .send({nomessage: 'invalid'})
+        .retry(3)
         .expect(400);
     });
 
@@ -35,6 +36,7 @@ describe('Unit Tests', () => {
             data: 'non-JSON value',
           },
         })
+        .retry(3)
         .expect(400);
     });
 
@@ -48,6 +50,7 @@ describe('Unit Tests', () => {
               data: Buffer.from('{ "json": "value" }').toString('base64'),
             },
           })
+          .retry(3)
           .expect(400);
       });
       it('missing just the "name" property', async () => {
@@ -59,6 +62,7 @@ describe('Unit Tests', () => {
               data: Buffer.from('{ "name": "value" }').toString('base64'),
             },
           })
+          .retry(3)
           .expect(400);
       });
       it('missing just the "bucket" property', async () => {
@@ -70,6 +74,7 @@ describe('Unit Tests', () => {
               data: Buffer.from('{ "bucket": "value" }').toString('base64'),
             },
           })
+          .retry(3)
           .expect(400);
       });
     });
@@ -88,6 +93,7 @@ describe('Integration Tests', () => {
           ).toString('base64'),
         },
       })
+      .retry(3)
       .expect(204);
   });
 });

run/markdown-preview/editor/test/app.test.js

@@ -23,7 +23,7 @@ describe('Editor unit tests', () => {
     it('should successfully load the index page', async () => {
       const {app} = require(path.join(__dirname, '..', 'app'));
       const request = supertest(app);
-      await request.get('/').expect(200);
+      await request.get('/').retry(3).expect(200);
     });
   });
 
@@ -53,7 +53,7 @@ describe('Integration tests', () => {
     });
 
     it('responds 404 Not Found on "GET /render"', async () => {
-      await request.get('/render').expect(404);
+      await request.get('/render').retry(3).expect(404);
     });
 
     it('responds 200 OK on "POST /render" with valid JSON', async () => {
@@ -64,6 +64,7 @@ describe('Integration tests', () => {
         .type('json')
         .set('Accept', 'text/html')
         .send({data: 'markdown'})
+        .retry(3)
         .expect(200)
         .expect('content-type', 'text/html; charset=utf-8');
     });
@@ -74,6 +75,7 @@ describe('Integration tests', () => {
         .post('/render')
         .type('json')
         .send('string: incorrect data type')
+        .retry(3)
         .expect(400);
     });
   });

run/markdown-preview/renderer/test/app.test.js

@@ -42,6 +42,7 @@ describe('Unit Tests', () => {
       .post('/')
       .type('text')
       .send(markdown)
+      .retry(3)
       .expect(200);
     const body = response.text;
     assert.equal(body, '<p><strong>markdown text</strong></p>\n');
@@ -53,6 +54,7 @@ describe('Unit Tests', () => {
       .post('/')
       .type('text')
       .send(markdown)
+      .retry(3)
       .expect(200)
       .then(res => {
         const body = res.text;
@@ -67,6 +69,7 @@ describe('Unit Tests', () => {
       .post('/')
       .type('text')
       .send(markdown)
+      .retry(3)
       .expect(200)
       .then(res => {
         const body = res.text;

run/pubsub/test/app.test.js

@@ -34,19 +34,25 @@ describe('Unit Tests', () => {
 
   describe('should fail', () => {
     it('on a Bad Request with an empty payload', async () => {
-      await request.post('/').type('json').send('').expect(400);
+      await request.post('/').type('json').send('').retry(3).expect(400);
     });
 
     it('on a Bad Request with an invalid payload', async () => {
       await request
         .post('/')
         .type('json')
         .send({nomessage: 'invalid'})
+        .retry(3)
         .expect(400);
     });
 
     it('on a Bad Request with an invalid mimetype', async () => {
-      await request.post('/').type('text').send('{message: true}').expect(400);
+      await request
+        .post('/')
+        .type('text')
+        .send('{message: true}')
+        .retry(3)
+        .expect(400);
     });
   });
 
@@ -65,6 +71,7 @@ describe('Unit Tests', () => {
         .post('/')
         .type('json')
         .send({message: true})
+        .retry(3)
         .expect(204)
         .expect(() => assert.ok(console.log.calledWith('Hello World!')));
     });
@@ -77,6 +84,7 @@ describe('Unit Tests', () => {
         .post('/')
         .type('json')
         .send({message: {data}})
+        .retry(3)
         .expect(204)
         .expect(() => assert.ok(console.log.calledWith(`Hello ${name}!`)));
     });

run/system-package/test/app.test.js

@@ -28,6 +28,7 @@ describe('Unit Tests', () => {
       await request
         .get('/diagram.png')
         .type('text')
+        .retry(3)
         .expect(400)
         .expect('Content-Type', errorContentType)
         .expect(res => {
@@ -42,6 +43,7 @@ describe('Unit Tests', () => {
         .get('/diagram.png')
         .type('text')
         .query({dot: ''})
+        .retry(3)
         .expect(400)
         .expect('Content-Type', errorContentType)
         .expect(res => {
@@ -56,6 +58,7 @@ describe('Unit Tests', () => {
         .get('/diagram.png')
         .type('text')
         .query({dot: 'digraph'})
+        .retry(3)
         .expect(400)
         .expect('Content-Type', errorContentType)
         .expect(res => {
@@ -72,6 +75,7 @@ describe('Unit Tests', () => {
         .get('/diagram.png')
         .type('text')
         .query({dot: 'digraph G { A -> {B, C, D} -> {F} }'})
+        .retry(3)
         .expect(200)
         .expect('Content-Type', 'image/png')
         .expect('Cache-Control', 'public, max-age=86400');