Add checks for incorrect button names/commands

src/commands.js

@@ -56,6 +56,10 @@ class Commands {
         }
     }
 
+    static getFunctionNames() {
+        return Object.getOwnPropertyNames(this.prototype).filter(name => typeof this.prototype[name] === 'function' && name !== 'constructor');
+    }
+
     constructor(onstar) {
         this.onstar = onstar;
     }

src/mqtt.js

@@ -1,5 +1,5 @@
 const _ = require('lodash');
-//const Buttons = require('./buttons');
+const commands = require('./commands');
 
 /**
  * Supports Home Assistant MQTT Discovery (https://www.home-assistant.io/docs/mqtt/discovery/)
@@ -80,11 +80,11 @@ class MQTT {
                 Icon: 'mdi:archive-lock-open',
             },
             Start: {
-                Name: 'start',
+                Name: 'startVehicle',
                 Icon: 'mdi:car-key',
             },
             CancelStart: {
-                Name: 'cancelStart',
+                Name: 'cancelStartVehicle',
                 Icon: 'mdi:car-off',
             },
             GetLocation: {
@@ -118,6 +118,18 @@ class MQTT {
         }
     };
 
+    static validateButtonNames() {
+        const buttonNames = Object.values(MQTT.CONSTANTS.BUTTONS).map(button => button.Name);
+        const commandNames = commands.getFunctionNames();
+
+        buttonNames.forEach(buttonName => {
+            if (!commandNames.includes(buttonName)) {
+                //console.log(`Button name "${buttonName}" does not match any command in commands.js`);
+                throw new Error(`Button name "${buttonName}" does not match any command in commands.js`);
+            }
+        });
+    }
+
     constructor(vehicle, prefix = 'homeassistant', namePrefix) {
         this.prefix = prefix;
         this.vehicle = vehicle;
@@ -804,4 +816,6 @@ class MQTT {
     }
 }
 
+MQTT.validateButtonNames();
+
 module.exports = MQTT;

test/commands.spec.js

@@ -28,6 +28,15 @@ describe('Commands', () => {
         commands = new Commands(onstarMock);
     });
 
+    it('should return an array of function names', () => {
+        const functionNames = Commands.getFunctionNames();
+        assert(Array.isArray(functionNames));
+        assert(functionNames.length > 0);
+        functionNames.forEach(name => {
+            assert(typeof commands[name] === 'function');
+        });
+    });
+
     it('should call getAccountVehicles method', async () => {
         const result = await commands.getAccountVehicles();
         assert.strictEqual(result, undefined);

test/mqtt.spec.js

@@ -1518,4 +1518,4 @@ describe('MQTT', () => {
             assert.deepStrictEqual(result, expected);
         });
     });
-});
+});