Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6.x] Test: canvas functional smoke test (#25262) #26047

Merged
merged 6 commits into from
Nov 22, 2018

Conversation

w33ble
Copy link
Contributor

@w33ble w33ble commented Nov 21, 2018

Backports the following commits to 6.x:

  • Test: canvas functional smoke test

Blocked by #23046 (pending #25828) merged

Adds functional smoke tests for Canvas.

  • Loads and checks the list of workpads
  • Loads the first workpad and checks that elements render

This is the simple workpad it's testing

screenshot 2018-11-19 12 37 07 (#25262)

@w33ble w33ble changed the title [6.x] Test: canvas functional smoke test (#25262) ~~Blocked by https://github.com/elastic/kibana/issues/23046 (pending https://github.com/elastic/kibana/pull/25828)~~ merged Adds functional smoke tests [6.x] Test: canvas functional smoke test (#25262) Nov 21, 2018
w33ble and others added 3 commits November 21, 2018 16:35
~~Blocked by elastic#23046 (pending elastic#25828 merged

Adds functional smoke tests for Canvas. 

- Loads and checks the list of workpads
- Loads the first workpad and checks that elements render

This is the simple workpad it's testing

![screenshot 2018-11-19 12 37 07](https://user-images.githubusercontent.com/404731/48730518-da7ee980-ebf7-11e8-9abb-cf294079bb5f.png)
@w33ble
Copy link
Contributor Author

w33ble commented Nov 21, 2018

This pr now also reverts #25921

Backports #26054

@elasticmachine
Copy link
Contributor

💔 Build Failed

@spalger
Copy link
Contributor

spalger commented Nov 22, 2018

Retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@w33ble w33ble merged commit 26fc0a9 into elastic:6.x Nov 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants