FlowFuse / FlowFuse/device-agent

Update aedes to 1.x to resolve dependabot #676

Open
#705 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
20
Forks
10
Avg merge
9d 4h
Merged PRs (30d)
9

Description

Small rework on unit tests is required on how aedes is initialised

### Before (v0.x):
```js
const aedes = require('aedes')
const broker = aedes(opts)
```

### After (v1.0.0):
```js
const { Aedes } = require('aedes')
const broker = await Aedes.createBroker(opts)
```

_Originally posted by @Steve-Mcl in https://github.com/FlowFuse/device-agent/issues/676#issuecomment-5102773320_

> breaking changes in aedes 1.x
>
> https://github.com/moscajs/aedes/releases#release-v1.0.0
>
> Small rework on unit tests is required

Contributor guide

Open the contributing guide

Research direction

Locate the aedes dependency and the unit tests that initialize the broker. Read the aedes 1.0.0 breaking changes, update the dependency, and adapt the test setup to the new asynchronous Aedes.createBroker API. Run the unit test suite; done means the tests pass with aedes 1.x.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
backend, testing-qa
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.