Skip to content

Commit 31d848a

Browse files
committed
chore: fix saving playwright artifacts
1 parent 8506d34 commit 31d848a

5 files changed

Lines changed: 16 additions & 7 deletions

File tree

.github/actions/setup/action.yml

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,12 +67,20 @@ runs:
6767
run: pnpm exec nx affected -t build lint test typecheck e2e-ci
6868
shell: bash
6969

70+
- name: List files
71+
if: ${{ !cancelled() }}
72+
shell: bash
73+
run: ls -R e2e/oidc-suites/.playwright && ls -R e2e/protect-suites/.playwright
74+
7075
- name: Upload Playwright report
7176
uses: actions/upload-artifact@v5
7277
if: ${{ !cancelled() }}
7378
with:
74-
name: playwright-report
79+
name: playwright-report-${{ github.run_id }}-${{ github.run_attempt }}
7580
path: |
76-
./e2e/*/.playwright/**
77-
./e2e/**/.playwright/**
81+
e2e/davinci-suites/.playwright/**
82+
e2e/journey-suites/.playwright/**
83+
e2e/oidc-suites/.playwright/**
84+
e2e/protect-suites/.playwright/**
7885
retention-days: 30
86+
include-hidden-files: true

.github/workflows/publish.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,7 @@ jobs:
129129

130130
- name: Publish packages snapshot with npm_tag
131131
id: npmpublish
132-
# run: pnpm publish -r --tag ${{ inputs.npm_tag }} --no-git-checks --access ${{ inputs.npm_access }}
133-
run: echo "Skipping npm publish"
132+
run: pnpm publish -r --tag ${{ inputs.npm_tag }} --no-git-checks --access ${{ inputs.npm_access }}
134133

135134
- name: Format published packages for Slack
136135
if: steps.npmpublish.outcome == 'success'

e2e/oidc-suites/playwright.config.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ export default defineConfig({
1212
testDir: './src',
1313
reporter: process.env.CI ? 'github' : 'list',
1414
timeout: 30000,
15+
retries: process.env.CI ? 1 : 0,
16+
workers: process.env.CI ? 1 : undefined,
1517
use: {
1618
baseURL,
1719
headless: true,

e2e/oidc-suites/src/login.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ test.describe('PingAM login and get token tests', () => {
2020
const { clickWithRedirect, navigate } = asyncEvents(page);
2121
await navigate('/ping-am/');
2222

23-
await clickWithRedirect('Login (Background)', '**/am/XUI/**');
23+
await clickWithRedirect('Login (Background-zzz)', '**/am/XUI/**');
2424

2525
await page.getByLabel('User Name').fill(pingAmUsername);
2626
await page.getByRole('textbox', { name: 'Password' }).fill(pingAmPassword);

e2e/protect-suites/src/protect-native.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ test.describe('Test basic login flow with Ping Protect', () => {
3030

3131
await page.getByPlaceholder('Username').fill(username);
3232
await page.getByPlaceholder('Password').fill(password);
33-
await page.getByRole('button', { name: 'Sign In' }).click();
33+
await page.getByRole('button', { name: 'Sign In - zzz' }).click();
3434

3535
await expect(page.getByText('Protect evaluating')).toBeVisible();
3636

0 commit comments

Comments
 (0)