Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test coverage for data storers and MQTT handler #263

Open
oscgonfer opened this issue Oct 6, 2023 · 0 comments
Open

Test coverage for data storers and MQTT handler #263

oscgonfer opened this issue Oct 6, 2023 · 0 comments
Milestone
Data Ingestion

Comments

@oscgonfer
Copy link
Contributor

oscgonfer commented Oct 6, 2023 •
edited

BTW @oscgonfer when this refactor is done and merged, I'd like to start thinking about refactoring and improving the test coverage for the data storers and MQTT handler - they're extremely "enmarañados" at the moment and with quite flaky coverage, which is making me very nervous about changing anything to do with them, and they're a pretty key part of the system. We can discuss next time we see each other!

Originally posted by @timcowlishaw in #246 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Data Ingestion
Development

No branches or pull requests
1 participant
@oscgonfer