migrate assert.strict => .strictEqual

koajs · Aug 18, 2021 · 596cfd5 · 596cfd5
1 parent be3e8b6
commit 596cfd5
Show file tree

Hide file tree

Showing 50 changed files with 403 additions and 403 deletions.
diff --git a/test/application/context.js b/test/application/context.js
@@ -12,7 +12,7 @@ describe('app.context', () => {
 
   it('should merge properties', () => {
     app1.use((ctx, next) => {
-      assert.equal(ctx.msg, 'hello');
+      assert.strictEqual(ctx.msg, 'hello');
       ctx.status = 204;
     });
 
@@ -23,7 +23,7 @@ describe('app.context', () => {
 
   it('should not affect the original prototype', () => {
     app2.use((ctx, next) => {
-      assert.equal(ctx.msg, undefined);
+      assert.strictEqual(ctx.msg, undefined);
       ctx.status = 204;
     });
 

diff --git a/test/application/index.js b/test/application/index.js
@@ -15,7 +15,7 @@ describe('app', () => {
     });
 
     app.on('error', err => {
-      assert.equal(err.message, 'boom');
+      assert.strictEqual(err.message, 'boom');
       done();
     });
 
@@ -51,7 +51,7 @@ describe('app', () => {
     process.env.NODE_ENV = '';
     const app = new Koa();
     process.env.NODE_ENV = NODE_ENV;
-    assert.equal(app.env, 'development');
+    assert.strictEqual(app.env, 'development');
   });
 
   it('should set env from the constructor', () => {

diff --git a/test/application/inspect.js b/test/application/inspect.js
@@ -9,7 +9,7 @@ const app = new Koa();
 describe('app.inspect()', () => {
   it('should work', () => {
     const str = util.inspect(app);
-    assert.equal("{ subdomainOffset: 2, proxy: false, env: 'test' }", str);
+    assert.strictEqual("{ subdomainOffset: 2, proxy: false, env: 'test' }", str);
   });
 
   it('should return a json representation', () => {

diff --git a/test/application/request.js b/test/application/request.js
@@ -12,7 +12,7 @@ describe('app.request', () => {
 
   it('should merge properties', () => {
     app1.use((ctx, next) => {
-      assert.equal(ctx.request.message, 'hello');
+      assert.strictEqual(ctx.request.message, 'hello');
       ctx.status = 204;
     });
 
@@ -23,7 +23,7 @@ describe('app.request', () => {
 
   it('should not affect the original prototype', () => {
     app2.use((ctx, next) => {
-      assert.equal(ctx.request.message, undefined);
+      assert.strictEqual(ctx.request.message, undefined);
       ctx.status = 204;
     });
 

diff --git a/test/application/respond.js b/test/application/respond.js
@@ -96,7 +96,7 @@ describe('app.respond', () => {
         .get('/')
         .expect(200);
 
-      assert.equal(res.headers.hasOwnProperty('Content-Type'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('Content-Type'), false);
     });
   });
 
@@ -114,8 +114,8 @@ describe('app.respond', () => {
         .head('/')
         .expect(200);
 
-      assert.equal(res.headers['content-type'], 'text/plain; charset=utf-8');
-      assert.equal(res.headers['content-length'], '5');
+      assert.strictEqual(res.headers['content-type'], 'text/plain; charset=utf-8');
+      assert.strictEqual(res.headers['content-length'], '5');
       assert(!res.text);
     });
 
@@ -132,8 +132,8 @@ describe('app.respond', () => {
         .head('/')
         .expect(200);
 
-      assert.equal(res.headers['content-type'], 'application/json; charset=utf-8');
-      assert.equal(res.headers['content-length'], '17');
+      assert.strictEqual(res.headers['content-type'], 'application/json; charset=utf-8');
+      assert.strictEqual(res.headers['content-length'], '17');
       assert(!res.text);
     });
 
@@ -150,8 +150,8 @@ describe('app.respond', () => {
         .head('/')
         .expect(200);
 
-      assert.equal(res.headers['content-type'], 'text/plain; charset=utf-8');
-      assert.equal(res.headers['content-length'], '11');
+      assert.strictEqual(res.headers['content-type'], 'text/plain; charset=utf-8');
+      assert.strictEqual(res.headers['content-length'], '11');
       assert(!res.text);
     });
 
@@ -168,8 +168,8 @@ describe('app.respond', () => {
         .head('/')
         .expect(200);
 
-      assert.equal(res.headers['content-type'], 'application/octet-stream');
-      assert.equal(res.headers['content-length'], '11');
+      assert.strictEqual(res.headers['content-type'], 'application/octet-stream');
+      assert.strictEqual(res.headers['content-length'], '11');
       assert(!res.text);
     });
 
@@ -189,7 +189,7 @@ describe('app.respond', () => {
         .head('/')
         .expect(200);
 
-      assert.equal(res.header['content-length'], length);
+      assert.strictEqual(res.header['content-length'], length);
       assert(!res.text);
     });
 
@@ -329,7 +329,7 @@ describe('app.respond', () => {
           .expect(204)
           .expect('');
 
-        assert.equal(res.headers.hasOwnProperty('content-type'), false);
+        assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
       });
     });
 
@@ -348,7 +348,7 @@ describe('app.respond', () => {
           .expect(205)
           .expect('');
 
-        assert.equal(res.headers.hasOwnProperty('content-type'), false);
+        assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
       });
     });
 
@@ -367,7 +367,7 @@ describe('app.respond', () => {
           .expect(304)
           .expect('');
 
-        assert.equal(res.headers.hasOwnProperty('content-type'), false);
+        assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
       });
     });
 
@@ -387,7 +387,7 @@ describe('app.respond', () => {
           .expect(700)
           .expect('custom status');
 
-        assert.equal(res.res.statusMessage, 'custom status');
+        assert.strictEqual(res.res.statusMessage, 'custom status');
       });
     });
 
@@ -407,7 +407,7 @@ describe('app.respond', () => {
           .expect(200)
           .expect('ok');
 
-        assert.equal(res.res.statusMessage, 'ok');
+        assert.strictEqual(res.res.statusMessage, 'ok');
       });
     });
 
@@ -444,7 +444,7 @@ describe('app.respond', () => {
         .expect(204)
         .expect('');
 
-      assert.equal(res.headers.hasOwnProperty('content-type'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
     });
 
     it('should respond 204 with status=200', async() => {
@@ -462,7 +462,7 @@ describe('app.respond', () => {
         .expect(204)
         .expect('');
 
-      assert.equal(res.headers.hasOwnProperty('content-type'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
     });
 
     it('should respond 205 with status=205', async() => {
@@ -480,7 +480,7 @@ describe('app.respond', () => {
         .expect(205)
         .expect('');
 
-      assert.equal(res.headers.hasOwnProperty('content-type'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
     });
 
     it('should respond 304 with status=304', async() => {
@@ -498,7 +498,7 @@ describe('app.respond', () => {
         .expect(304)
         .expect('');
 
-      assert.equal(res.headers.hasOwnProperty('content-type'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
     });
   });
 
@@ -551,7 +551,7 @@ describe('app.respond', () => {
         .expect('Content-Type', 'application/json; charset=utf-8');
 
       const pkg = require('../../package');
-      assert.equal(res.headers.hasOwnProperty('content-length'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-length'), false);
       assert.deepEqual(res.body, pkg);
     });
 
@@ -571,7 +571,7 @@ describe('app.respond', () => {
         .expect('Content-Type', 'application/json; charset=utf-8');
 
       const pkg = require('../../package');
-      assert.equal(res.headers.hasOwnProperty('content-length'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-length'), false);
       assert.deepEqual(res.body, pkg);
     });
 
@@ -591,7 +591,7 @@ describe('app.respond', () => {
         .expect('Content-Type', 'application/json; charset=utf-8');
 
       const pkg = require('../../package');
-      assert.equal(res.headers.hasOwnProperty('content-length'), true);
+      assert.strictEqual(res.headers.hasOwnProperty('content-length'), true);
       assert.deepEqual(res.body, pkg);
     });
 
@@ -614,7 +614,7 @@ describe('app.respond', () => {
           .expect('Content-Type', 'application/json; charset=utf-8');
 
         const pkg = require('../../package');
-        assert.equal(res.headers.hasOwnProperty('content-length'), true);
+        assert.strictEqual(res.headers.hasOwnProperty('content-length'), true);
         assert.deepEqual(res.body, pkg);
       });
 
@@ -716,7 +716,7 @@ describe('app.respond', () => {
       });
 
       app.on('error', err => {
-        assert.equal(err.message, 'boom');
+        assert.strictEqual(err.message, 'boom');
         done();
       });
 
@@ -829,7 +829,7 @@ describe('app.respond', () => {
         .get('/')
         .expect(204);
 
-      assert.equal(res.headers.hasOwnProperty('content-type'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
     });
   });
 
@@ -866,7 +866,7 @@ describe('app.respond', () => {
         .expect('')
         .expect({});
 
-      assert.equal(res.headers.hasOwnProperty('content-type'), false);
+      assert.strictEqual(res.headers.hasOwnProperty('content-type'), false);
     });
   });
 });
diff --git a/test/application/response.js b/test/application/response.js
@@ -15,7 +15,7 @@ describe('app.response', () => {
 
   it('should merge properties', () => {
     app1.use((ctx, next) => {
-      assert.equal(ctx.response.msg, 'hello');
+      assert.strictEqual(ctx.response.msg, 'hello');
       ctx.status = 204;
     });
 
@@ -26,7 +26,7 @@ describe('app.response', () => {
 
   it('should not affect the original prototype', () => {
     app2.use((ctx, next) => {
-      assert.equal(ctx.response.msg, undefined);
+      assert.strictEqual(ctx.response.msg, undefined);
       ctx.status = 204;
     });
 
@@ -43,7 +43,7 @@ describe('app.response', () => {
     const response = await request(app3.listen())
       .get('/')
       .expect(404);
-    assert.equal(response.text, '404');
+    assert.strictEqual(response.text, '404');
   });
 
   it('should set ._explicitNullBody correctly', async() => {

diff --git a/test/context/assert.js b/test/context/assert.js
@@ -12,7 +12,7 @@ describe('ctx.assert(value, status)', () => {
       ctx.assert(false, 404);
       throw new Error('asdf');
     } catch (err) {
-      assert.equal(err.status, 404);
+      assert.strictEqual(err.status, 404);
       assert.strictEqual(err.expose, true);
     }
   });

diff --git a/test/context/cookies.js b/test/context/cookies.js
@@ -22,7 +22,7 @@ describe('ctx.cookies', () => {
         .expect(204);
 
       const cookie = res.headers['set-cookie'].some(cookie => /^name=/.test(cookie));
-      assert.equal(cookie, true);
+      assert.strictEqual(cookie, true);
     });
 
     describe('with .signed', () => {
@@ -62,8 +62,8 @@ describe('ctx.cookies', () => {
 
         const cookies = res.headers['set-cookie'];
 
-        assert.equal(cookies.some(cookie => /^name=/.test(cookie)), true);
-        assert.equal(cookies.some(cookie => /(,|^)name\.sig=/.test(cookie)), true);
+        assert.strictEqual(cookies.some(cookie => /^name=/.test(cookie)), true);
+        assert.strictEqual(cookies.some(cookie => /(,|^)name\.sig=/.test(cookie)), true);
       });
     });
 
@@ -87,9 +87,9 @@ describe('ctx.cookies', () => {
           .expect(204);
 
         const cookies = res.headers['set-cookie'];
-        assert.equal(cookies.some(cookie => /^name=/.test(cookie)), true);
-        assert.equal(cookies.some(cookie => /(,|^)name\.sig=/.test(cookie)), true);
-        assert.equal(cookies.every(cookie => /secure/.test(cookie)), true);
+        assert.strictEqual(cookies.some(cookie => /^name=/.test(cookie)), true);
+        assert.strictEqual(cookies.some(cookie => /(,|^)name\.sig=/.test(cookie)), true);
+        assert.strictEqual(cookies.every(cookie => /secure/.test(cookie)), true);
       });
     });
   });

diff --git a/test/context/onerror.js b/test/context/onerror.js
@@ -43,8 +43,8 @@ describe('ctx.onerror(err)', () => {
       .expect('Content-Type', 'text/plain; charset=utf-8')
       .expect('Content-Length', '4');
 
-    assert.equal(res.headers.hasOwnProperty('vary'), false);
-    assert.equal(res.headers.hasOwnProperty('x-csrf-token'), false);
+    assert.strictEqual(res.headers.hasOwnProperty('vary'), false);
+    assert.strictEqual(res.headers.hasOwnProperty('x-csrf-token'), false);
   });
 
   it('should set headers specified in the error', async() => {
@@ -72,16 +72,16 @@ describe('ctx.onerror(err)', () => {
       .expect('Content-Type', 'text/plain; charset=utf-8')
       .expect('X-New-Header', 'Value');
 
-    assert.equal(res.headers.hasOwnProperty('vary'), false);
-    assert.equal(res.headers.hasOwnProperty('x-csrf-token'), false);
+    assert.strictEqual(res.headers.hasOwnProperty('vary'), false);
+    assert.strictEqual(res.headers.hasOwnProperty('x-csrf-token'), false);
   });
 
   it('should ignore error after headerSent', done => {
     const app = new Koa();
 
     app.on('error', err => {
-      assert.equal(err.message, 'mock error');
-      assert.equal(err.headerSent, true);
+      assert.strictEqual(err.message, 'mock error');
+      assert.strictEqual(err.headerSent, true);
       done();
     });
 
@@ -269,14 +269,14 @@ describe('ctx.onerror(err)', () => {
 
       ctx.onerror(new Error('error'));
 
-      assert.equal(removed, 2);
+      assert.strictEqual(removed, 2);
     });
 
     it('should stringify error if it is an object', done => {
       const app = new Koa();
 
       app.on('error', err => {
-        assert.equal(err, 'Error: non-error thrown: {"key":"value"}');
+        assert.strictEqual(err, 'Error: non-error thrown: {"key":"value"}');
         done();
       });