feat: add unit test setup with node:test

Add comprehensive unit test infrastructure using Node.js native test runner:
- 38 tests across 5 utility functions
- Tests for mph2Beaufort, roundValue, cardinalWindDirection, convertWeatherType, getOrdinal
- Updated CI workflow to run tests before linting

Author: KristjanESPERANTO

.github/workflows/automated-tests.yaml

@@ -27,6 +27,8 @@ jobs:
           cache: npm
       - name: Install dependencies
         run: npm ci
+      - name: Run unit tests
+        run: node --run test:unit
       - name: Check spelling
         run: node --run test:spelling
       - name: Check linting

eslint.config.mjs

@@ -71,6 +71,27 @@ export default defineConfig([
       "sort-keys": "off"
     }
   },
+  {
+    files: ["tests/**/*.test.mjs"],
+    languageOptions: {
+      ecmaVersion: "latest",
+      globals: {
+        ...globals.node
+      },
+      sourceType: "module"
+    },
+    plugins: { js, stylistic },
+    extends: [importX.recommended, stylistic.configs.customize({ indent: "tab", quotes: "double", semi: true, commaDangle: "never" })],
+    rules: {
+      "@stylistic/indent": ["error", 2],
+      "init-declarations": "off",
+      "max-lines-per-function": "off",
+      "max-statements": "off",
+      "no-inline-comments": "off",
+      "no-magic-numbers": "off",
+      "no-undefined": "off"
+    }
+  },
   { files: ["demo.config.js"], rules: { "prefer-const": "off" } },
   { files: ["**/*.json"], ignores: ["package-lock.json"], plugins: { json }, extends: ["json/recommended"], language: "json/json" },
   { files: ["**/*.md"], plugins: { markdown }, extends: ["markdown/recommended"], language: "markdown/gfm" }

package.json

@@ -29,8 +29,9 @@
     "lint:fix": "eslint --fix && prettier . --write",
     "prepare": "node -e \"try{require('simple-git-hooks').run()}catch(e){}\"",
     "release": "commit-and-tag-version --commit-all --config ./changelog.config.js",
-    "test": "node --run lint && node --run test:spelling",
-    "test:spelling": "cspell ."
+    "test": "node --run test:unit && node --run lint && node --run test:spelling",
+    "test:spelling": "cspell .",
+    "test:unit": "node --test tests/**/*.test.mjs"
   },
   "simple-git-hooks": {
     "pre-commit": "npx lint-staged"

tests/cardinalWindDirection.test.mjs

@@ -0,0 +1,59 @@
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { weatherModule } from "./test-setup.mjs";
+
+describe("cardinalWindDirection", () => {
+  it("should return N for 0 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(0), "N");
+  });
+
+  it("should return N for 360 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(360), "N");
+  });
+
+  it("should return NE for 45 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(45), "NE");
+  });
+
+  it("should return E for 90 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(90), "E");
+  });
+
+  it("should return SE for 135 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(135), "SE");
+  });
+
+  it("should return S for 180 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(180), "S");
+  });
+
+  it("should return SW for 225 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(225), "SW");
+  });
+
+  it("should return W for 270 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(270), "W");
+  });
+
+  it("should return NW for 315 degrees", () => {
+    assert.equal(weatherModule.cardinalWindDirection(315), "NW");
+  });
+
+  it("should handle intermediate values", () => {
+    assert.equal(weatherModule.cardinalWindDirection(22), "NNE");
+    assert.equal(weatherModule.cardinalWindDirection(67), "ENE");
+    assert.equal(weatherModule.cardinalWindDirection(112), "ESE");
+    assert.equal(weatherModule.cardinalWindDirection(157), "SSE");
+    assert.equal(weatherModule.cardinalWindDirection(202), "SSW");
+    assert.equal(weatherModule.cardinalWindDirection(247), "WSW");
+    assert.equal(weatherModule.cardinalWindDirection(292), "WNW");
+    assert.equal(weatherModule.cardinalWindDirection(337), "NNW");
+  });
+
+  it("should handle boundary values", () => {
+    assert.equal(weatherModule.cardinalWindDirection(11.24), "N");
+    assert.equal(weatherModule.cardinalWindDirection(11.26), "NNE");
+    assert.equal(weatherModule.cardinalWindDirection(348.74), "NNW");
+    assert.equal(weatherModule.cardinalWindDirection(348.76), "N");
+  });
+});

