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

[Backport 2.x] Feature: Dev Tools Query Export/Import #4986

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 56dd85b from #3810.

* feat: Add export functionality for dev tool queries

Signed-off-by: kishor82 <[email protected]>

* feat: Add import functionality for dev tool queries

Signed-off-by: kishor82 <[email protected]>

* update: changelog.

Signed-off-by: kishor82 <[email protected]>

* feat: updated license headers for new file.

Signed-off-by: kishor82 <[email protected]>

* fix: typo:ImportFlyout and removed ImportModeControl prop (isLegacyFile)

Signed-off-by: kishor82 <[email protected]>

* fix: default query export

Signed-off-by: kishor82 <[email protected]>

* fix: added basic text validation for imported file.

Signed-off-by: kishor82 <[email protected]>

* update: removed legacy variable export.

Signed-off-by: kishor82 <[email protected]>

* test: Add unit tests for OverwriteModal component

Signed-off-by: kishor82 <[email protected]>

* test: Add unit tests for ImportModeControl component

Signed-off-by: kishor82 <[email protected]>

* test: Add unit tests for ImportFlyout component

Signed-off-by: kishor82 <[email protected]>

* test: updated unit tests for ImportFlyout component

Signed-off-by: kishor82 <[email protected]>

* test: Enhance async operation handling and complete async cycle in Enzyme test suites.

Signed-off-by: kishor82 <[email protected]>

* test: Enhanced test coverage for ImportFlyout component unit tests

Signed-off-by: kishor82 <[email protected]>

* fix: Add 'http' property to serviceContextMock and update test expectations

Signed-off-by: kishor82 <[email protected]>

---------

Signed-off-by: kishor82 <[email protected]>
(cherry picked from commit 56dd85b)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot added ux / ui Improvements or additions to user experience, flows, components, UI elements autocut Skip the changelog verification check on backports first-time-contributor v2.10.0 labels Sep 11, 2023
@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Merging #4986 (a5c33f5) into 2.x (062ac82) will decrease coverage by 0.03%.
Report is 1 commits behind head on 2.x.
The diff coverage is 92.70%.

@@            Coverage Diff             @@
##              2.x    #4986      +/-   ##
==========================================
- Coverage   66.50%   66.47%   -0.03%     
==========================================
  Files        3400     3402       +2     
  Lines       64981    64977       -4     
  Branches    10385    10374      -11     
==========================================
- Hits        43215    43196      -19     
- Misses      19210    19217       +7     
- Partials     2556     2564       +8     
Flag Coverage Δ
Linux_1 34.86% <ø> (+0.04%) ⬆️
Linux_2 55.19% <ø> (-0.15%) ⬇️
Linux_3 44.38% <92.70%> (-0.09%) ⬇️
Linux_4 35.05% <ø> (+0.02%) ⬆️
Windows_1 34.88% <ø> (+0.04%) ⬆️
Windows_2 55.15% <ø> (-0.16%) ⬇️
Windows_3 44.38% <92.70%> (-0.09%) ⬇️
Windows_4 35.05% <ø> (+0.02%) ⬆️

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

Files Changed Coverage Δ
...blic/application/contexts/services_context.mock.ts 100.00% <ø> (ø)
...e/public/application/contexts/services_context.tsx 77.77% <ø> (ø)
...lic/application/components/import_mode_control.tsx 92.30% <92.30%> (ø)
...le/public/application/components/import_flyout.tsx 92.50% <92.50%> (ø)
.../public/application/components/overwrite_modal.tsx 100.00% <100.00%> (ø)
...c/application/hooks/use_data_init/use_data_init.ts 8.00% <100.00%> (+3.83%) ⬆️

... and 15 files with indirect coverage changes

@ananzh ananzh merged commit 2c51a10 into 2.x Sep 13, 2023
75 checks passed
@github-actions github-actions bot deleted the backport/backport-3810-to-2.x branch September 13, 2023 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports first-time-contributor ux / ui Improvements or additions to user experience, flows, components, UI elements v2.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants