Skip to content

Commit 02df2a4

Browse files
committed
fix: enhance PR detection logic to skip Bulk Import tests on PR jobs
1 parent d90308c commit 02df2a4

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

e2e-tests/playwright/e2e/plugins/bulk-import.spec.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,13 @@ import {
99
UPDATED_CATALOG_INFO_YAML,
1010
} from "../../support/test-data/bulk-import";
1111

12+
const isPrJob =
13+
process.env.JOB_NAME?.startsWith("pull-ci-") ||
14+
process.env.JOB_TYPE === "presubmit" ||
15+
Boolean(process.env.PULL_NUMBER);
16+
17+
test.skip(isPrJob, "Skipping Bulk Import tests on PR (presubmit) jobs");
18+
1219
// Pre-req : plugin-bulk-import & plugin-bulk-import-backend-dynamic
1320
test.describe.serial("Bulk Import plugin", () => {
1421
test.skip(() => process.env.JOB_NAME?.includes("osd-gcp")); // skipping due to RHIDP-5704 on OSD Env
@@ -283,7 +290,7 @@ spec:
283290

284291
test.describe
285292
.serial("Bulk Import - Verify existing repo are displayed in bulk import Added repositories", () => {
286-
test.skip(() => process.env.JOB_NAME.includes("osd-gcp")); // skipping due to RHIDP-5704 on OSD Env
293+
test.skip(() => process.env.JOB_NAME?.includes("osd-gcp")); // skipping due to RHIDP-5704 on OSD Env
287294
let page: Page;
288295
let uiHelper: UIhelper;
289296
let common: Common;
@@ -341,7 +348,7 @@ test.describe
341348

342349
test.describe
343350
.serial("Bulk Import - Ensure users without bulk import permissions cannot access the bulk import plugin", () => {
344-
test.skip(() => process.env.JOB_NAME.includes("osd-gcp")); // skipping due to RHIDP-5704 on OSD Env
351+
test.skip(() => process.env.JOB_NAME?.includes("osd-gcp")); // skipping due to RHIDP-5704 on OSD Env
345352
let page: Page;
346353
let uiHelper: UIhelper;
347354
let common: Common;

0 commit comments

Comments
 (0)