Add unit tests for topic naming functions, switch to lower-case

src/publisher/mqtt/util.ts

@@ -4,8 +4,12 @@ import { TOPIC_PREFIX } from '../mqtt'
 import slugify from 'slugify'
 
 const slugifyName = (name: string): string => {
-  // We can't have "+" in MQTT topic names
-  return slugify(name, { remove: /[+]/ })
+  return slugify(name, {
+    // We can't have "+" in MQTT topic names
+    remove: /[+]/,
+    // Since the rest of the topic name is lower-case, just lower-case everything
+    lower: true,
+  })
 }
 
 export const createCharacteristicsSensorTopicName = (characteristics: Characteristics, value: string): string => {

tests/publisher/mqtt/util.test.ts

@@ -0,0 +1,19 @@
+import { Characteristics } from '../../../src/characteristics'
+import { createCharacteristicsSensorTopicName, createPowerSensorTopicName } from '../../../src/publisher/mqtt/util'
+import { Circuit } from '../../../src/circuit'
+
+test('creates correct topic names', () => {
+  const characteristics = {
+    name: 'Some characteristic',
+  } as unknown as Characteristics
+
+  let topic = createCharacteristicsSensorTopicName(characteristics, 'voltage')
+  expect(topic).toEqual('eachwatt/characteristic/some-characteristic/voltage')
+
+  const circuit = {
+    name: 'Some circuit + with plus sign',
+  } as unknown as Circuit
+
+  topic = createPowerSensorTopicName(circuit, 'power')
+  expect(topic).toEqual('eachwatt/circuit/some-circuit-with-plus-sign/power')
+})