Skip to content
This repository was archived by the owner on Oct 14, 2020. It is now read-only.

Commit 68780ac

Browse files
committed
Minor refactoring
1 parent ec192fb commit 68780ac

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

tests/integration/generic/read-only-hook.test.js

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,17 +25,21 @@ test(
2525

2626
let podName;
2727
await k8sApi.listNamespacedPod(NAMESPACE, 'true').then((res) => {
28-
let podArray = res.body.items.filter(containsPod);
29-
podName = podArray.pop().metadata.name;
28+
let podArray = res.body.items.filter((containsPod));
29+
if (podArray.length === 0) {
30+
throw new Error(`Did not find Pod for "${WEBHOOK}" Hook`);
31+
}
32+
33+
podName = podArray[0].metadata.name;
3034
});
3135

3236
const containerName = WEBHOOK;
3337

34-
let params = {
35-
k8sApi: k8sApi,
36-
podName: podName,
38+
const params = {
39+
k8sApi,
40+
podName,
3741
namespace: NAMESPACE,
38-
containerName: containerName
42+
containerName
3943
}
4044
const result = await delayedRepeat(isHookTriggered, params, 1000, 10);
4145

0 commit comments

Comments
 (0)