Multiple Device Support

[PietroLubini]

Make sure that the pull request meets DoD criteria:

Documentation

• [x] Changes that makes documentation outdated are described in README.md

Maintainability

• [x] Reasonable amount of logging has been added (Debug level)
• [x] All errors are logged with Error level
• [x] Validation errors are logged with Warning level

Testing

• [x] Plugin does not crash when configuration is not provided (default state)
• [x] New functionality is dev tested in Homebridge
• [x] Errors that are produced by plugin do not crash Homebridge

Changes:

• Added support for multiple devices:
  • Devices for the same EcoFlow account share a mqtt connection
  • Devices for different EcoFlow accounts use a mqtt connection that is established for each EcoFlow account
• Resolves #11
  • Stop crashing Homebridge when
    • Failed to establish Mqtt connection
    • Subscription to topic is failed
    • Sending of command is failed

[github-actions[bot]]

Coverage Report (20.x)

St.:grey_question:	Category	Percentage	Covered / Total
🟢	Statements	100%	582/582
🟢	Branches	100%	140/140
🟢	Functions	100%	159/159
🟢	Lines	100%	569/569

Show new covered files 🐣

|

St.^{:grey_question:}

| File | Statements | Branches | Functions | Lines | | :-: | :- | :- | :- | :- | :- | | 🟢 |

`...` / deviceInfo.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / mqttDevice.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / mqttApiContracts.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / ecoFlowMqttApiManager.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / mqttClient.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / ecoFlowHttpApiManager.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / httpApiContracts.ts

| 100% | 100% | 100% | 100% |

Test suite run success

216 tests passing in 16 suites.

Report generated by 🧪jest coverage report action from 680bc3044d0bc137c4c006402e595df42ff6202c

[github-actions[bot]]

Coverage Report (18.x)

St.:grey_question:	Category	Percentage	Covered / Total
🟢	Statements	100%	582/582
🟢	Branches	100%	140/140
🟢	Functions	100%	159/159
🟢	Lines	100%	569/569

Show new covered files 🐣

|

St.^{:grey_question:}

| File | Statements | Branches | Functions | Lines | | :-: | :- | :- | :- | :- | :- | | 🟢 |

`...` / deviceInfo.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / mqttDevice.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / mqttApiContracts.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / ecoFlowMqttApiManager.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / mqttClient.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / ecoFlowHttpApiManager.ts

| 100% | 100% | 100% | 100% | | 🟢 |

`...` / httpApiContracts.ts

| 100% | 100% | 100% | 100% |

Test suite run success

216 tests passing in 16 suites.

Report generated by 🧪jest coverage report action from 680bc3044d0bc137c4c006402e595df42ff6202c