Skip to content

Commit 555d14e

Browse files
authored
fix: update tier dropdown text assertions in LLMD mock test (#6044)
The test was expecting 'All resource tiers', 'No resource tiers', and 'Specific resource tiers' but the component uses 'All tiers', 'No tiers', and 'Specific tiers'. Updated both the page object helper and test assertions to match the actual component labels.
1 parent 787dc31 commit 555d14e

2 files changed

Lines changed: 9 additions & 11 deletions

File tree

packages/cypress/cypress/pages/modelsAsAService.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -259,9 +259,7 @@ class MaaSWizardField {
259259
return cy.findByTestId('maas/save-as-maas-checkbox-tier-dropdown');
260260
}
261261

262-
selectMaaSTierOption(
263-
option: 'All resource tiers' | 'No resource tiers' | 'Specific resource tiers',
264-
) {
262+
selectMaaSTierOption(option: 'All tiers' | 'No tiers' | 'Specific tiers') {
265263
this.findMaaSTierDropdown().click();
266264
return cy.findByRole('option', { name: option }).click();
267265
}

packages/cypress/cypress/tests/mocked/modelServing/modelServingLlmd.cy.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -660,16 +660,16 @@ describe('Model Serving LLMD', () => {
660660
maasWizardField.findSaveAsMaaSCheckbox().click();
661661
maasWizardField.findSaveAsMaaSCheckbox().should('be.checked');
662662

663-
// Verify default selection is "All resource tiers"
664-
maasWizardField.findMaaSTierDropdown().should('contain.text', 'All resource tiers');
663+
// Verify default selection is "All tiers"
664+
maasWizardField.findMaaSTierDropdown().should('contain.text', 'All tiers');
665665

666-
// Switch to "No resource tiers"
667-
maasWizardField.selectMaaSTierOption('No resource tiers');
668-
maasWizardField.findMaaSTierDropdown().should('contain.text', 'No resource tiers');
666+
// Switch to "No tiers"
667+
maasWizardField.selectMaaSTierOption('No tiers');
668+
maasWizardField.findMaaSTierDropdown().should('contain.text', 'No tiers');
669669

670-
// Switch to "Specific resource tiers" - Next button should be disabled until input is provided
671-
maasWizardField.selectMaaSTierOption('Specific resource tiers');
672-
maasWizardField.findMaaSTierDropdown().should('contain.text', 'Specific resource tiers');
670+
// Switch to "Specific tiers" - Next button should be disabled until input is provided
671+
maasWizardField.selectMaaSTierOption('Specific tiers');
672+
maasWizardField.findMaaSTierDropdown().should('contain.text', 'Specific tiers');
673673
maasWizardField.findMaaSTierNamesInput().should('be.visible');
674674
modelServingWizard.findNextButton().should('be.disabled');
675675

0 commit comments

Comments
 (0)