Update tests

test/commander.test.js

@@ -1,8 +1,8 @@
 /* api */
+import { strict as assert } from 'node:assert';
 import fs from 'node:fs';
 import path from 'node:path';
 import sinon from 'sinon';
-import { assert } from 'chai';
 import { describe, it } from 'mocha';
 
 /* test */
@@ -17,7 +17,7 @@ describe('clean directory', () => {
     cleanDirectory({ dir });
     const { called: rmCalled } = stubRm;
     stubRm.restore();
-    assert.isFalse(rmCalled, 'not called');
+    assert.strictEqual(rmCalled, false, 'not called');
   });
 
   it('should call funtion', () => {
@@ -29,8 +29,8 @@ describe('clean directory', () => {
     const { called: infoCalled } = stubInfo;
     stubRm.restore();
     stubInfo.restore();
-    assert.isTrue(rmCalled, 'called');
-    assert.isFalse(infoCalled, 'not called');
+    assert.strictEqual(rmCalled, true, 'called');
+    assert.strictEqual(infoCalled, false, 'not called');
   });
 
   it('should call funtion', () => {
@@ -42,8 +42,8 @@ describe('clean directory', () => {
     const { calledOnce: infoCalled } = stubInfo;
     stubRm.restore();
     stubInfo.restore();
-    assert.isTrue(rmCalled, 'called');
-    assert.isTrue(infoCalled, 'not called');
+    assert.strictEqual(rmCalled, true, 'called');
+    assert.strictEqual(infoCalled, true, 'called');
   });
 });
 

test/common.test.js

@@ -1,6 +1,6 @@
 /* api */
+import { strict as assert } from 'node:assert';
 import sinon from 'sinon';
-import { assert } from 'chai';
 import { describe, it } from 'mocha';
 
 /* test */
@@ -42,15 +42,15 @@ describe('isObjectNotEmpty', () => {
   it('should get false', () => {
     const items = [{}, [], ['foo'], '', 'foo', undefined, null, 1, true];
     for (const item of items) {
-      assert.isFalse(isObjectNotEmpty(item));
+      assert.strictEqual(isObjectNotEmpty(item), false);
     }
   });
 
   it('should get true', () => {
     const item = {
       foo: 'bar'
     };
-    assert.isTrue(isObjectNotEmpty(item));
+    assert.strictEqual(isObjectNotEmpty(item), true);
   });
 });
 
@@ -74,9 +74,9 @@ describe('logErr', () => {
     const res = logErr(new Error(msg));
     const { calledOnce } = consoleError;
     consoleError.restore();
-    assert.isTrue(calledOnce);
+    assert.strictEqual(calledOnce, true);
     assert.strictEqual(errMsg, msg);
-    assert.isFalse(res);
+    assert.strictEqual(res, false);
   });
 });
 
@@ -90,7 +90,7 @@ describe('logMsg', () => {
     const res = logMsg(msg);
     const { calledOnce } = consoleLog;
     consoleLog.restore();
-    assert.isTrue(calledOnce);
+    assert.strictEqual(calledOnce, true);
     assert.strictEqual(logMessage, msg);
     assert.strictEqual(res, msg);
   });
@@ -106,9 +106,9 @@ describe('logWarn', () => {
     const res = logWarn(msg);
     const { calledOnce } = consoleWarn;
     consoleWarn.restore();
-    assert.isTrue(calledOnce);
+    assert.strictEqual(calledOnce, true);
     assert.strictEqual(warnMsg, msg);
-    assert.isFalse(res);
+    assert.strictEqual(res, false);
   });
 });
 

test/constant.test.js

@@ -1,5 +1,5 @@
 /* api */
-import { assert } from 'chai';
+import { strict as assert } from 'node:assert';
 import { describe, it } from 'mocha';
 import { isString } from '../modules/common.js';
 
@@ -10,7 +10,7 @@ describe('string constants', () => {
   it('should get string', () => {
     const arr = [CHAR];
     arr.forEach(i => {
-      assert.isTrue(isString(i));
+      assert.strictEqual(isString(i), true);
     });
   });
 });
@@ -19,7 +19,7 @@ describe('number constants', () => {
   it('should get number', () => {
     const arr = [INDENT];
     arr.forEach(i => {
-      assert.isTrue(typeof i === 'number');
+      assert.strictEqual(typeof i, 'number');
     });
   });
 });
@@ -28,7 +28,7 @@ describe('boolean constants', () => {
   it('should get boolean', () => {
     const arr = [IS_WIN];
     arr.forEach(i => {
-      assert.isTrue(typeof i === 'boolean');
+      assert.strictEqual(typeof i, 'boolean');
     });
   });
 });

test/file-util.test.js

@@ -1,9 +1,9 @@
 /* api */
+import { strict as assert } from 'node:assert';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 import process from 'node:process';
-import { assert } from 'chai';
 import { describe, it } from 'mocha';
 import { IS_WIN } from '../modules/constant.js';
 
@@ -71,13 +71,13 @@ describe('convertUriToFilePath', () => {
   });
 
   it('should throw if string is not given', () => {
-    assert.throws(() => convertUriToFilePath(),
+    assert.throws(() => convertUriToFilePath(), TypeError,
       'Expected String but got Undefined');
   });
 
   it('should get null if protocol does not match', () => {
     const uri = 'http://example.com';
-    assert.isNull(convertUriToFilePath(uri));
+    assert.deepEqual(convertUriToFilePath(uri), null);
   });
 });
 
