fix: resolve skipped cypress tests (#11722)

* fix: update cypress tests

* fix: cypress

* fix: font loading issue for production

* fix: cypress tests

* fix: alert cypress

* fix: add enter in operation

* fix: alert cypress

* remove un-neccessary check for add dbt ingestion title

---------

Co-authored-by: Shailesh Parmar <shailesh.parmar.webdev@gmail.com>
This commit is contained in:
karanh37 2023-05-24 18:14:21 +05:30 committed by GitHub
parent 0d2e4a30b5
commit a81941f9eb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
8 changed files with 20 additions and 30 deletions

View File

@ -172,7 +172,6 @@ describe('RedShift Ingestion', () => {
verifyResponseStatusCode('@getIngestionPipelineStatus', 200); verifyResponseStatusCode('@getIngestionPipelineStatus', 200);
// Add DBT ingestion // Add DBT ingestion
cy.contains('Add dbt Ingestion').should('be.visible');
cy.get('[data-testid="dbt-source"]').should('be.visible').click(); cy.get('[data-testid="dbt-source"]').should('be.visible').click();
cy.get('.ant-select-item-option-content') cy.get('.ant-select-item-option-content')
.contains('HTTP Config Source') .contains('HTTP Config Source')
@ -217,7 +216,7 @@ describe('RedShift Ingestion', () => {
}); });
}); });
it.skip('Validate DBT is ingested properly', () => { it('Validate DBT is ingested properly', () => {
// Verify DBT tags // Verify DBT tags
interceptURL( interceptURL(
'GET', 'GET',
@ -227,12 +226,12 @@ describe('RedShift Ingestion', () => {
cy.get('[data-testid="governance"]') cy.get('[data-testid="governance"]')
.should('exist') .should('exist')
.should('be.visible') .should('be.visible')
.click({ force: true }); .click();
cy.get('[data-testid="appbar-item-tags"]') cy.get('[data-testid="appbar-item-tags"]')
.should('exist') .should('exist')
.should('be.visible') .should('be.visible')
.click(); .click({ waitForAnimations: true });
verifyResponseStatusCode('@getTagList', 200); verifyResponseStatusCode('@getTagList', 200);
// Verify DBT tag category is added // Verify DBT tag category is added

View File

@ -101,9 +101,10 @@ describe('Alerts page should work properly', () => {
// Enter email // Enter email
cy.get('#subscriptionConfig_receivers') cy.get('#subscriptionConfig_receivers')
.click() .click()
.type('testuser@openmetadata.org'); .type('testuser@openmetadata.org')
.type('{enter}');
// Click save // Click save
cy.get('[data-testid="save"]').click(); cy.get('[data-testid="save"]').scrollIntoView().click();
verifyResponseStatusCode('@createAlert', 201); verifyResponseStatusCode('@createAlert', 201);
toastNotification('Alerts created successfully.'); toastNotification('Alerts created successfully.');
cy.get('table').should('contain', alertForAllAssets); cy.get('table').should('contain', alertForAllAssets);
@ -246,9 +247,10 @@ describe('Alerts page should work properly', () => {
// Enter email // Enter email
cy.get('#subscriptionConfig_receivers') cy.get('#subscriptionConfig_receivers')
.click() .click()
.type('testuser@openmetadata.org'); .type('testuser@openmetadata.org')
.type('{enter}');
// Click save // Click save
cy.get('[data-testid="save"]').click(); cy.get('[data-testid="save"]').scrollIntoView().click();
toastNotification('Alerts created successfully.'); toastNotification('Alerts created successfully.');
cy.get('table').should('contain', TEST_CASE.testCaseAlert); cy.get('table').should('contain', TEST_CASE.testCaseAlert);
cy.get('.ant-table-cell') cy.get('.ant-table-cell')

View File

@ -423,7 +423,7 @@ describe('Data Quality and Profiler should work properly', () => {
cy.get('.ant-modal-footer').contains('Cancel').click(); cy.get('.ant-modal-footer').contains('Cancel').click();
}); });
it.skip('Delete Column Test Case should work properly', () => { it('Delete Column Test Case should work properly', () => {
interceptURL('GET', '/api/v1/dataQuality/testCases?*', 'testCase'); interceptURL('GET', '/api/v1/dataQuality/testCases?*', 'testCase');
goToProfilerTab(); goToProfilerTab();
verifyResponseStatusCode('@testCase', 200); verifyResponseStatusCode('@testCase', 200);
@ -437,7 +437,7 @@ describe('Data Quality and Profiler should work properly', () => {
.click(); .click();
[columnTestName, nonTeamTypeColumnTestName].map((test) => { [columnTestName, nonTeamTypeColumnTestName].map((test) => {
cy.get(`[data-testid="${test}"]`).should('be.visible'); cy.get(`[data-testid="${test}"]`).scrollIntoView().should('be.visible');
cy.get(`[data-testid="delete-${test}"]`) cy.get(`[data-testid="delete-${test}"]`)
.scrollIntoView() .scrollIntoView()
.should('be.visible') .should('be.visible')

View File

@ -311,19 +311,19 @@ describe('Entity Details Page', () => {
cy.clickOnLogo(); cy.clickOnLogo();
}; };
it.skip('Add Owner, Tier and tags for entity', () => { it('Add Owner, Tier and tags for entity', () => {
addOwnerTierAndTag(DELETE_ENTITY.table); addOwnerTierAndTag(DELETE_ENTITY.table);
}); });
it.skip('Remove Owner and Tier for entity', () => { it('Remove Owner and Tier for entity', () => {
removeOwnerAndTier(DELETE_ENTITY.table); removeOwnerAndTier(DELETE_ENTITY.table);
}); });
it.skip('Add and check active announcement for the entity', () => { it('Add and check active announcement for the entity', () => {
addAnnouncement(DELETE_ENTITY.table); addAnnouncement(DELETE_ENTITY.table);
}); });
it.skip('Delete entity flow should work properly', () => { it('Delete entity flow should work properly', () => {
deleteEntity(DELETE_ENTITY.table); deleteEntity(DELETE_ENTITY.table);
}); });
}); });

View File

@ -187,7 +187,9 @@ const updateSynonyms = (uSynonyms) => {
.click({ force: true, multiple: true }); .click({ force: true, multiple: true });
cy.get('.ant-select-selection-overflow') cy.get('.ant-select-selection-overflow')
.should('exist') .should('exist')
.type(uSynonyms.join('{enter}')); .type(uSynonyms.join('{enter}'))
.type('{enter}');
interceptURL('PATCH', '/api/v1/glossaryTerms/*', 'saveSynonyms'); interceptURL('PATCH', '/api/v1/glossaryTerms/*', 'saveSynonyms');
cy.get('[data-testid="save-synonym-btn"]').should('be.visible').click(); cy.get('[data-testid="save-synonym-btn"]').should('be.visible').click();
verifyResponseStatusCode('@saveSynonyms', 200); verifyResponseStatusCode('@saveSynonyms', 200);

View File

@ -161,6 +161,7 @@ describe('Roles page should work properly', () => {
cy.get('table').should('be.visible').should('contain', roleName); cy.get('table').should('be.visible').should('contain', roleName);
cy.get(`[data-row-key="${roleName}"]`) cy.get(`[data-row-key="${roleName}"]`)
.find('[data-testid="plus-more-count"]') .find('[data-testid="plus-more-count"]')
.scrollIntoView()
.should('be.visible') .should('be.visible')
.click(); .click();

View File

@ -13,7 +13,6 @@
import { BASE_URL } from '../../constants/constants'; import { BASE_URL } from '../../constants/constants';
import { import {
LEFT_PANEL_DETAILS,
NAVBAR_DETAILS, NAVBAR_DETAILS,
SETTINGS_LEFT_PANEL, SETTINGS_LEFT_PANEL,
} from '../../constants/redirections.constants'; } from '../../constants/redirections.constants';
@ -43,16 +42,7 @@ describe('Redirection link should work properly', () => {
}); });
}); });
it.skip(`Check mydata redirection mydata links on left panel.`, () => { it('Check redirection links on settings page', () => {
Object.values(LEFT_PANEL_DETAILS).map((leftpanel) => {
cy.get(leftpanel.testid).should('be.visible').click();
validateURL(leftpanel.url);
cy.clickOnLogo();
validateURL(`${BASE_URL}/my-data`);
});
});
it.skip('Check redirection links on settings page', () => {
cy.get(NAVBAR_DETAILS.settings.testid).should('be.visible').click(); cy.get(NAVBAR_DETAILS.settings.testid).should('be.visible').click();
Object.values(SETTINGS_LEFT_PANEL).map((settingsLeftPanel) => { Object.values(SETTINGS_LEFT_PANEL).map((settingsLeftPanel) => {
cy.get(settingsLeftPanel.testid) cy.get(settingsLeftPanel.testid)

View File

@ -33,10 +33,6 @@
font-weight: 600; font-weight: 600;
} }
.right-panel-announcement:hover {
background-color: initial;
}
.view-all-btn { .view-all-btn {
color: @group-title-color !important; color: @group-title-color !important;
} }