-
Notifications
You must be signed in to change notification settings - Fork 918
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
[Manual Backport 2.x] Refactor getClient and getLegacyClient to use authentication method registry (#5881) #5940
[Manual Backport 2.x] Refactor getClient and getLegacyClient to use authentication method registry (#5881) #5940
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.x #5940 +/- ##
==========================================
+ Coverage 67.03% 67.04% +0.01%
==========================================
Files 3303 3305 +2
Lines 63626 63655 +29
Branches 10149 10157 +8
==========================================
+ Hits 42653 42679 +26
- Misses 18508 18510 +2
- Partials 2465 2466 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
dabc3e1
to
ee11f87
Compare
c5130b9
…egistry (opensearch-project#5881) * Refactor getClient and getLegacyClient to use authentication method registry Signed-off-by: Bandini Bhopi <bandinib@amazon.com> * Adds changelog and UT Signed-off-by: Bandini Bhopi <bandinib@amazon.com> --------- Signed-off-by: Bandini Bhopi <bandinib@amazon.com> (cherry picked from commit d56b04d)
Signed-off-by: Bandini Bhopi <bandinib@amazon.com>
c5130b9
to
42a90ba
Compare
Failing cypress test is not related to this change. |
Description
Backport from d56b04 from #5881
Issues Resolved
Screenshot
Testing the changes
Check List
yarn test:jest
yarn test:jest_integration