Ignore zigbee model casing. Koenkk#518

Oliviakrkk · Jul 15, 2019 · ab1ff3c · ab1ff3c
1 parent d967f82
commit ab1ff3c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -7,7 +7,7 @@ const fromZigbee = require('./converters/fromZigbee');
 const byZigbeeModel = new Map();
 for (const device of devices) {
     for (const zigbeeModel of device.zigbeeModel) {
-        byZigbeeModel.set(zigbeeModel, device);
+        byZigbeeModel.set(zigbeeModel.toLowerCase(), device);
     }
 }
 
@@ -18,6 +18,8 @@ module.exports = {
             return null;
         }
 
+        model = model.toLowerCase();
+
         let device = byZigbeeModel.get(model);
 
         if (!device) {

diff --git a/test/verify.js b/test/verify.js
@@ -51,14 +51,14 @@ devices.forEach((device) => {
 
     // Check for duplicate zigbee model ids
     device.zigbeeModel.forEach((m) => {
-        if (foundZigbeeModels.includes(m)) {
+        if (foundZigbeeModels.includes(m.toLowerCase())) {
             assert.fail(`Duplicate zigbee model ${m}`);
         }
     });
 
     // Check for duplicate model ids
     assert(!foundModels.includes(device.model), `Duplicate model ${device.model}`);
 
-    foundZigbeeModels = foundZigbeeModels.concat(device.zigbeeModel);
+    foundZigbeeModels = foundZigbeeModels.concat(device.zigbeeModel.map((z) => z.toLowerCase()));
     foundModels.push(device.model);
 });