Skip to content

Commit

Permalink
test: ensure all resources are deleted by deleteKubernetesDeploy
Browse files Browse the repository at this point in the history
A metadata `ConfigMap` describing what was last deployed must be deleted too.
  • Loading branch information
vvagaytsev committed Sep 6, 2023
1 parent 6263ab9 commit adf847a
Showing 1 changed file with 28 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,9 @@ import { ModuleConfig } from "../../../../../../src/config/module"
import { BaseResource, KubernetesResource } from "../../../../../../src/plugins/kubernetes/types"
import { DeleteDeployTask } from "../../../../../../src/tasks/delete-deploy"
import {
kubernetesDeploy,
deleteKubernetesDeploy,
getKubernetesDeployStatus,
kubernetesDeploy,
} from "../../../../../../src/plugins/kubernetes/kubernetes-type/handlers"
import { buildHelmModules } from "../helm/common"
import { gardenAnnotationKey } from "../../../../../../src/util/string"
Expand Down Expand Up @@ -482,5 +483,31 @@ describe("kubernetes-type handlers", () => {
expect(await getDeployedResource(ctx, ctx.provider, ns1Manifest!, log), "ns1resource").to.exist
expect(await getDeployedResource(ctx, ctx.provider, ns2Manifest!, log), "ns2resource").to.not.exist
})

it("deletes all resources including a metadata ConfigMap describing what was last deployed", async () => {
const { resolvedAction, deployParams, manifests } = await prepareActionDeployParamsWithManifests(
"module-simple",
{}
)

const status = await kubernetesDeploy(deployParams)
expect(status.state).to.eql("ready")

const namespace = await getActionNamespace({
ctx,
log,
action: resolvedAction,
provider: ctx.provider,
skipCreate: true,
})

const deleteDeployParams = { ctx, action: resolvedAction, log: actionLog }
await deleteKubernetesDeploy(deleteDeployParams)

for (const manifest of manifests) {
const metadataResource = await api.readBySpecOrNull({ log, namespace, manifest })
expect(metadataResource).to.be.null
}
})
})
})

0 comments on commit adf847a

Please sign in to comment.