Skip to content

OCPBUGS-82506, OCPBUGS-82507: Re-enable OLM Cypress tests disabled for createRoot #16531

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
openshift-merge-bot merged 1 commit into from
Jun 4, 2026
+15 −9
Merged

OCPBUGS-82506, OCPBUGS-82507: Re-enable OLM Cypress tests disabled for createRoot #16531

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
12 changes: 8 additions & 4 deletions frontend/packages/operator-lifecycle-manager/integration-tests/tests/operator-hub.cy.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import { checkErrors, testName } from '@console/cypress-integration-tests/support';

// Disabled due to createRoot concurrent rendering failures (OCPBUGS-82506)
xdescribe('Interacting with Operators', () => {
describe('Interacting with Operators', () => {
before(() => {
cy.login();
cy.createProjectWithCLI(testName);
Expand Down Expand Up @@ -40,11 +39,16 @@ xdescribe('Interacting with Operators', () => {
cy.log('more than one tile should be present');
cy.get('.co-catalog-tile').its('length').should('be.gt', 0);
cy.log('the first tile title text for Certified should not be the same as Community');
// Wait for catalog to re-render with new filter - under React 18 concurrent rendering,
// tile updates are batched and happen asynchronously. Use .should() callback to retry
// until the tile content actually changes.
cy.get('.co-catalog-tile')
.first()
.find('.catalog-tile-pf-title')
.invoke('text')
.should('not.equal', origCatalogTitleTxt);
.should(($title) => {
const newTitleTxt = $title.text();
expect(newTitleTxt).not.to.equal(origCatalogTitleTxt);
});
});

cy.log('filters Operators by name');
Expand Down
3 changes: 1 addition & 2 deletions ...packages/operator-lifecycle-manager/integration-tests/tests/operator-install-global.cy.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,7 @@ const cleanupOperatorResources = () => {
);
};

// Disabled due to createRoot concurrent rendering failures (OCPBUGS-82507)
xdescribe(`Globally installing "${testOperator.name}" operator in ${GlobalInstalledNamespace}`, () => {
describe(`Globally installing "${testOperator.name}" operator in ${GlobalInstalledNamespace}`, () => {
before(() => {
cy.login();
cleanupOperatorResources();
Expand Down
9 changes: 6 additions & 3 deletions frontend/packages/operator-lifecycle-manager/integration-tests/views/operator.view.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,9 @@ export const operator = {
*/
if (installToNamespace !== GlobalInstalledNamespace) {
cy.log('configure Operator install for single namespace');
// Wait for radio button to be visible before checking to avoid race conditions
cy.byTestID('A specific namespace on the cluster-radio-input', { timeout: 30000 })
// Under React 18 concurrent rendering, install mode radios mount after async data loads.
// Increase timeout to handle deferred rendering.
cy.byTestID('A specific namespace on the cluster-radio-input', { timeout: 60000 })
.should('be.visible')
.check();
if (useOperatorRecommendedNamespace) {
Expand All @@ -60,7 +61,9 @@ export const operator = {
});
}
} else {
cy.byTestID('All namespaces on the cluster-radio-input', { timeout: 30000 }).should(
// Under React 18 concurrent rendering, install mode radios mount after async data loads.
// Increase timeout to handle deferred rendering.
cy.byTestID('All namespaces on the cluster-radio-input', { timeout: 60000 }).should(
'be.checked',
);
}
Expand Down