CoAP causes build issues

[danielpeintner]

In #727 I wanted to update all version numbers from 0.8.0 to 0.8.1.

Doing so I removed package-lock.json file and recreated it calling npm install.
Unfortunately this caused issues in binding-coap.

I wonder also why we use out-dated libraries.
e.g., "node-coap-client": "1.0.8" vs "node-coap-client": "2.0.2"

@JKRhb do you have a clue why this issue is happening?

[JKRhb]

Oh, I think this is probably caused by the fact that @types/bl is only declared as a devDependency and not as a dependency :/ I will open a PR for that in node-coap

[JKRhb]

I opened coapjs/node-coap#333, I hope that will fix the issue.

[danielpeintner]

I added those types as devDependencies for the time being
see 897d3dd

I think doing so should remove the issue...

We should remove them again if the root cause is resolved.

[danielpeintner]

I reverted 897d3dd and updated to 1.0.5 but I get new issues in coap-server-test.ts

[danielpeintner]

FYI, I see

  1) CoAP server implementation
       should read property:
     Uncaught TypeError: Cannot read properties of null (reading 'formIndex')
      at ExposedThing.handleUnobserveProperty (C:\Projects\WoT\thingweb\thingweb.node-wot-danielpeintner\packages\core\src\exposed-thing.ts:1277:25)
      at ObserveWriteStream.<anonymous> (src\coap-server.ts:369:43)

OR

  2) CoAP server implementation
       should read property:
     done() called multiple times in test <CoAP server implementation should read property> of file C:\Projects\WoT\thingweb\thingweb.node-wot-danielpeintner\packages\binding-coap\test\coap-server-test.ts; in addition, done() received error: AssertionError: expected '' to equal '"testValue"'
    at CoapServerTest.<anonymous> (C:\Projects\WoT\thingweb\thingweb.node-wot-danielpeintner\packages\binding-coap\test\coap-server-test.ts:68:82)

[JKRhb]

It took me a while, but I found the issue that caused the strange behavior in node-coap version 1.0.5 (the observe option was always set) and opened coapjs/node-coap#337 to finally fix it :)