Improve test coverage

README.md

@@ -6,6 +6,7 @@
   <a href="https://www.npmjs.com/package/hap-nodejs/v/beta"><img title="npm version beta" src="https://badgen.net/npm/v/hap-nodejs/beta" ></a>
   <a href="https://www.npmjs.com/package/hap-nodejs"><img title="npm downloads" src="https://badgen.net/npm/dt/hap-nodejs" ></a>
   <a href="https://github.com/homebridge/HAP-NodeJS/actions/workflows/build.yml"><img title="Node Build" src="https://github.com/homebridge/HAP-NodeJS/actions/workflows/build.yml/badge.svg" ></a>
+  <a href='https://coveralls.io/github/homebridge/HAP-NodeJS'><img src='https://coveralls.io/repos/github/homebridge/HAP-NodeJS/badge.svg' alt='Coverage Status' /></a>
 
 </span>
 

jest.config.json

@@ -0,0 +1,12 @@
+{
+  "preset": "ts-jest",
+  "testEnvironment": "node",
+  "coverageReporters": ["lcov"],
+  "collectCoverageFrom": [
+    "src/**",
+    "!src/accessories/**",
+    "!src/lib/definitions/generate-definitions.ts",
+    "!src/lib/definitions/generator-configuration.ts",
+    "!src/test-utils"
+  ]
+}

package.json

@@ -53,9 +53,9 @@
   "dependencies": {
     "@homebridge/ciao": "^1.1.5",
     "@homebridge/dbus-native": "^0.5.0",
-    "bonjour-hap": "~3.6.3",
+    "bonjour-hap": "~3.6.4",
     "debug": "^4.3.4",
-    "fast-srp-hap": "2.0.4",
+    "fast-srp-hap": "~2.0.4",
     "futoin-hkdf": "~1.4.3",
     "node-persist": "^0.0.11",
     "source-map-support": "^0.5.21",
@@ -71,6 +71,7 @@
     "@typescript-eslint/parser": "^5.21.0",
     "commander": "^6.2.1",
     "escape-html": "^1.0.3",
+    "http-parser-js": "^0.5.8",
     "eslint": "^8.14.0",
     "jest": "^27.5.1",
     "rimraf": "^3.0.2",
@@ -79,6 +80,7 @@
     "ts-jest": "^27.1.4",
     "ts-node": "^10.7.0",
     "typedoc": "^0.22.15",
-    "typescript": "~4.5.4"
+    "typescript": "~4.5.4",
+    "axios": "^0.27.2"
   }
 }

src/accessories/Light_accessory.ts

@@ -14,7 +14,7 @@ import {
 
 class LightControllerClass {
 
-  name: CharacteristicValue = "Simple Light"; //name of accessory
+  name = "Simple Light"; //name of accessory
   pincode: CharacteristicValue = "031-45-154";
   username: CharacteristicValue = "FA:3C:ED:5A:1A:1A"; // MAC like address used by HomeKit to differentiate accessories.
   manufacturer: CharacteristicValue = "HAP-NodeJS"; //manufacturer (optional)
@@ -135,7 +135,7 @@ lightbulb.getCharacteristic(Characteristic.On)
     // requests from HomeKit.
     callback();
   })
-  // We want to intercept requests for our current power state so we can query the hardware itself instead of
+  // We want to intercept requests for our current power state, so we can query the hardware itself instead of
   // allowing HAP-NodeJS to return the cached Characteristic.value.
   .on(CharacteristicEventTypes.GET, (callback: NodeCallback<CharacteristicValue>) => {
     callback(null, LightController.getPower());

src/index.spec.ts

@@ -0,0 +1,8 @@
+// we just test that we can import index e.g. without any cyclic imports
+import "./index";
+
+describe("index", () => {
+  test("test index import", () => {
+    expect(true).toBeTruthy();
+  });
+});