tests/convertWeatherType.test.mjs

@@ -0,0 +1,39 @@
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { weatherModule } from "./test-setup.mjs";
+
+describe("convertWeatherType", () => {
+  it("should convert day icons correctly", () => {
+    assert.equal(weatherModule.convertWeatherType("01d"), "day-clear-sky");
+    assert.equal(weatherModule.convertWeatherType("02d"), "day-few-clouds");
+    assert.equal(weatherModule.convertWeatherType("03d"), "day-scattered-clouds");
+    assert.equal(weatherModule.convertWeatherType("04d"), "day-broken-clouds");
+    assert.equal(weatherModule.convertWeatherType("09d"), "day-shower-rain");
+    assert.equal(weatherModule.convertWeatherType("10d"), "day-rain");
+    assert.equal(weatherModule.convertWeatherType("11d"), "day-thunderstorm");
+    assert.equal(weatherModule.convertWeatherType("13d"), "day-snow");
+    assert.equal(weatherModule.convertWeatherType("50d"), "day-mist");
+  });
+
+  it("should convert night icons correctly", () => {
+    assert.equal(weatherModule.convertWeatherType("01n"), "night-clear-sky");
+    assert.equal(weatherModule.convertWeatherType("02n"), "night-few-clouds");
+    assert.equal(weatherModule.convertWeatherType("03n"), "night-scattered-clouds");
+    assert.equal(weatherModule.convertWeatherType("04n"), "night-broken-clouds");
+    assert.equal(weatherModule.convertWeatherType("09n"), "night-shower-rain");
+    assert.equal(weatherModule.convertWeatherType("10n"), "night-rain");
+    assert.equal(weatherModule.convertWeatherType("11n"), "night-thunderstorm");
+    assert.equal(weatherModule.convertWeatherType("13n"), "night-snow");
+    assert.equal(weatherModule.convertWeatherType("50n"), "night-mist");
+  });
+
+  it("should return null for unknown weather types", () => {
+    assert.equal(weatherModule.convertWeatherType("99x"), null);
+    assert.equal(weatherModule.convertWeatherType(""), null);
+    assert.equal(weatherModule.convertWeatherType("invalid"), null);
+  });
+
+  it("should handle undefined input", () => {
+    assert.equal(weatherModule.convertWeatherType(undefined), null);
+  });
+});

tests/getOrdinal.test.mjs

@@ -0,0 +1,55 @@
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { weatherModule } from "./test-setup.mjs";
+
+describe("getOrdinal", () => {
+  it("should return N for 0 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(0), "N");
+  });
+
+  it("should return N for 360 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(360), "N");
+  });
+
+  it("should return NNE for 22.5 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(22.5), "NNE");
+  });
+
+  it("should return NE for 45 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(45), "NE");
+  });
+
+  it("should return E for 90 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(90), "E");
+  });
+
+  it("should return SE for 135 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(135), "SE");
+  });
+
+  it("should return S for 180 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(180), "S");
+  });
+
+  it("should return SW for 225 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(225), "SW");
+  });
+
+  it("should return W for 270 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(270), "W");
+  });
+
+  it("should return NW for 315 degrees", () => {
+    assert.equal(weatherModule.getOrdinal(315), "NW");
+  });
+
+  it("should handle values between cardinal directions", () => {
+    assert.equal(weatherModule.getOrdinal(11), "N");
+    assert.equal(weatherModule.getOrdinal(12), "NNE");
+  });
+
+  it("should handle large bearing values via modulo", () => {
+    assert.equal(weatherModule.getOrdinal(720), "N"); // 720 % 360 = 0
+    assert.equal(weatherModule.getOrdinal(405), "NE"); // 405 % 360 = 45
+  });
+});

tests/mph2Beaufort.test.mjs

@@ -0,0 +1,42 @@
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { weatherModule } from "./test-setup.mjs";
+
+describe("mph2Beaufort", () => {
+  it("should convert calm winds (0-1 mph) to Beaufort 0", () => {
+    assert.equal(weatherModule.mph2Beaufort(0), 0);
+    assert.equal(weatherModule.mph2Beaufort(0.5), 0);
+  });
+
+  it("should convert light air (1-3 mph) to Beaufort 1", () => {
+    assert.equal(weatherModule.mph2Beaufort(1), 1);
+    assert.equal(weatherModule.mph2Beaufort(2), 1);
+  });
+
+  it("should convert light breeze (4-7 mph) to Beaufort 2", () => {
+    assert.equal(weatherModule.mph2Beaufort(4), 2);
+    assert.equal(weatherModule.mph2Beaufort(6), 2);
+  });
+
+  it("should convert gentle breeze (8-12 mph) to Beaufort 3", () => {
+    assert.equal(weatherModule.mph2Beaufort(8), 3);
+    assert.equal(weatherModule.mph2Beaufort(11), 3);
+  });
+
+  it("should convert moderate breeze (13-18 mph) to Beaufort 4", () => {
+    assert.equal(weatherModule.mph2Beaufort(13), 4);
+    assert.equal(weatherModule.mph2Beaufort(17), 4);
+  });
+
+  it("should convert hurricane force (73+ mph) to Beaufort 12", () => {
+    assert.equal(weatherModule.mph2Beaufort(73), 12);
+    assert.equal(weatherModule.mph2Beaufort(100), 12);
+    assert.equal(weatherModule.mph2Beaufort(200), 12);
+  });
+
+  it("should handle edge cases", () => {
+    // Test boundary values
+    assert.equal(weatherModule.mph2Beaufort(3.1), 1); // Still Beaufort 1
+    assert.equal(weatherModule.mph2Beaufort(72), 11); // Just under hurricane force
+  });
+});

tests/roundValue.test.mjs

@@ -0,0 +1,36 @@
+import { beforeEach, describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { weatherModule } from "./test-setup.mjs";
+
+describe("roundValue", () => {
+  beforeEach(() => {
+    weatherModule.config = { roundTemp: false };
+  });
+
+  it("should round to 1 decimal when roundTemp is false", () => {
+    assert.equal(weatherModule.roundValue(23.456), "23.5");
+    assert.equal(weatherModule.roundValue(23.444), "23.4");
+  });
+
+  it("should round to integer when roundTemp is true", () => {
+    weatherModule.config = { roundTemp: true };
+    assert.equal(weatherModule.roundValue(23.456), "23");
+    assert.equal(weatherModule.roundValue(23.6), "24");
+  });
+
+  it("should handle negative temperatures", () => {
+    weatherModule.config = { roundTemp: false };
+    assert.equal(weatherModule.roundValue(-5.67), "-5.7");
+
+    weatherModule.config = { roundTemp: true };
+    assert.equal(weatherModule.roundValue(-5.67), "-6");
+  });
+
+  it("should handle zero", () => {
+    weatherModule.config = { roundTemp: false };
+    assert.equal(weatherModule.roundValue(0), "0.0");
+
+    weatherModule.config = { roundTemp: true };
+    assert.equal(weatherModule.roundValue(0), "0");
+  });
+});

tests/test-setup.mjs

@@ -0,0 +1,38 @@
+// Mock MagicMirror globals
+globalThis.config = { units: "metric" };
+
+// Mock the Module.register to extract methods
+let weatherModule = null;
+
+globalThis.Module = {
+  register(name, definition) {
+    weatherModule = definition;
+    // Set config defaults for methods that need them
+    weatherModule.config = {
+      roundTemp: false,
+      labelOrdinals: [
+        "N",
+        "NNE",
+        "NE",
+        "ENE",
+        "E",
+        "ESE",
+        "SE",
+        "SSE",
+        "S",
+        "SSW",
+        "SW",
+        "WSW",
+        "W",
+        "WNW",
+        "NW",
+        "NNW"
+      ]
+    };
+  }
+};
+
+// Load the module
+await import("../MMM-OneCallWeather.js");
+
+export { weatherModule };