Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

[DPM] UT enhancement for core logic #288

Closed
xieus opened this issue Jul 6, 2020 · 3 comments · Fixed by #389, #394 or #398
Closed

[DPM] UT enhancement for core logic #288

xieus opened this issue Jul 6, 2020 · 3 comments · Fixed by #389, #394 or #398
Assignees
@kevin-zhonghao @haboy52581
Labels
P0 Priority 0 unit test Unit test
Milestone
Version 0.7.2020....

Comments

@xieus
Copy link
Contributor

xieus commented Jul 6, 2020

Context

The DPM UTs currently cover the GSManager only. We will also need to cover the controller layer and others.

Please leverage the code coverage report to cover more core logic: https://codecov.io/gh/futurewei-cloud/alcor/tree/f0bcc66f4256ffb75859fc91019bf56bbd68629d/services/data_plane_manager/src/main/java/com/futurewei/alcor/dataplane
@xieus xieus added the unit test Unit test label Jul 6, 2020
@xieus xieus added this to the Version 0.7.2020.07.30 milestone Jul 6, 2020
@xieus xieus mentioned this issue Jul 6, 2020
Merged
@haboy52581
Copy link
Contributor

added in 1f5ffa3, would add more

@xieus
Copy link
Contributor Author

xieus commented Oct 13, 2020

@kevin-zhonghao for awareness.

This was linked to pull requests Oct 16, 2020
[DPM] Refactor UTs #389
Merged
[Data Plane Manager] UT Test Automation #394
Merged
[Data Plane Mgr] UT Test Automation - Part II #398
Merged
@xieus xieus added the P0 Priority 0 label Oct 16, 2020
@xieus xieus mentioned this issue Oct 16, 2020
Closed
@haboy52581
Copy link
Contributor

per the context "The DPM UTs currently cover the GSManager only. We will also need to cover the controller layer and others.
" , @kevin-zhonghao please add this part for controllers

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kevin-zhonghao kevin-zhonghao

@haboy52581 haboy52581

Labels
P0 Priority 0 unit test Unit test
Projects
None yet
Milestone
Version 0.7.2020.07.30
3 participants
@xieus @kevin-zhonghao @haboy52581