test(deps): bump dev-scripts, adjust decodeableMap test to avoid constructor (#1384)

Author: mshanemc

package.json

@@ -41,7 +41,7 @@
   "devDependencies": {
     "@jsforce/jsforce-node": "^3.4.0",
     "@salesforce/cli-plugins-testkit": "^5.3.20",
-    "@salesforce/dev-scripts": "^10.2.8",
+    "@salesforce/dev-scripts": "^10.2.9",
     "@types/deep-equal-in-any-order": "^1.0.1",
     "@types/fast-levenshtein": "^0.0.4",
     "@types/graceful-fs": "^4.1.9",

src/collections/decodeableMap.ts

@@ -22,6 +22,8 @@ import { isString } from '@salesforce/ts-types';
  *
  * `decodeableMap.has('layout#Layout__Broker__c-v1%2E1 Broker Layout')` --> returns `true`
  * `decodeableMap.has('layout#Layout__Broker__c-v9.2 Broker Layout')` --> returns `true`
+ *
+ * DO NOT PASS VALUES to the Constructor.  Instantiate the class and use the methods.
  */
 export class DecodeableMap<K extends string, V> extends Map<string, V> {
   // Internal map of decoded keys to encoded keys.

test/collections/decodeableMap.test.ts

@@ -10,7 +10,7 @@ import { DecodeableMap } from '../../src/collections/decodeableMap';
 
 // passes on dev-scripts 10.2.2, fails on 10.2.4.  I don't know why.
 // possibly mocha or types/node
-describe.skip('DecodeableMap', () => {
+describe('DecodeableMap', () => {
   let dMap: DecodeableMap<string, string>;
   const layout1_key_encoded = 'Layout-v1%2E1 Layout';
   const layout1_key_decoded = 'Layout-v1.1 Layout';
@@ -28,10 +28,9 @@ describe.skip('DecodeableMap', () => {
   let deleteMapSpy: sinon.SinonSpy;
 
   beforeEach(() => {
-    dMap = new DecodeableMap([
-      [layout1_key_encoded, layout1_value],
-      [layout9_key_decoded, layout9_value],
-    ]);
+    dMap = new DecodeableMap();
+    dMap.set(layout1_key_encoded, layout1_value);
+    dMap.set(layout9_key_decoded, layout9_value);
     hasMapSpy = sandbox.spy(Map.prototype, 'has');
     getMapSpy = sandbox.spy(Map.prototype, 'get');
     setMapSpy = sandbox.spy(Map.prototype, 'set');