Skip to content
This repository has been archived by the owner on Jun 10, 2021. It is now read-only.

Refactor mock nodes and unit tests #64

Open
mm318 opened this issue Jan 9, 2020 · 0 comments
Open

Refactor mock nodes and unit tests #64

mm318 opened this issue Jan 9, 2020 · 0 comments
Labels
enhancement New feature or request technical debt Code that could use some cleanup or refactoring, not necessarily a new feature

Comments

@mm318
Copy link
Member

mm318 commented Jan 9, 2020

There is currently quite a bit of duplicate logic in testing the publishing and receiving of MetricsMessages under test_linux_cpu_measurement.cpp, test_linux_memory_measurement.cpp, and test_linux_process_cpu_measurement_node.cpp. The common code should be refactored out into test_utilities.hpp/test_utilities.cpp.
@mm318 mm318 added the enhancement New feature or request label Jan 9, 2020
@emersonknapp emersonknapp added the technical debt Code that could use some cleanup or refactoring, not necessarily a new feature label Mar 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request technical debt Code that could use some cleanup or refactoring, not necessarily a new feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@emersonknapp @mm318