cryostatio / cryostatio/cryostat

Set up test harness or unit test framework to exercise Kubernetes integrations

Open
#910 0 comments 0 reactions 0 assignees View on GitHub
question test
Dominant language
Java
Stars
57
Forks
17
Avg merge
15h 39m
Merged PRs (30d)
42

Description

This is difficult to add an automated test for at this point because the "new" (since 3.0) test framework doesn't really have any traditional unit tests. Even the unit tests actually build and run the whole process and hit some actual testcontainers dependency services. In order to add a test for this bug we would need a much more robust discovery test harness that can exercise Kubernetes discovery (or any other discovery that allows for multilevel trees where these node collisions can appear). Or else we would need to set up a traditional unit test that exercises only the DiscoveryNode class, or Kubernetes API discovery class, and uses simple mocks for things like the k8s API client - but then this entails setting up a lot of other mock dependencies like a database, so it becomes quite complicated.

_Originally posted by @andrewazores in https://github.com/cryostatio/cryostat/issues/901#issuecomment-2863530823_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.