Bluetooth service

[atrovato]

Pull Request check-list

To ensure your Pull Request can be accepted as fast as possible, make sure to review and check all of these items:

• If your changes affects code, did your write the tests?
• Are tests passing? (npm test on both front/server)
• Is the linter passing? (npm run eslint on both front/server)
• Did you run prettier? (npm run prettier on both front/server)
• If your changes modify the API (REST or Node.js), did you modify the documentation? (Documentation is based on comments in code)

NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.

Description of change

Bluetooth service to handle Bluetooth devices.

[atrovato]

Wait #512 PR to be merged to get device / feature definition updates.

[Pierre-Gilles]

Hello! De retour de congés, je reprend les PR une par une.

A quel niveau es-tu sur celle-ci?

[atrovato]

Salut, suite à notre discussion sur MQTT et les sub-services, ce service nécessite de grandes modifications. Du coup, la relecture n'est pas encore nécessaire.

[atrovato]

The service is still in standby, the community is moving on noble library, and some others may appear soon.
I'll try new version as there are available.

[Pierre-Gilles]

Ok!

[codecov]

Codecov Report

Merging #505 into master will increase coverage by 0.02%.
The diff coverage is 93.86%.

@@            Coverage Diff             @@
##           master     #505      +/-   ##
==========================================
+ Coverage   93.50%   93.53%   +0.02%     
==========================================
  Files         428      461      +33     
  Lines        5607     6112     +505     
==========================================
+ Hits         5243     5717     +474     
- Misses        364      395      +31     

Impacted Files	Coverage Δ
server/utils/constants.js	100.00% <ø> (ø)
...vices/bluetooth/lib/commands/bluetooth.setValue.js	33.33% <33.33%> (ø)
...ices/bluetooth/lib/device/bluetooth.information.js	75.40% <75.40%> (ø)
...bluetooth/lib/commands/bluetooth.connectDevices.js	87.50% <87.50%> (ø)
...ervices/bluetooth/lib/events/bluetooth.scanStop.js	87.50% <87.50%> (ø)
...th/lib/utils/characteristic/bluetooth.subscribe.js	87.50% <87.50%> (ø)
.../services/bluetooth/lib/commands/bluetooth.scan.js	90.32% <90.32%> (ø)
...ooth/lib/commands/bluetooth.subscribePeripheral.js	90.90% <90.90%> (ø)
...etooth/lib/commands/bluetooth.applyOnPeripheral.js	92.85% <92.85%> (ø)
...etooth/lib/utils/characteristic/bluetooth.write.js	93.33% <93.33%> (ø)
... and 58 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc98c16...cd036e6. Read the comment docs.

[Pierre-Gilles]

Back on Bluetooth ? 😍

[atrovato]

Just tried with last @abandonware/noble, but not working on windows...
And update branch to fix merge conflicts by the way.

[Pierre-Gilles]

Thank you very much for this PR, this is gold 🥇

I just put one small comment.

Do you think it could be merged in master? I tested it on my side, it works fine on the UI/discovery part.

I don't have much devices to tests so I'm not the best user to test this, but if it's stable we can definitely merge it on master so all users can try it before the RC :)

[Pierre-Gilles]

Hey! Thanks for your changes.

Is this ready to merge?

[LM1LC3N7]

🥳

[Pierre-Gilles]

Many thanks for your work on this major PR @atrovato ! 🥳

This was the n°1 requests for Gladys 4, and we now have it :)

Can't wait to see more integrations around the bluetooth in Gladys.