devfile / api

Kube-native API for cloud development workspaces specification
Apache License 2.0
261 stars 63 forks source link

Improve devfile registry index server testing #1192

Open michael-valdron opened 1 year ago

michael-valdron commented 1 year ago

Which area/kind this issue is related to?

/area registry /area integration-tests

Issue Description

The devfile registry index server has limited testing and test cases across unit tests and integration tests. We need to improve the existing coverage to ensure less area of unknown problems.

michael-valdron commented 1 year ago

Acceptance criteria for missing tests and test cases needed.

feloy commented 1 year ago

Needs some research to find which tests need to be added

github-actions[bot] commented 10 months ago

This issue is stale because it has been open for 90 days with no activity. Remove stale label or comment or this will be closed in 60 days.

Jdubrick commented 10 months ago

@michael-valdron is this issue still being planned for? Going through the waiting queue and noticed this was stale.

michael-valdron commented 10 months ago

@Jdubrick This should still be planned to improve the code coverage of testing under registry-support, which is only 35% atm. In addition, seeing why this is under waiting I think we should create a spike for this issue to investigate what are the test cases we still need to include.

Needs some research to find which tests need to be added

Jdubrick commented 10 months ago

Opened spike issue designed to add more clarity to this issue: https://github.com/devfile/api/issues/1420

thepetk commented 8 months ago

Adding blocked label for the spike issue

maysunfaisal commented 5 months ago

moving back to backlog

michael-valdron commented 5 months ago

moving back to backlog

@maysunfaisal should we move into refinement as it's not sized yet?

github-actions[bot] commented 1 month ago

This issue is stale because it has been open for 90 days with no activity. Remove stale label or comment or this will be closed in 60 days.