You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While working on PR #934, Codecov reported a lack of tests for the changes made to the PolicyServerReconciler and related changes, which added the required configuration in the policy server deployment. As a result, code coverage in the codebase has dropped.
The telemetry configuration is stored in the reconciler struct, which poses a challenge for writing proper tests for these changes. This issue cannot be easily addressed due to the fact that reconciler instances are created during test initialization, and it is not possible to have multiple test suites. Therefore, we need to find another way to enable configuration or recreation of the reconciler before certain tests.
Acceptance Criteria
Write integration tests to cover all telemetry configuration possibilities.
The text was updated successfully, but these errors were encountered:
While working on PR #934, Codecov reported a lack of tests for the changes made to the PolicyServerReconciler and related changes, which added the required configuration in the policy server deployment. As a result, code coverage in the codebase has dropped.
The telemetry configuration is stored in the reconciler struct, which poses a challenge for writing proper tests for these changes. This issue cannot be easily addressed due to the fact that reconciler instances are created during test initialization, and it is not possible to have multiple test suites. Therefore, we need to find another way to enable configuration or recreation of the reconciler before certain tests.
Acceptance Criteria
The text was updated successfully, but these errors were encountered: