Skip to content

Automation: Stabilize OIDC provider tests #17032

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
yonasberhe23 merged 1 commit into from
Mar 28, 2026
+37 −25
Merged

Automation: Stabilize OIDC provider tests #17032

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cypress/e2e/po/pages/users-and-auth/oidc-client.po.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export default class OidcClientsPagePo extends PagePo {
const sideNav = new ProductNavPo();

BurgerMenuPo.burgerMenuNavToMenubyLabel('Users & Authentication');
sideNav.navToSideMenuGroupByLabel('Client Application');
sideNav.navToSideMenuEntryByLabel('OIDC Apps');
}

createOidcClient() {
Expand Down
60 changes: 36 additions & 24 deletions cypress/e2e/tests/pages/users-and-auth/oidcProvider.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,10 @@ import PromptRemove from '@/cypress/e2e/po/prompts/promptRemove.po';
import { promptModal } from '@/cypress/e2e/po/prompts/shared/modalInstances.po';
import OIDCClientDetailPo from '@/cypress/e2e/po/detail/management.cattle.io.oidcclient.po';
import { MEDIUM_TIMEOUT_OPT } from '@/cypress/support/utils/timeouts';
import HomePagePo from '@/cypress/e2e/po/pages/home.po';
import { qase } from '@/cypress/support/qase';

describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalSettings', '@adminUser'] }, () => {
describe('Rancher as an OIDC Provider', { tags: ['@globalSettings', '@adminUser'] }, () => {
const OIDC_CREATE_DATA = {
APP_NAME: 'some-app-name',
APP_DESC: 'some-app-desc',
Expand Down Expand Up @@ -35,15 +37,16 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS
const oidcClientCreatePage = new OidcClientCreateEditPo(clusterId);
const oidcClientEditPage = new OidcClientCreateEditPo(clusterId, OIDC_CREATE_DATA.APP_NAME, true);

before(() => {
beforeEach(() => {
cy.login();
});

it('should be able to create an OIDC client application', () => {
qase(9761, it('should be able to create an OIDC client application', () => {
cy.intercept('POST', `/v1/management.cattle.io.oidcclients`).as('createRequest');

oidcClientsPage.goTo();
oidcClientsPage.waitForPage();
HomePagePo.goTo();
OidcClientsPagePo.navTo(); // Do not use goTo() here — deep link (going directly to the list page) makes the full-secret copy row behavior flaky.
oidcClientsPage.waitForUrlPathWithoutContext();

// check title and list view
oidcClientsPage.list().title().should('contain', 'OIDC Apps');
Expand All @@ -57,7 +60,7 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS

// let's create an oidc client
oidcClientsPage.createOidcClient();
oidcClientCreatePage.waitForPage();
oidcClientCreatePage.waitForUrlPathWithoutContext();

oidcClientCreatePage.nameNsDescription().name().set(OIDC_CREATE_DATA.APP_NAME);
oidcClientCreatePage.nameNsDescription().description().set(OIDC_CREATE_DATA.APP_DESC);
Expand All @@ -79,16 +82,16 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS
expect(response?.body.spec.tokenExpirationSeconds).to.equal(OIDC_CREATE_DATA.TOKEN_EXP);
});

oidcClientDetailPage.waitForPage();
oidcClientDetailPage.waitForUrlPathWithoutContext();

oidcClientDetailPage.clientID().exists();
oidcClientDetailPage.clientFullSecretCopy(0).exists();

oidcClientDetailPage.clientID().copyToClipboard();
oidcClientDetailPage.clientFullSecretCopy(0).copyToClipboard();
});
}));

it('should be able to edit an OIDC client application', () => {
qase(9762, it('should be able to edit an OIDC client application', () => {
cy.intercept('PUT', `/v1/management.cattle.io.oidcclients/${ OIDC_CREATE_DATA.APP_NAME }`).as('editRequest');

OidcClientsPagePo.goTo();
Expand Down Expand Up @@ -116,14 +119,18 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS
expect(response?.body.spec.refreshTokenExpirationSeconds).to.equal(OIDC_EDIT_DATA.REF_TOKEN_EXP);
expect(response?.body.spec.tokenExpirationSeconds).to.equal(OIDC_EDIT_DATA.TOKEN_EXP);
});
});
}));

it('should be able to add a new secret for an OIDC provider', () => {
qase(9763, it('should be able to add a new secret for an OIDC provider', () => {
cy.intercept('PUT', `/v1/management.cattle.io.oidcclients/${ OIDC_CREATE_DATA.APP_NAME }`).as('addNewSecret');

oidcClientDetailPage.goTo();
oidcClientDetailPage.waitForPage();

HomePagePo.goTo();
OidcClientsPagePo.navTo(); // Do not use goTo() here — deep link (going directly to the detail page) makes the full-secret copy row behavior flaky.
oidcClientsPage.waitForUrlPathWithoutContext();
oidcClientsPage.list().resourceTable().sortableTable().checkLoadingIndicatorNotVisible();
oidcClientsPage.list().resourceTable().sortableTable().noRowsShouldNotExist();
oidcClientsPage.list().resourceTable().goToDetailsPage(OIDC_CREATE_DATA.APP_NAME);
oidcClientDetailPage.waitForUrlPathWithoutContext();
oidcClientDetailPage.addNewSecretBtnClick();

// check data from network request
Expand All @@ -133,20 +140,25 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS
});

// Wait for the page to refresh and show the new secret with proper timeout
oidcClientDetailPage.waitForPage();
oidcClientDetailPage.waitForUrlPathWithoutContext();

// Wait for the secret element to appear before trying to interact with it
oidcClientDetailPage.clientFullSecretCopy(1).checkVisible();

oidcClientDetailPage.clientFullSecretCopy(1).exists();
oidcClientDetailPage.clientFullSecretCopy(1).copyToClipboard();
});
}));

it('should be able to regenerate a secret for an OIDC provider', () => {
qase(9764, it('should be able to regenerate a secret for an OIDC provider', () => {
cy.intercept('PUT', `/v1/management.cattle.io.oidcclients/${ OIDC_CREATE_DATA.APP_NAME }`).as('regenSecret');

oidcClientDetailPage.goTo();
oidcClientDetailPage.waitForPage();
HomePagePo.goTo();
OidcClientsPagePo.navTo(); // Do not use goTo() here — deep link (going directly to the detail page) makes the full-secret copy row behavior flaky.
oidcClientsPage.waitForUrlPathWithoutContext();
oidcClientsPage.list().resourceTable().sortableTable().checkLoadingIndicatorNotVisible();
oidcClientsPage.list().resourceTable().sortableTable().noRowsShouldNotExist();
oidcClientsPage.list().resourceTable().goToDetailsPage(OIDC_CREATE_DATA.APP_NAME);
oidcClientDetailPage.waitForUrlPathWithoutContext();

// let's regen the secret we've added the step before
oidcClientDetailPage.secretCardActionMenuToggle(1);
Expand All @@ -163,9 +175,9 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS

oidcClientDetailPage.clientFullSecretCopy(1).exists();
oidcClientDetailPage.clientFullSecretCopy(1).copyToClipboard();
});
}));

it('should be able to delete a secret for an OIDC provider', () => {
qase(9765, it('should be able to delete a secret for an OIDC provider', () => {
cy.intercept('PUT', `/v1/management.cattle.io.oidcclients/${ OIDC_CREATE_DATA.APP_NAME }`).as('deleteSecret');

oidcClientDetailPage.goTo();
Expand All @@ -186,9 +198,9 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS

// check that the card doesn't exist anymore
cy.get('[data-testid="item-card-client-secret-2"]').should('not.exist');
});
}));

it('should be able to delete an OIDC client application', () => {
qase(9766, it('should be able to delete an OIDC client application', () => {
cy.intercept('DELETE', `/v1/management.cattle.io.oidcclients/${ OIDC_CREATE_DATA.APP_NAME }`).as('deleteRequest');

OidcClientsPagePo.goTo();
Expand All @@ -204,5 +216,5 @@ describe('Rancher as an OIDC Provider', { testIsolation: 'off', tags: ['@globalS

oidcClientsPage.waitForPage();
cy.contains(OIDC_CREATE_DATA.APP_NAME).should('not.exist');
});
}));
});
Loading