@@ -92,17 +92,15 @@ describe('createDirectory', () => {
 
   it('should throw if given argument is not a string', async () => {
     await createDirectory().catch(e => {
-      assert.instanceOf(e, TypeError, 'error');
-      assert.strictEqual(e.message, 'Expected String but got Undefined.',
-        'message');
+      assert.deepStrictEqual(e,
+        new TypeError('Expected String but got Undefined.'));
     });
   });
 
   it('should throw if given second argument is not a number', async () => {
     await createDirectory('/foo/bar', 'baz').catch(e => {
-      assert.instanceOf(e, TypeError, 'error');
-      assert.strictEqual(e.message, 'Expected Number but got String.',
-        'message');
+      assert.deepStrictEqual(e,
+        new TypeError('Expected Number but got String.'));
     });
   });
 });
@@ -123,7 +121,8 @@ describe('createFile', () => {
 
   it('should throw if first argument is not a string', () => {
     createFile().catch(e => {
-      assert.strictEqual(e.message, 'Expected String but got Undefined.');
+      assert.deepStrictEqual(e,
+        new TypeError('Expected String but got Undefined.'));
     });
   });
 
@@ -132,10 +131,8 @@ describe('createFile', () => {
     () => {
       const file = path.join(TMPDIR, 'webext-schema', 'test.txt');
       createFile(file).catch(e => {
-        assert.strictEqual(
-          e.message,
-          'Expected String, Buffer, Uint8Array but got Undefined.'
-        );
+        assert.deepStrictEqual(e,
+          new TypeError('Expected String, Buffer, Uint8Array but got Undefined.'));
       });
     }
   );
@@ -169,7 +166,7 @@ describe('removeDir', () => {
 
   it('should ignore if dir is not a directory', () => {
     const foo = path.resolve('foo');
-    assert.isFalse(isDir(foo));
+    assert.strictEqual(isDir(foo), false);
     assert.doesNotThrow(() => removeDir(foo, TMPDIR));
   });
 
@@ -178,7 +175,7 @@ describe('removeDir', () => {
     const foo = path.join(TMPDIR, 'foo');
     await fs.mkdirSync(dirPath);
     await fs.mkdirSync(foo);
-    assert.throws(() => removeDir(foo, dirPath),
+    assert.throws(() => removeDir(foo, dirPath), Error,
                   `${foo} is not a subdirectory of ${dirPath}.`);
     await fs.rmdirSync(dirPath);
     await fs.rmdirSync(foo);
@@ -213,7 +210,7 @@ describe('removeDirectory', () => {
 
   it('should ignore if dir is not a directory', async () => {
     const foo = path.resolve('foo');
-    assert.isFalse(isDir(foo));
+    assert.strictEqual(isDir(foo), false);
     await removeDirectory(foo, TMPDIR).catch(e => {
       assert.isUndefined(e);
     });
@@ -225,9 +222,8 @@ describe('removeDirectory', () => {
     await fs.mkdirSync(dirPath);
     await fs.mkdirSync(foo);
     await removeDirectory(foo, dirPath).catch(e => {
-      assert.instanceOf(e, Error);
-      assert.strictEqual(e.message,
-                         `${foo} is not a subdirectory of ${dirPath}.`);
+      assert.deepStrictEqual(e,
+        new Error(`${foo} is not a subdirectory of ${dirPath}.`));
     });
     await fs.rmdirSync(dirPath);
     await fs.rmdirSync(foo);
@@ -274,7 +270,7 @@ describe('getFileNameFromFilePath', () => {
 describe('getFileTimestamp', () => {
   it('should get positive integer', () => {
     const p = path.resolve(path.join('test', 'file', 'test.sh'));
-    assert.isAbove(getFileTimestamp(p), 0);
+    assert.strictEqual(getFileTimestamp(p) > 0, true);
   });
 
   it('should get 0 if file does not exist', () => {
@@ -286,16 +282,16 @@ describe('getFileTimestamp', () => {
 describe('getStat', () => {
   it('should be an object', () => {
     const p = path.resolve(path.join('test', 'file', 'test.sh'));
-    assert.property(getStat(p), 'mode');
+    assert.strictEqual(typeof getStat(p), 'object');
   });
 
   it('should get null if given argument is not string', () => {
-    assert.isNull(getStat());
+    assert.deepEqual(getStat(), null);
   });
 
   it('should get null if file does not exist', () => {
     const p = path.resolve(path.join('test', 'file', 'foo.txt'));
-    assert.isNull(getStat(p));
+    assert.deepEqual(getStat(p), null);
   });
 });
 

test/index.test.js

@@ -1,5 +1,5 @@
 /* api */
-import { assert } from 'chai';
+import { strict as assert } from 'node:assert';
 import { describe, it } from 'mocha';
 
 /* test */
@@ -8,6 +8,6 @@ import { Schema } from '../index.js';
 describe('Schema', () => {
   it('should be instance of Schema', () => {
     const schema = new Schema();
-    assert.instanceOf(schema, Schema);
+    assert.strictEqual(schema instanceof Schema, true);
   });
 });