mirror of
https://github.com/grafana/grafana.git
synced 2025-02-25 18:55:37 -06:00
Chore: Update @grafana/e2e (#66365)
* Update e2e package - Ensure health request is monitored for addDatasource flow - Add custom timeout to configurePanel flow * Make health check optional
This commit is contained in:
parent
85e20768b6
commit
b2e32a70b9
@ -49,6 +49,7 @@ function provisionAzureMonitorDatasources(datasources: AzureMonitorProvision[])
|
|||||||
expectedAlertMessage: 'Successfully connected to all Azure Monitor endpoints',
|
expectedAlertMessage: 'Successfully connected to all Azure Monitor endpoints',
|
||||||
// Reduce the timeout from 30s to error faster when an invalid alert message is presented
|
// Reduce the timeout from 30s to error faster when an invalid alert message is presented
|
||||||
timeout: 10000,
|
timeout: 10000,
|
||||||
|
awaitHealth: true,
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -14,6 +14,7 @@ export interface AddDataSourceConfig {
|
|||||||
skipTlsVerify: boolean;
|
skipTlsVerify: boolean;
|
||||||
type: string;
|
type: string;
|
||||||
timeout?: number;
|
timeout?: number;
|
||||||
|
awaitHealth?: boolean;
|
||||||
}
|
}
|
||||||
|
|
||||||
// @todo this actually returns type `Cypress.Chainable<AddDaaSourceConfig>`
|
// @todo this actually returns type `Cypress.Chainable<AddDaaSourceConfig>`
|
||||||
@ -40,8 +41,15 @@ export const addDataSource = (config?: Partial<AddDataSourceConfig>) => {
|
|||||||
skipTlsVerify,
|
skipTlsVerify,
|
||||||
type,
|
type,
|
||||||
timeout,
|
timeout,
|
||||||
|
awaitHealth,
|
||||||
} = fullConfig;
|
} = fullConfig;
|
||||||
|
|
||||||
|
if (awaitHealth) {
|
||||||
|
e2e()
|
||||||
|
.intercept(/health/)
|
||||||
|
.as('health');
|
||||||
|
}
|
||||||
|
|
||||||
e2e().logToConsole('Adding data source with name:', name);
|
e2e().logToConsole('Adding data source with name:', name);
|
||||||
e2e.pages.AddDataSource.visit();
|
e2e.pages.AddDataSource.visit();
|
||||||
e2e.pages.AddDataSource.dataSourcePluginsV2(type)
|
e2e.pages.AddDataSource.dataSourcePluginsV2(type)
|
||||||
@ -76,6 +84,10 @@ export const addDataSource = (config?: Partial<AddDataSourceConfig>) => {
|
|||||||
|
|
||||||
e2e.pages.DataSource.saveAndTest().click();
|
e2e.pages.DataSource.saveAndTest().click();
|
||||||
|
|
||||||
|
if (awaitHealth) {
|
||||||
|
e2e().wait('@health', { timeout: timeout ?? e2e.config().defaultCommandTimeout });
|
||||||
|
}
|
||||||
|
|
||||||
// use the timeout passed in if it exists, otherwise, continue to use the default
|
// use the timeout passed in if it exists, otherwise, continue to use the default
|
||||||
e2e.pages.DataSource.alert()
|
e2e.pages.DataSource.alert()
|
||||||
.should('exist')
|
.should('exist')
|
||||||
|
@ -34,6 +34,7 @@ interface ConfigurePanelOptional {
|
|||||||
panelTitle?: string;
|
panelTitle?: string;
|
||||||
timeRange?: TimeRangeConfig;
|
timeRange?: TimeRangeConfig;
|
||||||
visualizationName?: string;
|
visualizationName?: string;
|
||||||
|
timeout?: number;
|
||||||
}
|
}
|
||||||
|
|
||||||
interface ConfigurePanelRequired {
|
interface ConfigurePanelRequired {
|
||||||
@ -80,6 +81,7 @@ export const configurePanel = (config: PartialAddPanelConfig | PartialEditPanelC
|
|||||||
timeRange,
|
timeRange,
|
||||||
visitDashboardAtStart,
|
visitDashboardAtStart,
|
||||||
visualizationName,
|
visualizationName,
|
||||||
|
timeout,
|
||||||
} = fullConfig;
|
} = fullConfig;
|
||||||
|
|
||||||
if (visitDashboardAtStart) {
|
if (visitDashboardAtStart) {
|
||||||
@ -158,7 +160,7 @@ export const configurePanel = (config: PartialAddPanelConfig | PartialEditPanelC
|
|||||||
e2e.components.RefreshPicker.runButtonV2().first().click({ force: true });
|
e2e.components.RefreshPicker.runButtonV2().first().click({ force: true });
|
||||||
|
|
||||||
// Wait for RxJS
|
// Wait for RxJS
|
||||||
e2e().wait(500);
|
e2e().wait(timeout ?? e2e.config().defaultCommandTimeout);
|
||||||
|
|
||||||
if (matchScreenshot) {
|
if (matchScreenshot) {
|
||||||
let visualization;
|
let visualization;
|
||||||
|
Loading…
Reference in New Issue
Block a user