mirror of
https://github.com/elastic/kibana.git
synced 2025-04-24 01:38:56 -04:00
# Backport This will backport the following commits from `main` to `8.18`: - [[Visualize] Unskip smoke test (#214960)](https://github.com/elastic/kibana/pull/214960) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Marco Vettorello","email":"marco.vettorello@elastic.co"},"sourceCommit":{"committedDate":"2025-03-25T09:30:07Z","message":"[Visualize] Unskip smoke test (#214960)\n\n## Summary\n\nUnskipping and fixing skipped test\n\nThe reported issue was referring to an issue with a TSVB chart that was\ntrying to load a missing data view.\nIf not explicitly configured in the configuration, TSVB can load the\ndefault data view configured in the Advanced Settings.\nThere is no mention of the missing dataview ID in the the dashboard nor\nin the TSVB panel configuration.\nThe TSVB configuration uses at this point the default index coming from\nthe Advanced Settings, so that ID seems to be configured as the default\nID but there is no loaded dataview with that name.\nThe data view ID appearing in the error was referencing the ecommerce\ndashboard, that was not used and not loaded in this test (all charts\nrefers to the logs* data view).\n\n\n\n\nI believe this issues was caused by other tests in the same suite/config\nthat configured the `defaultIndex` setting without unsetting it.","sha":"12aa3fc6ca8124db8cf5e940baec121482fd1a4b","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","Team:Visualizations","skipped-test","release_note:skip","backport:prev-minor","backport:prev-major","v9.1.0"],"title":"[Visualize] Unskip smoke test","number":214960,"url":"https://github.com/elastic/kibana/pull/214960","mergeCommit":{"message":"[Visualize] Unskip smoke test (#214960)\n\n## Summary\n\nUnskipping and fixing skipped test\n\nThe reported issue was referring to an issue with a TSVB chart that was\ntrying to load a missing data view.\nIf not explicitly configured in the configuration, TSVB can load the\ndefault data view configured in the Advanced Settings.\nThere is no mention of the missing dataview ID in the the dashboard nor\nin the TSVB panel configuration.\nThe TSVB configuration uses at this point the default index coming from\nthe Advanced Settings, so that ID seems to be configured as the default\nID but there is no loaded dataview with that name.\nThe data view ID appearing in the error was referencing the ecommerce\ndashboard, that was not used and not loaded in this test (all charts\nrefers to the logs* data view).\n\n\n\n\nI believe this issues was caused by other tests in the same suite/config\nthat configured the `defaultIndex` setting without unsetting it.","sha":"12aa3fc6ca8124db8cf5e940baec121482fd1a4b"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/214960","number":214960,"mergeCommit":{"message":"[Visualize] Unskip smoke test (#214960)\n\n## Summary\n\nUnskipping and fixing skipped test\n\nThe reported issue was referring to an issue with a TSVB chart that was\ntrying to load a missing data view.\nIf not explicitly configured in the configuration, TSVB can load the\ndefault data view configured in the Advanced Settings.\nThere is no mention of the missing dataview ID in the the dashboard nor\nin the TSVB panel configuration.\nThe TSVB configuration uses at this point the default index coming from\nthe Advanced Settings, so that ID seems to be configured as the default\nID but there is no loaded dataview with that name.\nThe data view ID appearing in the error was referencing the ecommerce\ndashboard, that was not used and not loaded in this test (all charts\nrefers to the logs* data view).\n\n\n\n\nI believe this issues was caused by other tests in the same suite/config\nthat configured the `defaultIndex` setting without unsetting it.","sha":"12aa3fc6ca8124db8cf5e940baec121482fd1a4b"}}]}] BACKPORT--> Co-authored-by: Marco Vettorello <marco.vettorello@elastic.co>
This commit is contained in:
parent
e66cd7b47a
commit
7b963600ce
3 changed files with 6 additions and 4 deletions
|
@ -28,8 +28,9 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
|
|||
await kibanaServer.uiSettings.replace({ defaultIndex: 'test-histogram' });
|
||||
});
|
||||
|
||||
after(function () {
|
||||
return esArchiver.unload('x-pack/test/functional/es_archives/pre_calculated_histogram');
|
||||
after(async function () {
|
||||
await esArchiver.unload('x-pack/test/functional/es_archives/pre_calculated_histogram');
|
||||
await kibanaServer.uiSettings.unset('defaultIndex');
|
||||
});
|
||||
|
||||
it('appears correctly in discover', async function () {
|
||||
|
|
|
@ -65,6 +65,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
|
|||
body: { query: { match_all: {} } },
|
||||
});
|
||||
await kibanaServer.uiSettings.unset('timepicker:timeDefaults');
|
||||
await kibanaServer.uiSettings.unset('defaultIndex');
|
||||
});
|
||||
|
||||
afterEach(async () => {
|
||||
|
@ -115,6 +116,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
|
|||
await kibanaServer.importExport.unload(
|
||||
'x-pack/test/functional/fixtures/kbn_archiver/reporting/ecommerce_76.json'
|
||||
);
|
||||
await kibanaServer.uiSettings.unset('defaultIndex');
|
||||
});
|
||||
|
||||
afterEach(async () => {
|
||||
|
|
|
@ -23,8 +23,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
|
|||
const queryBar = getService('queryBar');
|
||||
const retry = getService('retry');
|
||||
|
||||
// Failing: See https://github.com/elastic/kibana/issues/207045
|
||||
describe.skip('smoke telemetry tests', function () {
|
||||
describe('smoke telemetry tests', function () {
|
||||
let uiCounterEvents: UiCounterEvent[] = [];
|
||||
|
||||
before(async function () {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue