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

Hide/Show authentication method in multi data source plugin based on configuration #5916

Merged
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
333987f
[AuthType Config] Add EnabledAuthType configuration in yml file and p…
xinruiba Feb 20, 2024
5327710
[AuthType Config] provide default auth types when types array is empty
xinruiba Feb 20, 2024
cc91319
[AuthRegistry Onboard - DataSourceCreationForm] Integrate auth cofig …
xinruiba Feb 22, 2024
0339047
[AuthRegistry Onboard] Support default selected auth type in datasour…
xinruiba Feb 23, 2024
9b138fe
[AuthRegistry Onboard] SDisable auth option selector when auth option…
xinruiba Feb 23, 2024
f0b4c88
[AuthRegistry Onboard] clear credential attribute every time use sele…
xinruiba Feb 23, 2024
c6bb34c
Merge branch 'opensearch-project:main' into xinrui_AuthMethod_Config
xinruiba Feb 23, 2024
1ebd2e2
Update yml file to disable config
xinruiba Feb 23, 2024
91e734e
Update change.md file
xinruiba Feb 23, 2024
fc0369e
[UT] Fix broken test cases
xinruiba Feb 23, 2024
2bf8abe
Merge branch 'main' into xinrui_AuthMethod_Config
xinruiba Feb 23, 2024
f0201ef
[UT] Add more unit tests
xinruiba Feb 23, 2024
ae1b8c8
Merge branch 'main' into xinrui_AuthMethod_Config
xinruiba Feb 23, 2024
246b1d3
Merge branch 'opensearch-project:main' into xinrui_AuthMethod_Config
xinruiba Feb 26, 2024
551f87a
Merge branch 'main' into xinrui_AuthMethod_Config
xinruiba Feb 27, 2024
c2da4e2
[UT] update unit test and handle scenario when no options enabled
xinruiba Feb 27, 2024
8f2c75a
[UT] update description of unit test case
xinruiba Feb 27, 2024
6d7463e
[UT] update unit test cases to resolve comment
xinruiba Feb 27, 2024
f11cebf
Merge branch 'main' into xinrui_AuthMethod_Config
xinruiba Feb 27, 2024
b0b1918
Merge branch 'main' into xinrui_AuthMethod_Config
xinruiba Feb 27, 2024
9d39598
[UT] Snapshot auth option super selector for unit test. Also upate th…
xinruiba Feb 27, 2024
070195d
Merge branch 'main' into xinrui_AuthMethod_Config
xinruiba Feb 27, 2024
39f77b9
Merge branch 'main' into xinrui_AuthMethod_Config
xinruiba Feb 27, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[UT] update description of unit test case
Signed-off-by: Xinrui Bai <[email protected]>
  • Loading branch information
xinruiba committed Feb 27, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
commit 8f2c75a016a1643b919a1fecf97ae7564d96b174
Original file line number Diff line number Diff line change
@@ -267,7 +267,7 @@ describe('DataSourceManagement: Utils.ts', () => {
});
});

test('default auth method is NoAuth when no auth options configured', () => {
test('default auth type is NoAuth when no auth options registered in authenticationMethodRegistery, this should not happen in real customer scenario for MD', () => {
const authenticationMethodRegistery = new AuthenticationMethodRegistery();
expect(getDefaultAuthMethod(authenticationMethodRegistery)?.name).toBe(AuthType.NoAuth);
});
Loading