refactor: Upgrade commander from 5.1.0 to 10.0.1 (#8557)

Author: dblythy

package-lock.json

@@ -28,7 +28,7 @@
     "@parse/push-adapter": "4.1.2",
     "bcryptjs": "2.4.3",
     "body-parser": "1.20.1",
-    "commander": "5.1.0",
+    "commander": "10.0.1",
     "cors": "2.8.5",
     "deepcopy": "2.1.0",
     "express": "4.18.2",

package.json

@@ -74,7 +74,7 @@ describe('commander additions', () => {
     done();
   });
 
-  it('should load properly use args over env', done => {
+  it('should load properly use args over env', () => {
     commander.loadDefinitions(testDefinitions);
     commander.parse(['node', './CLI.spec.js', '--arg0', 'arg0Value', '--arg4', ''], {
       PROGRAM_ARG_0: 'arg0ENVValue',
@@ -86,7 +86,6 @@ describe('commander additions', () => {
     expect(commander.arg1).toEqual('arg1ENVValue');
     expect(commander.arg2).toEqual(4);
     expect(commander.arg4).toEqual('');
-    done();
   });
 
   it('should fail in action as port is invalid', done => {

spec/CLI.spec.js

@@ -136,5 +136,5 @@ Command.prototype.getOptions = function () {
   }, {});
 };
 
-export default new Command();
+export default new Command().storeOptionsAsProperties();
 /* eslint-enable no-console */