Skip to content

Commit b270441

Browse files
Skip recovery codes require an auth app if we cannot remove them. (#1019)
* Skip recovery codes require an auth app if we cannot remove them. * Fix placement of test.skip
1 parent da79567 commit b270441

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

test/e2e/tests/mfa.spec.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import {
55
ACCTS_HUB_URL,
66
ACCTS_OIDC_EMAIL,
77
ACCTS_OIDC_PWORD,
8+
KEYCLOAK_ADMIN_CLIENT_SECRET,
89
PLAYWRIGHT_TAG_E2E_SUITE,
910
TIMEOUT_30_SECONDS,
1011
} from '../const/constants';
@@ -34,6 +35,9 @@ import { makeTotpCode } from '../utils/totp';
3435
test.describe('multi-factor authentication', {
3536
tag: [PLAYWRIGHT_TAG_E2E_SUITE],
3637
}, () => {
38+
39+
test.skip((!KEYCLOAK_ADMIN_CLIENT_SECRET) || KEYCLOAK_ADMIN_CLIENT_SECRET === 'undefined', 'KEYCLOAK_ADMIN_CLIENT_SECRET is needed for this test to run.');
40+
3741
// Manage MFA is gated behind the SHOW_MFA feature flag; enable it per-browser so the
3842
// /manage-mfa route is registered before any test navigates to it.
3943
test.beforeEach(async ({ page }) => {

0 commit comments

Comments
 (0)