Skip to content

Fix: Add tags to canvas functional suite#26046

Merged
w33ble merged 1 commit intoelastic:masterfrom
w33ble:fix/tag-canvas-functional-tests
Nov 21, 2018
Merged

Fix: Add tags to canvas functional suite#26046
w33ble merged 1 commit intoelastic:masterfrom
w33ble:fix/tag-canvas-functional-tests

Conversation

@w33ble
Copy link
Copy Markdown
Contributor

@w33ble w33ble commented Nov 21, 2018

Summary

this.tags is required now, I guess.

@w33ble w33ble added review v7.0.0 Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v6.6.0 v6.5.2 labels Nov 21, 2018
@w33ble w33ble requested a review from spalger November 21, 2018 20:33
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-canvas

Copy link
Copy Markdown
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM

@w33ble w33ble merged commit a6c6706 into elastic:master Nov 21, 2018
@w33ble
Copy link
Copy Markdown
Contributor Author

w33ble commented Nov 21, 2018

backport is happening via #26037 (6.5) and #26047 (6.x)

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

w33ble added a commit that referenced this pull request Nov 21, 2018
w33ble added a commit that referenced this pull request Nov 22, 2018
* chore: revert #25921 / d76c0ae

* Test: canvas functional smoke test (#25262)

~~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](https://user-images.githubusercontent.com/404731/48730518-da7ee980-ebf7-11e8-9abb-cf294079bb5f.png)

* test: add ci group tags

backport of #26046

* chore: shot in the dark beforeAll fix

* chore: add some debug logging

in the beforeAll hook, to see where it's getting stuck

* [canvas/archive] remove _1 from .kibana archives to avoid lockup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v6.5.2 v6.6.0 v7.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants