mqtt-plus 1.4.0 → 1.4.1

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 ChangeLog
 =========
 
+1.4.1 (2026-02-21)
+------------------
+
+- BUGFIX: fix file references and TypeScript configuration in test directory
+- CLEANUP: cleanup code and tests
+
 1.4.0 (2026-02-21)
 ------------------
 

package/etc/stx.conf

@@ -50,6 +50,7 @@ dev
 
 #   run test suite
 test
+    tsc --project tst/tsc.json --noEmit && \
     eslint --config etc/eslint.mts tst/mqtt-plus-0-fixture.ts tst/mqtt-plus-0-mosquitto.ts tst/*.spec.ts && \
     NODE_OPTIONS="--import=tsx --trace-warnings" mocha --require tst/mqtt-plus-0-fixture.ts tst/*.spec.ts
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name":                           "mqtt-plus",
-    "version":                        "1.4.0",
+    "version":                        "1.4.1",
     "description":                    "MQTT Communication Patterns",
     "keywords":                       [   "mqtt",
                                           "event", "emit",

package/tst/mqtt-plus-4-sink.spec.ts

@@ -61,7 +61,7 @@ describe("MQTT+ Sink Push", function () {
             info.buffer!.then((buf: Uint8Array) => {
                 spy("buffer")
                 expect(Buffer.from(buf)).to.deep.equal(data)
-            })
+            }).catch(() => {})
         })
 
         /*  push a buffer (instead of a stream)  */
@@ -93,7 +93,7 @@ describe("MQTT+ Sink Push", function () {
             if (name !== "foo")
                 throw new Error("invalid sink push")
             expect(name).to.be.equal("foo")
-            expect(info).to.be.of.an("object")
+            expect(info).to.be.an("object")
             expect(info.stream).to.be.instanceOf(stream.Readable)
             const chunks: Buffer[] = []
             info.stream!.on("data", (chunk: Buffer) => {
@@ -148,7 +148,7 @@ describe("MQTT+ Sink Push", function () {
             info.buffer!.then((buf: Uint8Array) => {
                 spy("buffer")
                 expect(Buffer.from(buf)).to.deep.equal(data)
-            })
+            }).catch(() => {})
         })
 
         /*  push with metadata  */

package/tst/mqtt-plus-5-source.spec.ts

@@ -65,7 +65,7 @@ describe("MQTT+ Source Fetch", function () {
         expect(error2).to.be.equal("invalid source")
 
         /*  fetch non-existing source (invalid source name)  */
-        const result3 = await ctx.apiC.fetch("example/server/download-invalid", "foo").catch((err) => err.message)
+        const result3 = await ctx.apiC.fetch("example/server/download-invalid", "foo")
         const error3 = await result3.buffer.catch((err: Error) => {
             return err.message
         })
@@ -110,7 +110,7 @@ describe("MQTT+ Source Fetch", function () {
         ctx.apiS.meta("server-version", "1.0")
 
         /*  establish source  */
-        const sourcing = await ctx.apiS.source("example/server/download", async (filename, info) => {
+        const sourcing = await ctx.apiS.source("example/server/download", async (_filename, info) => {
             info.buffer = Promise.resolve(Buffer.from("data"))
         })
 
@@ -121,7 +121,7 @@ describe("MQTT+ Source Fetch", function () {
         expect(meta!["server-version"]).to.be.equal("1.0")
 
         /*  cleanup  */
-        ctx.apiS.meta("server-version", undefined)
+        ctx.apiS.meta("server-version", null)
         await sourcing.destroy()
     })
 })

package/tst/mqtt-plus-6-misc.spec.ts

@@ -46,7 +46,7 @@ const { expect } = chai
 
 /*  test suite  */
 describe("MQTT+ Miscellaneous", function () {
-    /*  test case: Dry-Run & Last-Will */
+    /*  test case: Dry-Run & Last-Will  */
     it("MQTT+ Dry-Run & MQTT Last-Will", async function () {
         this.timeout(3000)
 
@@ -67,7 +67,7 @@ describe("MQTT+ Miscellaneous", function () {
 
         /*  observe connection events  */
         const spy = sinon.spy()
-        apiServer.event("example/server/connection", (state) => {
+        const eventReg = await apiServer.event("example/server/connection", (state) => {
             expect(state).to.match(/^(?:open|close)$/)
             spy(state)
         })
@@ -96,8 +96,9 @@ describe("MQTT+ Miscellaneous", function () {
         await new Promise((resolve) => { setTimeout(resolve, 1000) })
 
         /*  perform regular destruction of client  */
+        await eventReg.destroy()
         apiServer.destroy()
-        mqttServer.end()
+        await mqttServer.endAsync()
 
         /*  ensure connection open and close events were seen  */
         expect(spy.getCalls().map((call) => call.firstArg))
@@ -141,13 +142,12 @@ describe("MQTT+ Miscellaneous", function () {
         })
 
         /*  call service (without token)  */
-        await ctx.apiC.call("example/server/hello", "world", 42).then(async (result) => {
+        await ctx.apiC.call("example/server/hello", "world", 42).then(async (_result) => {
             spy("call1-success")
-        }).catch((err: Error) => {
+        }).catch((_err: Error) => {
             spy("call1-error")
         })
-        expect(spy.getCalls()
-            .map((call) => call.firstArg))
+        expect(spy.getCalls().map((call) => call.firstArg))
             .to.be.deep.equal([ "call1-error" ])
         spy.resetHistory()
 
@@ -155,35 +155,32 @@ describe("MQTT+ Miscellaneous", function () {
         await ctx.apiC.call("example/server/login", userCred).then(async (token) => {
             spy("login-success")
             expect(token).to.be.equal(userToken)
-        }).catch((err: Error) => {
+        }).catch((_err: Error) => {
             spy("login-error")
         })
-        expect(spy.getCalls()
-            .map((call) => call.firstArg))
+        expect(spy.getCalls().map((call) => call.firstArg))
             .to.be.deep.equal([ "login", "login-success" ])
         spy.resetHistory()
 
         /*  call service (with wrong token)  */
         await ctx.apiC.authenticate("wrong")
-        await ctx.apiC.call("example/server/hello", "world", 42).then(async (result) => {
+        await ctx.apiC.call("example/server/hello", "world", 42).then(async (_result) => {
             spy("call2-success")
-        }).catch((err: Error) => {
+        }).catch((_err: Error) => {
             spy("call2-error")
         })
-        expect(spy.getCalls()
-            .map((call) => call.firstArg))
+        expect(spy.getCalls().map((call) => call.firstArg))
             .to.be.deep.equal([ "call2-error" ])
         spy.resetHistory()
 
         /*  call service (with correct token)  */
         await ctx.apiC.authenticate(userToken)
-        await ctx.apiC.call("example/server/hello", "world", 42).then(async (result) => {
+        await ctx.apiC.call("example/server/hello", "world", 42).then(async (_result) => {
             spy("call3-success")
-        }).catch((err: Error) => {
+        }).catch((_err: Error) => {
             spy("call3-error")
         })
-        expect(spy.getCalls()
-            .map((call) => call.firstArg))
+        expect(spy.getCalls().map((call) => call.firstArg))
             .to.be.deep.equal([ "hello", "call3-success" ])
 
         /*  destroy service  */

package/tst/tsc.json

@@ -22,7 +22,7 @@
         }
     },
     "include": [
-        "../tst/**/*.spec.ts"
+        "*.ts"
     ],
     "exclude": [
         "../node_modules"