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

[TSVB]: initial markdown test implementation #31893

Merged
merged 23 commits into from
Mar 5, 2019

Conversation

vitalics
Copy link
Contributor

@vitalics vitalics commented Feb 25, 2019

Summary

Extanding test coverage for TSVB mardown

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] Documentation was added for features that require explanation or tutorials

- [ ] This was checked for keyboard-only and screenreader accessibility

For maintainers

- [ ] This was checked for breaking API changes and was labeled appropriately
- [ ] This includes a feature addition or change that requires a release note and was labeled appropriately

@vitalics vitalics requested a review from dmlemeshko February 25, 2019 12:13
@vitalics vitalics mentioned this pull request Feb 25, 2019
7 tasks
@vitalics vitalics changed the title initial implementation with skipping all tests [TSVB]: initial markdown test implementation Feb 25, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💔 Build Failed

@dmlemeshko
Copy link
Member

Flakiness is caused by the way visualisation loading handled here. Check my commit 738afe0388b5131f637a18da0469c2b6a1d894c1
I was able to make it work without retry

@vitalics vitalics requested review from a team as code owners February 26, 2019 12:01
@elasticmachine
Copy link
Contributor

💔 Build Failed

@@ -22,7 +22,6 @@ export const advancedFilter = () => ({
);

handlers.onDestroy(() => {
handlers.setFilter('');
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this the result of a bad rebase/merge? I think all the Canvas changes you have here are in master already.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, it's result of a bad rebase/merge

@vitalics vitalics force-pushed the tsvb-markdown-tests branch from 176500b to be9146c Compare February 27, 2019 14:15
@elasticmachine
Copy link
Contributor

💔 Build Failed

@vitalics vitalics force-pushed the tsvb-markdown-tests branch from be9146c to 80d8ace Compare February 28, 2019 10:06
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@vitalics vitalics force-pushed the tsvb-markdown-tests branch from be4df47 to 4630026 Compare February 28, 2019 13:53
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@vitalics vitalics force-pushed the tsvb-markdown-tests branch from adba49c to 493d8f7 Compare March 1, 2019 11:32
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@vitalics vitalics requested a review from dmlemeshko March 1, 2019 12:23
test/functional/page_objects/visual_builder_page.ts Outdated Show resolved Hide resolved
test/functional/apps/visualize/_tsvb_markdown.ts Outdated Show resolved Hide resolved
test/functional/apps/visualize/_tsvb_markdown.ts Outdated Show resolved Hide resolved
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@dmlemeshko
Copy link
Member

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@vitalics vitalics added test Team:QA Team label for QA Team test_ui_functional Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Mar 4, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-qa

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@vitalics vitalics added the Feature:TSVB TSVB (Time Series Visual Builder) label Mar 4, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@dmlemeshko
Copy link
Member

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

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

LGTM. Tested

Copy link
Contributor

@alexwizp alexwizp left a comment

Choose a reason for hiding this comment

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

Code LGTM

@vitalics vitalics merged commit 0623763 into elastic:master Mar 5, 2019
@vitalics vitalics deleted the tsvb-markdown-tests branch March 5, 2019 09:15
vitalics added a commit to vitalics/kibana that referenced this pull request Mar 5, 2019
* Improve test coverage for TSVB markdown
vitalics added a commit that referenced this pull request Mar 5, 2019
* Improve test coverage for TSVB markdown
vitalics added a commit to vitalics/kibana that referenced this pull request Mar 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:TSVB TSVB (Time Series Visual Builder) Team:QA Team label for QA Team Team:Visualizations Visualization editors, elastic-charts and infrastructure test_ui_functional test v7.2.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants