Kuadrant / Kuadrant/testsuite

AllNamespaces API Key test is now irrelevant

Open
#579 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Python
Stars
7
Forks
29
Avg merge
1d 9h
Merged PRs (30d)
7

Description

**Summary**

Kuadrant creates all AuthConfigs in the same namespace where Authorino is running. Therefore we changed default value for `spec.authentication.apiKey.allNamespaces` to `True`. This makes `test_all_namespaces_api_key.py` irrelevant.

Refactor/remove this test and implement one that creates a secret in the same namespace as Authorino.

authorino-standalone is not affected, but needs to be think of when implementing this issue

Contributor guide

Open the contributing guide

Research direction

Start with test_all_namespaces_api_key.py and inspect how the tests create namespaces, AuthConfigs, and secrets. Replace the irrelevant coverage with a case that creates the secret in Authorino's namespace, while checking the authorino-standalone setup for any required distinction. Done means the relevant tests pass and the new case verifies same-namespace API key behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing-qa
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.