feat: add testing coverage and corner case tests

Author: gagik

packages/machine-id/package.json

@@ -9,7 +9,8 @@
     "bootstrap": "npm run compile",
     "pretest": "npm run compile",
     "test": "mocha",
-    "test-ci": "npm run test",
+    "test-cov": "nyc -x \"**/*.spec.*\" --reporter=lcov --reporter=text --reporter=html npm run test",
+    "test-ci": "npm run test-cov",
     "lint": "eslint . && prettier --check .",
     "check": "npm run lint && npm run test",
     "prepublishOnly": "npm run compile",

packages/machine-id/src/bin/machine-id.ts

@@ -1,12 +1,7 @@
 #!/usr/bin/env node
 import { getMachineId } from '..';
 
-// Check if --raw flag is provided
-const rawFlag = process.argv.includes('--raw');
-
-// Get the machine ID, passing the raw option if requested
-const id =
-  getMachineId({ raw: rawFlag }) || 'Machine ID not available on this platform';
+const id = getMachineId({ raw: process.argv.includes('--raw') }) || '';
 
 // eslint-disable-next-line no-console
 console.log(id);

packages/machine-id/src/index.spec.ts

@@ -1,8 +1,13 @@
 /* eslint-disable no-console */
 import { getMachineId } from '.';
 import { machineIdSync as otherMachineId } from 'node-machine-id';
-import { expect } from 'chai';
+import chai, { expect } from 'chai';
 import { createHash } from 'crypto';
+import sinonChai from 'sinon-chai';
+import sinon from 'sinon';
+import bindings from 'bindings';
+
+chai.use(sinonChai);
 
 describe('machine-id', function () {
   this.timeout(5_000);
@@ -53,6 +58,43 @@ describe('machine-id', function () {
       const hashRegex = /^[0-9a-f]{64}$/i;
 
       expect(hashRegex.test(id));
+
+      expect(id).equals(
+        createHash('sha256')
+          .update(getMachineId({ raw: true }) || '')
+          .digest('hex'),
+      );
+    });
+  });
+
+  describe('edge cases', function () {
+    afterEach(function () {
+      sinon.restore();
+    });
+
+    describe('returns undefined', function () {
+      it('if something goes wrong with the binding function', function () {
+        sinon
+          .stub(bindings('machine_id'), 'getMachineId')
+          .throws(new Error('Binding error'));
+
+        expect(getMachineId({ raw: true })).to.be.undefined;
+        expect(getMachineId()).to.be.undefined;
+      });
+
+      it('if the binding function returns an empty string', function () {
+        sinon.stub(bindings('machine_id'), 'getMachineId').returns('');
+
+        expect(getMachineId({ raw: true })).to.be.undefined;
+        expect(getMachineId()).to.be.undefined;
+      });
+
+      it('if the binding function returns undefined', function () {
+        sinon.stub(bindings('machine_id'), 'getMachineId').returns(undefined);
+
+        expect(getMachineId({ raw: true })).to.be.undefined;
+        expect(getMachineId()).to.be.undefined;
+      });
     });
   });
 });

packages/machine-id/src/index.ts

@@ -15,7 +15,14 @@ export type GetMachineIdOptions = {
 export function getMachineId({ raw = false }: GetMachineIdOptions = {}):
   | string
   | undefined {
-  const machineId: string | undefined = binding.getMachineId();
+  let machineId: string | undefined;
+
+  try {
+    machineId = binding.getMachineId();
+  } catch {
+    // If the binding fails, we can assume the machine ID is not available.
+    return undefined;
+  }
 
   if (!machineId) {
     return undefined;