X-Git-Url: https://git.squeep.com/?a=blobdiff_plain;f=test%2Flib%2Fmystery-box.js;h=510362bd06c8ab3a7884909e41404e0350d40494;hb=HEAD;hp=e949139aefb97c91c2dace523d5c268f4d28010b;hpb=6c9e123b3c10ef5caafc1a5f352a4705a8579ca9;p=squeep-mystery-box

diff --git a/test/lib/mystery-box.js b/test/lib/mystery-box.js
index e949139..c0b5d9a 100644
--- a/test/lib/mystery-box.js
+++ b/test/lib/mystery-box.js
@@ -1,12 +1,10 @@
-/* eslint-env mocha */
-/* eslint-disable capitalized-comments */
 'use strict';
 
-const assert = require('assert');
-const sinon = require('sinon'); // eslint-disable-line node/no-unpublished-require
+const assert = require('node:assert');
+const sinon = require('sinon');
 const MysteryBox = require('../../lib/mystery-box');
 const { MysteryBoxError } = require('../../lib/errors');
-const crypto = require('crypto');
+const crypto = require('node:crypto');
 
 function _verbose(mb) {
   if (process.env.VERBOSE_TESTS) {
@@ -59,10 +57,27 @@ describe('MysteryBox', function () {
 
     it('covers missing ciphers', function () {
       sinon.stub(crypto, 'getCiphers').returns(['rot13']);
-      assert.rejects(() => new MysteryBox(options));
+      assert.rejects(() => new MysteryBox(options), MysteryBoxError);
     });
   }); // constructor
 
+  describe('_ensureArray', function () {
+    it('returns empty array for no data', function () {
+      const result = MysteryBox._ensureArray();
+      assert.deepStrictEqual(result, []);
+    });
+    it('returns same array passed in', function () {
+      const expected = [1, 2, 3, 'foo'];
+      const result = MysteryBox._ensureArray(expected);
+      assert.deepStrictEqual(result, expected);
+    });
+    it('returns array containing non-array data', function () {
+      const data = 'bar';
+      const result = MysteryBox._ensureArray(data);
+      assert.deepStrictEqual(result, [data]);
+    });
+  }); // _ensureArray
+
   describe('_keyFromSecret', function () {
     it('covers invalid', async function () {
       assert.rejects(() => MysteryBox._keyFromSecret('unknown deriver', 'secret', 'salt', 32), MysteryBoxError);
@@ -214,22 +229,24 @@ describe('MysteryBox', function () {
       _checkEncodeRange(268435445, 268435455, 4, 0xe0);
     });
     it('covers quintuple-byte-packable versions', function () {
-      _checkEncodeRange(268435456, 268435465, 5, 0xf0)
+      _checkEncodeRange(268435456, 268435465, 5, 0xf0);
       /* ... */
-      _checkEncodeRange(4294967294, 4294967296, 5, 0xf0)
+      _checkEncodeRange(4294967294, 4294967296, 5, 0xf0);
       /* ... */
-      _checkEncodeRange(34359738360, 34359738367, 5, 0xf0)
+      _checkEncodeRange(34359738360, 34359738367, 5, 0xf0);
     });
     it('covers sextuple-byte-packable versions', function () {
-      _checkEncodeRange(34359738368, 34359738377, 6, 0xf8)
+      _checkEncodeRange(34359738368, 34359738377, 6, 0xf8);
       /* ... */
-      _checkEncodeRange(4398046511093, 4398046511103, 6, 0xf8)
+      _checkEncodeRange(4398046511093, 4398046511103, 6, 0xf8);
     });
     it('covers too large', function () {
       const version = 277076930199552;
       assert.rejects(() => MysteryBox._versionEncode(version), MysteryBoxError);
     });
+    /* eslint-disable array-element-newline */
     it('recipricates', function () {
+      // eslint-disable array-element-newline
       [
         0, 127,
         128, 16383,
@@ -240,6 +257,7 @@ describe('MysteryBox', function () {
       ].forEach((v) => _cheeckReciprical(v));
     });
   }); // _versionEncode
+  /* eslint-enable array-element-newline */
 
   describe('pack, unpack', function () {
     beforeEach(function () {
@@ -318,7 +336,9 @@ describe('MysteryBox', function () {
     });
 
     it('encrypts and decrypts all available versions no compression', async function () {
-      Object.keys(mb.versionParameters).map((v) => Number(v)).forEach(async (version) => {
+      this.slow(Object.keys(mb.versionParameters).length * 200);
+      const availableVersions = Object.keys(mb.versionParameters).map((v) => Number(v));
+      for await (const version of availableVersions) {
         object = {
           foo: 'bar',
           baz: 'quux',
@@ -327,11 +347,13 @@ describe('MysteryBox', function () {
         const encryptedResult = await mb.pack(object, version, 0x00);
         const decryptedResult = await mb.unpack(encryptedResult);
         assert.deepStrictEqual(decryptedResult, object, `${version} results not symmetric`);
-      });
+      }
     });
 
     it('encrypts and decrypts all available versions +brotli', async function () {
-      Object.keys(mb.versionParameters).map((v) => Number(v)).forEach(async (version) => {
+      this.slow(Object.keys(mb.versionParameters).length * 200);
+      const availableVersions = Object.keys(mb.versionParameters).map((v) => Number(v));
+      for await (const version of availableVersions) {
         object = {
           foo: 'bar',
           baz: 'quux',
@@ -340,11 +362,13 @@ describe('MysteryBox', function () {
         const encryptedResult = await mb.pack(object, version, mb.Flags.Brotli);
         const decryptedResult = await mb.unpack(encryptedResult);
         assert.deepStrictEqual(decryptedResult, object, `${version} results not symmetric`);
-      });
+      };
     });
 
     it('encrypts and decrypts all available versions +flate', async function () {
-      Object.keys(mb.versionParameters).map((v) => Number(v)).forEach(async (version) => {
+      this.slow(Object.keys(mb.versionParameters).length * 200);
+      const availableVersions = Object.keys(mb.versionParameters).map((v) => Number(v));
+      for await (const version of availableVersions) {
         object = {
           foo: 'bar',
           baz: 'quux',
@@ -353,7 +377,7 @@ describe('MysteryBox', function () {
         const encryptedResult = await mb.pack(object, version, mb.Flags.Flate);
         const decryptedResult = await mb.unpack(encryptedResult);
         assert.deepStrictEqual(decryptedResult, object, `${version} results not symmetric`);
-      });
+      };
     });
 
     it('handles large object +brotli', async function () {
@@ -362,7 +386,7 @@ describe('MysteryBox', function () {
       const firstChar = 32, lastChar = 126;
       const rnd = () => {
         return Math.floor(firstChar + (lastChar - firstChar + 1) * Math.random());
-      }
+      };
       object = {
         longProperty: 'x'.repeat(384 * 1024).split('').map(() => String.fromCharCode(rnd())).join(''),
       };
@@ -377,7 +401,7 @@ describe('MysteryBox', function () {
       const firstChar = 32, lastChar = 126;
       const rnd = () => {
         return Math.floor(firstChar + (lastChar - firstChar + 1) * Math.random());
-      }
+      };
       object = {
         longProperty: 'x'.repeat(384 * 1024).split('').map(() => String.fromCharCode(rnd())).join(''),
       };
@@ -399,7 +423,7 @@ describe('MysteryBox', function () {
     it('covers internal error, incorrect version byte size, pack', async function () {
       this.slow(500);
       const version = 1;
-      sinon.stub(mb.versionParameters[version], 'versionBytes').value(10);
+      sinon.stub(mb.versionParameters[version], 'versionBytes').value(10); // eslint-disable-line security/detect-object-injection
       assert.rejects(() => mb.pack({}, version), Error);
     });
 
@@ -407,7 +431,7 @@ describe('MysteryBox', function () {
       this.slow(500);
       const version = 1;
       const encryptedResult = await mb.pack({}, version);
-      sinon.stub(mb.versionParameters[version], 'versionBytes').value(10);
+      sinon.stub(mb.versionParameters[version], 'versionBytes').value(10); // eslint-disable-line security/detect-object-injection
       assert.rejects(() => mb.unpack(encryptedResult), Error);
     });