-
Notifications
You must be signed in to change notification settings - Fork 917
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
[Test] unit test get_telemetry_saved_object.test.ts is skipped now #515
Comments
ananzh
added a commit
to ananzh/OpenSearch-Dashboards
that referenced
this issue
Jun 22, 2021
…project#515) All the unit tests related to unused telemetry are temporarily skipped after the fork. Unit tests of the disabled telemetry functions should also be modified correspondingly. To build a clean unit test, we decide to modify and enable all the working unit tests. This PR checks and enables get_telemetry_saved_object.test.ts. Signed-off-by: Anan Zhuang <[email protected]>
5 tasks
5 tasks
ananzh
added a commit
that referenced
this issue
Jun 24, 2021
All the unit tests related to unused telemetry are temporarily skipped after the fork. Unit tests of the disabled telemetry functions should also be modified correspondingly. To build a clean unit test, we decide to modify and enable all the working unit tests. This PR checks and enables get_telemetry_saved_object.test.ts. Signed-off-by: Anan Zhuang <[email protected]>
kavilla
pushed a commit
that referenced
this issue
Jun 26, 2021
All the unit tests related to unused telemetry are temporarily skipped after the fork. Unit tests of the disabled telemetry functions should also be modified correspondingly. To build a clean unit test, we decide to modify and enable all the working unit tests. This PR checks and enables get_telemetry_saved_object.test.ts. Signed-off-by: Anan Zhuang <[email protected]>
kavilla
pushed a commit
that referenced
this issue
Jun 26, 2021
All the unit tests related to unused telemetry are temporarily skipped after the fork. Unit tests of the disabled telemetry functions should also be modified correspondingly. To build a clean unit test, we decide to modify and enable all the working unit tests. This PR checks and enables get_telemetry_saved_object.test.ts. Signed-off-by: Anan Zhuang <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the issue
Currently,
get_telemetry_saved_object.test.ts
test suite is skipped. To make a clean unit test, we want to explore and decide whether we still need to keep it as skipped.To Reproduce
Run:
yarn test:jest {path to get_telemetry_saved_object.test.ts}
Expected behavior
explore the skipped unit tests, check:
can we enable all? if yes, create a PR and close the issue
can we remove all? if yes, create a PR and close the issue
should we keep all? if yes, describe the reasons in the issue.
The final output is expected to be a cleaned unit test with all the skipped ones being checked.
OpenSearch Version
OpenSearch 1.0.0
Dashboards Version
Dashboards 1.0.0
The text was updated successfully, but these errors were encountered: