Skip to content

Commit 6ff4679

Browse files
authored
Merge pull request #15753 from yonasberhe23/fix_logging_chart_test
Automation: fix logging chart test
2 parents f3711d2 + edfd4e4 commit 6ff4679

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

cypress/e2e/po/components/kubectl.po.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export default class Kubectl extends ComponentPo {
2929
}
3030

3131
waitForTerminalStatus(status: 'Connected' | 'Disconnected', options?: GetOptions) {
32-
this.self().find('.active .status').contains(status, options).should('be.visible');
32+
this.self().contains('.active .status', status, options);
3333
}
3434

3535
/**

cypress/e2e/tests/pages/charts/logging.spec.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import PromptRemove from '@/cypress/e2e/po/prompts/promptRemove.po';
1010
import ChartInstalledAppsListPagePo from '@/cypress/e2e/po/pages/chart-installed-apps.po';
1111
import { MEDIUM_TIMEOUT_OPT } from '@/cypress/support/utils/timeouts';
1212
import { CLUSTER_APPS_BASE_URL } from '@/cypress/support/utils/api-endpoints';
13+
import CardPo from '~/cypress/e2e/po/components/card.po';
1314

1415
describe('Logging Chart', { testIsolation: 'off', tags: ['@charts', '@adminUser'] }, () => {
1516
const kubectl = new Kubectl();
@@ -115,6 +116,7 @@ describe('Logging Chart', { testIsolation: 'off', tags: ['@charts', '@adminUser'
115116
installedAppsPage.goTo();
116117
installedAppsPage.waitForPage();
117118
cy.wait('@getCharts', MEDIUM_TIMEOUT_OPT).its('response.statusCode').should('eq', 200);
119+
installedAppsPage.appsList().checkVisible(MEDIUM_TIMEOUT_OPT);
118120
installedAppsPage.appsList().sortableTable().checkLoadingIndicatorNotVisible();
119121
installedAppsPage.appsList().sortableTable().noRowsShouldNotExist();
120122
installedAppsPage.appsList().resourceTableDetails(chartApp, 1).should('exist');
@@ -134,6 +136,10 @@ describe('Logging Chart', { testIsolation: 'off', tags: ['@charts', '@adminUser'
134136
promptRemove.checkbox().set();
135137
promptRemove.checkbox().isChecked();
136138
promptRemove.remove();
139+
140+
const card = new CardPo();
141+
142+
card.checkNotExists(MEDIUM_TIMEOUT_OPT);
137143
cy.wait('@chartUninstall').its('response.statusCode').should('eq', 201);
138144
cy.wait('@crdUninstall').its('response.statusCode').should('eq', 201);
139145

@@ -145,6 +151,7 @@ describe('Logging Chart', { testIsolation: 'off', tags: ['@charts', '@adminUser'
145151
installedAppsPage.goTo();
146152
installedAppsPage.waitForPage();
147153
cy.wait('@getCharts', MEDIUM_TIMEOUT_OPT).its('response.statusCode').should('eq', 200);
154+
installedAppsPage.appsList().checkVisible(MEDIUM_TIMEOUT_OPT);
148155
installedAppsPage.appsList().sortableTable().checkLoadingIndicatorNotVisible();
149156
installedAppsPage.appsList().sortableTable().noRowsShouldNotExist();
150157
installedAppsPage.appsList().sortableTable().rowNames('.col-link-detail', MEDIUM_TIMEOUT_OPT)

0 commit comments

Comments
 (0)