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

[Tests-refactor] Remove angular code from plugin_functional and update tests #5221

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

ananzh
Copy link
Member

@ananzh ananzh commented Oct 4, 2023

Description

  • remove angular-based view from DocViewsPlugin
  • remove the test case that validates the display and content of the angular doc view

Issue Resolved

#5020

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@ananzh ananzh changed the title [Refactor] Remove angular code from plugin_functional and update tests [Tests-refactor] Remove angular code from plugin_functional and update tests Oct 4, 2023
…e tests

* remove angular-based view from DocViewsPlugin
* remove the test case that validates the display and content of the angular doc view

Issue Resolve
opensearch-project#5020

Signed-off-by: ananzh <ananzh@amazon.com>
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #5221 (a907e47) into main (9601c6c) will decrease coverage by 0.01%.
Report is 9 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5221      +/-   ##
==========================================
- Coverage   66.76%   66.76%   -0.01%     
==========================================
  Files        3278     3278              
  Lines       62999    63008       +9     
  Branches    10031    10034       +3     
==========================================
+ Hits        42059    42065       +6     
- Misses      18473    18554      +81     
+ Partials     2467     2389      -78     
Flag Coverage Δ
Linux_1 35.32% <ø> (+<0.01%) ⬆️
Linux_2 55.24% <ø> (ø)
Linux_3 43.74% <ø> (ø)
Linux_4 35.42% <ø> (ø)
Windows_1 35.34% <ø> (+<0.01%) ⬆️
Windows_2 55.20% <ø> (ø)
Windows_3 43.75% <ø> (ø)
Windows_4 35.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 20 files with indirect coverage changes

Copy link
Member

@ashwin-pc ashwin-pc left a comment

Choose a reason for hiding this comment

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

How did we miss this?

@AMoo-Miki AMoo-Miki merged commit 70b9eea into opensearch-project:main Oct 5, 2023
58 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 5, 2023
…e tests (#5221)

* remove angular-based view from DocViewsPlugin
* remove the test case that validates the display and content of the angular doc view

Issue Resolve
#5020

Signed-off-by: ananzh <ananzh@amazon.com>
(cherry picked from commit 70b9eea)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
ananzh pushed a commit that referenced this pull request Oct 5, 2023
…e tests (#5221) (#5224)

* remove angular-based view from DocViewsPlugin
* remove the test case that validates the display and content of the angular doc view

Issue Resolve
#5020

Signed-off-by: ananzh <ananzh@amazon.com>
(cherry picked from commit 70b9eea)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
willie-hung pushed a commit to willie-hung/OpenSearch-Dashboards that referenced this pull request Oct 5, 2023
…e tests (opensearch-project#5221)

* remove angular-based view from DocViewsPlugin
* remove the test case that validates the display and content of the angular doc view

Issue Resolve
opensearch-project#5020

Signed-off-by: ananzh <ananzh@amazon.com>
Signed-off-by: Willie Hung <willie880201044@gmail.com>
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Oct 7, 2023
…e tests (opensearch-project#5221)

* remove angular-based view from DocViewsPlugin
* remove the test case that validates the display and content of the angular doc view

Issue Resolve
opensearch-project#5020

Signed-off-by: ananzh <ananzh@amazon.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.

4 participants