Skip to content

Commit 381bdb1

Browse files
authored
fix(tekton): e2e nightly failure (#2242)
* fix(tekton): e2e nightly failure * fix: topology nightly test flake
1 parent ce36e98 commit 381bdb1

File tree

3 files changed

+13
-7
lines changed

3 files changed

+13
-7
lines changed

workspaces/tekton/e2e-tests/tests/specs/tekton.spec.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,9 @@ test.describe("Test Tekton plugin", () => {
4242
}) => {
4343
const tekton = new TektonSupportHelper(page);
4444
await tekton.goToBackstageJanusProjectCITab();
45-
await tekton.clickOnExpandRowFromPipelineRunsTable();
45+
await tekton.clickOnExpandRowFromPipelineRunsTable(
46+
"hello-world-pipeline-run",
47+
);
4648
await tekton.openModalEchoHelloWorld();
4749
await tekton.verifyModalOpened();
4850
await tekton.checkPipelineStages(["echo-hello-world", "echo-bye"]);

workspaces/tekton/e2e-tests/tests/support/tekton-support-helper.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,12 +48,11 @@ export class TektonSupportHelper {
4848
return ["NAME", "STATUS", "TASK STATUS", "STARTED", "DURATION"];
4949
}
5050

51-
async clickOnExpandRowFromPipelineRunsTable(): Promise<void> {
51+
async clickOnExpandRowFromPipelineRunsTable(runName: string): Promise<void> {
5252
await this.page
53-
.locator(
54-
'table[aria-labelledby="Pipeline Runs"] button[aria-label="expand row"]',
55-
)
56-
.first()
53+
.getByRole("row")
54+
.filter({ hasText: runName })
55+
.getByRole("button", { name: "expand row" })
5756
.click();
5857
}
5958

workspaces/topology/e2e-tests/tests/specs/topology.spec.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ async function getResourceType(page: Page): Promise<"ingress" | "route"> {
2929
}
3030

3131
test.describe("Test Topology plugin", () => {
32+
const deploymentLocator = `[data-test-id="topology-test"]`;
33+
3234
test.beforeAll(async ({ rhdh }) => {
3335
test.setTimeout(800_000);
3436
const project = rhdh.deploymentConfig.namespace;
@@ -75,7 +77,9 @@ test.describe("Test Topology plugin", () => {
7577
await uiHelper.verifyText(/\d{1,5} (Succeeded|Failed|Cancelled|Running)/);
7678
}).toPass({ intervals: [2_000, 5_000], timeout: 30_000 });
7779
await topology.verifyDeployment("topology-test");
78-
await uiHelper.verifyButtonURL("Open URL", "topology-test-route");
80+
await uiHelper.verifyButtonURL("Open URL", "topology-test-route", {
81+
locator: deploymentLocator,
82+
});
7983
await uiHelper.clickTab("Details");
8084
await uiHelper.verifyText("Status");
8185
await uiHelper.verifyText("Active");
@@ -115,6 +119,7 @@ test.describe("Test Topology plugin", () => {
115119
await uiHelper.verifyButtonURL(
116120
"Edit source code",
117121
"https://github.com/janus-idp/backstage-showcase",
122+
{ locator: deploymentLocator },
118123
);
119124
await uiHelper.clickTab("Resources");
120125
await uiHelper.verifyText("P");

0 commit comments

Comments
 (0)