Skip to content

Unskip vislib tests#71452

Merged
flash1293 merged 14 commits intoelastic:masterfrom
flash1293:fix/flaky-vislib-charts-71245
Jul 22, 2020
Merged

Unskip vislib tests#71452
flash1293 merged 14 commits intoelastic:masterfrom
flash1293:fix/flaky-vislib-charts-71245

Conversation

@flash1293
Copy link
Copy Markdown
Contributor

@flash1293 flash1293 commented Jul 13, 2020

Fixes #71245
Fixes #21642

Unskips and fixes the xy axes tests. As these were skipped for a long time, a bunch of stuff broke. This PR fixes these things and stabilizes the test by relaxing the check on log scale ticks - instead of validating a static list, it just checks the bounds and a rough number of ticks. This is necessary because small differences in test environment (e.g. display resolution) can affect the number of ticks. Ran 100 times here https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/594/ without failures.

@flash1293
Copy link
Copy Markdown
Contributor Author

@flash1293
Copy link
Copy Markdown
Contributor Author

@flash1293 flash1293 added release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v8.0.0 labels Jul 16, 2020
@flash1293 flash1293 marked this pull request as ready for review July 16, 2020 11:18
@flash1293 flash1293 requested a review from a team July 16, 2020 11:18
@flash1293 flash1293 added the Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// label Jul 16, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@flash1293
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@ghost
Copy link
Copy Markdown

ghost commented Jul 21, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #71452 updated]

  • Start Time: 2020-07-21T12:17:09.697+0000

  • Duration: 5 min 55 sec

Copy link
Copy Markdown
Contributor

@dej611 dej611 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
visTypeVislib 1.3MB +1.0B 1.3MB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@flash1293
Copy link
Copy Markdown
Contributor Author

Merging without additional approval because @dej611 is member of kibana-app

@flash1293 flash1293 merged commit 1810dd1 into elastic:master Jul 22, 2020
flash1293 added a commit to flash1293/kibana that referenced this pull request Jul 22, 2020
# Conflicts:
#	test/functional/apps/visualize/_vertical_bar_chart_nontimeindex.js
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jul 22, 2020
* master: (23 commits)
  Stabilize closing toast (elastic#72097)
  stabilize failing test (elastic#72086)
  Stabilize filter bar test (elastic#72032)
  Unskip vislib tests (elastic#71452)
  [ML] Fix layout of anomaly chart tooltip for long field values (elastic#72689)
  fix preAuth/preRouting mocks (elastic#72663)
  [Security Solution] Hide KQL bar (all pages) and alerts filters (Detections) when Resolver is full screen (elastic#72788)
  [Uptime] Rename Whitelist to Allowlist in parse_filter_map (elastic#71584)
  [Security Solution] Fixes exception modal not loading content (elastic#72770)
  [Security Solution][Exceptions] - Require non empty entries and non empty string values in exception list items (elastic#72748)
  [Detections] Add validation for Threshold value field (elastic#72611)
  [SIEM][Detection Engine][Lists] Adds version and immutability data structures (elastic#72730)
  [Security Solution][Detections] Validate file type of value lists (elastic#72746)
  [pre-req] New Component Layout proposal (elastic#72385)
  [ML] do not throw an error when agg is not supported by UI (elastic#72685)
  [Resolver] Origin process (elastic#72382)
  [Ingest Manager] Allow to force unenroll from the UI (elastic#72386)
  skip 6.8 branch when triggering baseline-capture builds (elastic#72706)
  [CI] In-progress PR comments (elastic#72211)
  Fix sorting of scripted string fields (elastic#72681)
  ...
flash1293 added a commit that referenced this pull request Jul 22, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jul 22, 2020
* master: (34 commits)
  Adds Role Based Access-Control to the Alerting & Action plugins based on Kibana Feature Controls (elastic#67157)
  [Monitoring] Revert direct shipping code (elastic#72505)
  Use server basepath  when creating reporting jobs (elastic#72722)
  Adding api test for transaction_groups /breakdown and /avg_duration_by_browser (elastic#72623)
  [Task Manager] Addresses flaky test introduced by buffered store (elastic#72815)
  [Observability] filter "hasData" api by processor event (elastic#72810)
  do  not pass title as part of tsvb request (elastic#72619)
  [Lens] Legend config (elastic#70619)
  Stabilize closing toast (elastic#72097)
  stabilize failing test (elastic#72086)
  Stabilize filter bar test (elastic#72032)
  Unskip vislib tests (elastic#71452)
  [ML] Fix layout of anomaly chart tooltip for long field values (elastic#72689)
  fix preAuth/preRouting mocks (elastic#72663)
  [Security Solution] Hide KQL bar (all pages) and alerts filters (Detections) when Resolver is full screen (elastic#72788)
  [Uptime] Rename Whitelist to Allowlist in parse_filter_map (elastic#71584)
  [Security Solution] Fixes exception modal not loading content (elastic#72770)
  [Security Solution][Exceptions] - Require non empty entries and non empty string values in exception list items (elastic#72748)
  [Detections] Add validation for Threshold value field (elastic#72611)
  [SIEM][Detection Engine][Lists] Adds version and immutability data structures (elastic#72730)
  ...
flash1293 added a commit to flash1293/kibana that referenced this pull request Jul 24, 2020
# Conflicts:
#	test/functional/apps/visualize/_vertical_bar_chart_nontimeindex.js
flash1293 added a commit that referenced this pull request Jul 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.9.0 v7.10.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unskip flaky vertical bar char tests Scale tick tests are flaky

4 participants