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

Update table vis test in dashboard_filtering_spec.js #682

Merged
merged 3 commits into from
May 26, 2023

Conversation

ananzh
Copy link
Member

@ananzh ananzh commented May 26, 2023

Description

table test fails when running dashboard_filtering_spec.js due to updates on the table vis. This PR modified the mappings and updates the tests. Meanwhile commonUI.setDateRange is not very stable. Always see failure. Switch it to a more stable function.

Screenshot 2023-05-25 at 16 45 30 Screenshot 2023-05-25 at 16 56 05

Issues Resolved

#681

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Issue Resolve
opensearch-project#681

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
@peterzhuamazon
Copy link
Member

I will fix the rebase of @ananzh branch if you are ok, thanks.

@peterzhuamazon
Copy link
Member

@peterzhuamazon
Copy link
Member

The failure now is related to a different problem that OS or OSD not started. Can merge.

@peterzhuamazon peterzhuamazon merged commit f3768f8 into opensearch-project:main May 26, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request May 26, 2023
Issue Resolve
#681

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
(cherry picked from commit f3768f8)
peterzhuamazon pushed a commit that referenced this pull request May 26, 2023
Issue Resolve
#681

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
(cherry picked from commit f3768f8)

Co-authored-by: Anan Zhuang <ananzh@amazon.com>
leanneeliatra pushed a commit to leanneeliatra/opensearch-dashboards-functional-test-fork that referenced this pull request Sep 15, 2023
…ect#682) (opensearch-project#691)

Issue Resolve
opensearch-project#681

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
(cherry picked from commit f3768f8)

Co-authored-by: Anan Zhuang <ananzh@amazon.com>
Signed-off-by: leanne.laceybyrne@eliatra.com <leanne.laceybyrne@eliatra.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] table vis test fails in dashboard_filtering_spec.js test due to updates on table vis
3 participants