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.

Sign up for GitHub

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

Jump to bottom

[Table Visualization] Move all the table vis related functional tests to functional test repo #401

Closed
ananzh opened this issue Nov 17, 2022 · 0 comments
Closed

[Table Visualization] Move all the table vis related functional tests to functional test repo #401

ananzh opened this issue Nov 17, 2022 · 0 comments
Assignees
@kavilla
Labels
dashboards OpenSearch Dashboards functions

Comments

@ananzh
Copy link
Member

ananzh commented Nov 17, 2022

Currently, we are working on documenting and understanding all the functional tests for table vis: opensearch-project/OpenSearch-Dashboards#2857

This is a tracking issue to make sure we re-create all the documented tests.

Exit criteria:

  • All the tests are re-created and merged.
@ananzh ananzh added bug Something isn't working untriaged labels Nov 17, 2022
@ananzh ananzh self-assigned this Nov 17, 2022
@ananzh ananzh removed bug Something isn't working untriaged labels Nov 17, 2022
@ananzh ananzh moved this to Todo in Visualizations Nov 17, 2022
@ananzh ananzh added the dashboards OpenSearch Dashboards functions label Dec 5, 2022
ananzh added a commit to ananzh/opensearch-dashboards-functional-test that referenced this issue Dec 13, 2022
@ananzh
[Table visualization] Add functional tests for new table vis
58c371b 
There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
opensearch-project#401

Signed-off-by: Anan Zhuang <[email protected]>
@ananzh ananzh mentioned this issue Dec 13, 2022
Merged
1 task
ananzh added a commit to ananzh/opensearch-dashboards-functional-test that referenced this issue Dec 13, 2022
@ananzh
[Table visualization] Add functional tests for new table vis
15e6f2e 
There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
opensearch-project#401

Signed-off-by: Anan Zhuang <[email protected]>
@ashwin-pc ashwin-pc assigned kavilla and unassigned ananzh May 9, 2023
kavilla pushed a commit to ananzh/opensearch-dashboards-functional-test that referenced this issue May 18, 2023
@ananzh @kavilla
[Table visualization] Add functional tests for new table vis
08170e2 
There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
opensearch-project#401

Signed-off-by: Anan Zhuang <[email protected]>
opensearch-trigger-bot bot pushed a commit that referenced this issue May 24, 2023
@ananzh @github-actions
[Table visualization] Add functional tests for new table vis (#423)
a298cff 
* [Table visualization] Add functional tests for new table vis

There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
#401

Signed-off-by: Anan Zhuang <[email protected]>

* fix PR comments

Signed-off-by: Anan Zhuang <[email protected]>

* fix test error and add embed test suite

Signed-off-by: Anan Zhuang <[email protected]>

* Fix multiple submit bug
Currently there are two submit type button in data source
which cause cypress fail to find and click. In this PR, I
switch to use data-test-subj to uniquely identify the btn.

Issue Resolved:
#439

Signed-off-by: Anan Zhuang <[email protected]>

* missed conflict resolution

Signed-off-by: Kawika Avilla <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 3dffb5f)
opensearch-trigger-bot bot pushed a commit that referenced this issue May 24, 2023
@ananzh @github-actions
[Table visualization] Add functional tests for new table vis (#423)
f83f287 
* [Table visualization] Add functional tests for new table vis

There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
#401

Signed-off-by: Anan Zhuang <[email protected]>

* fix PR comments

Signed-off-by: Anan Zhuang <[email protected]>

* fix test error and add embed test suite

Signed-off-by: Anan Zhuang <[email protected]>

* Fix multiple submit bug
Currently there are two submit type button in data source
which cause cypress fail to find and click. In this PR, I
switch to use data-test-subj to uniquely identify the btn.

Issue Resolved:
#439

Signed-off-by: Anan Zhuang <[email protected]>

* missed conflict resolution

Signed-off-by: Kawika Avilla <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 3dffb5f)
kavilla pushed a commit that referenced this issue May 24, 2023
@opensearch-trigger-bot @ananzh
[Table visualization] Add functional tests for new table vis (#423) (#…
c76c8b8 
…671)

* [Table visualization] Add functional tests for new table vis

There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
#401

Signed-off-by: Anan Zhuang <[email protected]>

* fix PR comments

Signed-off-by: Anan Zhuang <[email protected]>

* fix test error and add embed test suite

Signed-off-by: Anan Zhuang <[email protected]>

* Fix multiple submit bug
Currently there are two submit type button in data source
which cause cypress fail to find and click. In this PR, I
switch to use data-test-subj to uniquely identify the btn.

Issue Resolved:
#439

Signed-off-by: Anan Zhuang <[email protected]>

* missed conflict resolution

Signed-off-by: Kawika Avilla <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 3dffb5f)

Co-authored-by: Anan Zhuang <[email protected]>
peterzhuamazon pushed a commit that referenced this issue May 26, 2023
@opensearch-trigger-bot @ananzh
[Table visualization] Add functional tests for new table vis (#423) (#…
ed47791 
…672)

* [Table visualization] Add functional tests for new table vis

There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
#401

Signed-off-by: Anan Zhuang <[email protected]>

* fix PR comments

Signed-off-by: Anan Zhuang <[email protected]>

* fix test error and add embed test suite

Signed-off-by: Anan Zhuang <[email protected]>

* Fix multiple submit bug
Currently there are two submit type button in data source
which cause cypress fail to find and click. In this PR, I
switch to use data-test-subj to uniquely identify the btn.

Issue Resolved:
#439

Signed-off-by: Anan Zhuang <[email protected]>

* missed conflict resolution

Signed-off-by: Kawika Avilla <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 3dffb5f)

Co-authored-by: Anan Zhuang <[email protected]>
leanneeliatra pushed a commit to leanneeliatra/opensearch-dashboards-functional-test-fork that referenced this issue Sep 15, 2023
@opensearch-trigger-bot @ananzh @leanneeliatra
[Table visualization] Add functional tests for new table vis (opensea…
c63f874 
…rch-project#423) (opensearch-project#671)

* [Table visualization] Add functional tests for new table vis

There are four test cases:
* basic.spec.js contains tests related to vis page, like
inspector, save an object.
* data.spec.js tests diff metrics and buckets aggregations.
* options.spec.js contains tests on options panel.
* split.spec.js tests table functions, like sort, adjust col
width, split tables and etc.

Most tests are the same as the removed funtional tests.
split.spec.js is the only one with all new tests.

Partially Resolved:
opensearch-project#401

Signed-off-by: Anan Zhuang <[email protected]>

* fix PR comments

Signed-off-by: Anan Zhuang <[email protected]>

* fix test error and add embed test suite

Signed-off-by: Anan Zhuang <[email protected]>

* Fix multiple submit bug
Currently there are two submit type button in data source
which cause cypress fail to find and click. In this PR, I
switch to use data-test-subj to uniquely identify the btn.

Issue Resolved:
opensearch-project#439

Signed-off-by: Anan Zhuang <[email protected]>

* missed conflict resolution

Signed-off-by: Kawika Avilla <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 3dffb5f)

Co-authored-by: Anan Zhuang <[email protected]>
Signed-off-by: [email protected] <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kavilla kavilla

Labels
dashboards OpenSearch Dashboards functions
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kavilla @ananzh