[TESTS] Low test covarage for Endpoints handler
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
kuryr-kubernetes |
New
|
Medium
|
Agha Saad Fraz |
Bug Description
The testing coverage for the EndpointsHandler[1] is below the ideal. New unit tests[2] need to be created to cover possible scenarios.
The following scenarios could be added:
- on_present method: check the event is skipped (meaning no crd_spec is created or updated) when the Endpoint has no Pods defined or it's a headless Service.
- on_present method: check the event is skipped (meaning no attempt to updated crd) when the Endpoints has annotations defined, as the
Endpoints annotations are going to be converted to CRD.
[1] https:/
[2] https:/
Example of Endpoints with annotation http://
Changed in kuryr-kubernetes: | |
importance: | Undecided → Medium |
tags: | added: low-hanging-fruit |
summary: |
- Low test covarage for Service and Endpoints handler + [TESTS] Low test covarage for Service and Endpoints handler |
Changed in kuryr-kubernetes: | |
assignee: | nobody → Agha Saad Fraz (aghasaad04) |
summary: |
- [TESTS] Low test covarage for Service and Endpoints handler + [TESTS] Low test covarage for Endpoints handler |
description: | updated |
description: | updated |
description: | updated |