Skip to content

Commit

Permalink
Testing registry against LightKube
Browse files Browse the repository at this point in the history
  • Loading branch information
juditnovak committed Jun 16, 2023
1 parent cbf7d15 commit 7bc7d7b
Show file tree
Hide file tree
Showing 2 changed files with 53 additions and 8 deletions.
15 changes: 13 additions & 2 deletions tests/integration/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,19 @@ def lightkubeinterface(defs_with_kubeconf):
interface.client.delete(Namespace, name=ns)


def _clearnup_registry(registry):
[registry.delete(account.id) for account in registry.all()]


@pytest.fixture
def registry(kubeinterface):
def kube_registry(kubeinterface):
registry = K8sServiceAccountRegistry(kubeinterface)
yield registry
[registry.delete(account.id) for account in registry.all()]
_clearnup_registry(registry)


@pytest.fixture
def lightkube_registry(lightkubeinterface):
registry = K8sServiceAccountRegistry(lightkubeinterface)
yield registry
_clearnup_registry(registry)
46 changes: 40 additions & 6 deletions tests/integration/test_registry.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,27 @@


@pytest.mark.usefixtures("integration_test")
@pytest.mark.parametrize(
"kubeinterface_name, kuberegistry_name",
[
("kubeinterface", "kube_registry"),
pytest.param(
"lightkubeinterface",
"lightkube_registry",
marks=pytest.mark.xfail(
reason="https://warthogs.atlassian.net/browse/DPE-2131"
),
),
],
)
@pytest.mark.parametrize(
"namespace, user",
[("default-namespace", "spark"), ("spark-namespace", "spark-user")],
[("default-a-namespace", "spark"), ("spark-a-namespace", "spark-user")],
)
def test_registry_io(namespace, user, kubeinterface, registry):
def test_registry_io(kubeinterface_name, kuberegistry_name, namespace, user, request):
kubeinterface = request.getfixturevalue(kubeinterface_name)
registry = request.getfixturevalue(kuberegistry_name)

kubeinterface.create(resource_type="namespace", resource_name=namespace)

service_account = ServiceAccount(
Expand All @@ -23,7 +39,7 @@ def test_registry_io(namespace, user, kubeinterface, registry):

registry.create(service_account)

assert len(registry.all()) == 1
assert len(registry.all(namespace=namespace)) == 1

retrieved_service_account = registry.get(service_account.id)

Expand All @@ -37,11 +53,29 @@ def test_registry_io(namespace, user, kubeinterface, registry):


@pytest.mark.usefixtures("integration_test")
@pytest.mark.parametrize(
"kubeinterface_name, kuberegistry_name",
[
("kubeinterface", "kube_registry"),
pytest.param(
"lightkubeinterface",
"lightkube_registry",
marks=pytest.mark.xfail(
reason="https://warthogs.atlassian.net/browse/DPE-2131"
),
),
],
)
@pytest.mark.parametrize(
"namespace, username",
[("default-namespace", "spark"), ("spark-namespace", "spark-user")],
[("default-b-namespace", "spark"), ("spark-b-namespace", "spark-user")],
)
def test_registry_change_primary_account(namespace, username, kubeinterface, registry):
def test_registry_change_primary_account(
kubeinterface_name, kuberegistry_name, namespace, username, request
):
kubeinterface = request.getfixturevalue(kubeinterface_name)
registry = request.getfixturevalue(kuberegistry_name)

kubeinterface.create(resource_type="namespace", resource_name=namespace)

sa1 = ServiceAccount(
Expand Down Expand Up @@ -69,7 +103,7 @@ def test_registry_change_primary_account(namespace, username, kubeinterface, reg


@pytest.mark.usefixtures("integration_test")
def test_merge_configurations(registry):
def test_merge_configurations():
k1 = str(uuid.uuid4())
v11 = str(uuid.uuid4())
v12 = str(uuid.uuid4())
Expand Down

0 comments on commit 7bc7d7b

Please sign in to comment.