Skip to content

Commit 4d27c50

Browse files
authored
CORE-489 support NIH account and RAS simultaneously (#5324)
1 parent 2800043 commit 4d27c50

File tree

2 files changed

+3
-28
lines changed

2 files changed

+3
-28
lines changed

src/profile/external-identities/ExternalIdentities.test.tsx

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -104,32 +104,7 @@ describe('ExternalIdentities', () => {
104104
});
105105
});
106106

107-
it('sorts providers based on desiredOrder without RAS', async () => {
108-
// Arrange
109-
asMockedFn(getConfig).mockReturnValue(
110-
partial<AppConfigSettings>({
111-
externalCreds: partial<AppConfigSettings['externalCreds']>({
112-
providers: ['era-commons', 'fence', 'dcf-fence', 'kids-first', 'anvil'],
113-
}),
114-
})
115-
);
116-
117-
// Act
118-
render(<ExternalIdentities queryParams={{}} />);
119-
120-
// Assert
121-
const providerElements = Array.from(screen.getByRole('main').querySelectorAll('div')).map((div) => div.textContent);
122-
expect(providerElements).toStrictEqual([
123-
'Nih Account',
124-
'eRA Commons',
125-
'NHLBI BioData Catalyst Framework Services',
126-
'NCI CRDC Framework Services',
127-
'Kids First DRC Framework Services',
128-
'NHGRI AnVIL Data Commons Framework Services',
129-
]);
130-
});
131-
132-
it('sorts providers based on desiredOrder with RAS', async () => {
107+
it('sorts providers based on desiredOrder', async () => {
133108
// Arrange
134109
asMockedFn(getConfig).mockReturnValue(
135110
partial<AppConfigSettings>({
@@ -145,6 +120,7 @@ describe('ExternalIdentities', () => {
145120
// Assert
146121
const providerElements = Array.from(screen.getByRole('main').querySelectorAll('div')).map((div) => div.textContent);
147122
expect(providerElements).toStrictEqual([
123+
'Nih Account',
148124
'NIH Researcher Auth Service (RAS)',
149125
'NHLBI BioData Catalyst Framework Services',
150126
'NCI CRDC Framework Services',

src/profile/external-identities/ExternalIdentities.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ type ExternalIdentitiesProps = {
1515
export const ExternalIdentities = (props: ExternalIdentitiesProps): ReactNode => {
1616
const { queryParams } = props;
1717
const providers = (getConfig().externalCreds?.providers || []) as OAuth2ProviderKey[];
18-
const hasRas = providers.includes('ras');
1918
const filteredProviders = providers.filter(
2019
(p: any) =>
2120
(p !== 'github' && p !== 'sage') ||
@@ -25,7 +24,7 @@ export const ExternalIdentities = (props: ExternalIdentitiesProps): ReactNode =>
2524

2625
return (
2726
<PageBox role='main' style={{ flexGrow: 1 }} variant={PageBoxVariants.light}>
28-
{!hasRas && <NihAccount nihToken={queryParams?.['nih-username-token']} />}
27+
<NihAccount nihToken={queryParams?.['nih-username-token']} />
2928
{filteredProviders.map((providerKey: OAuth2ProviderKey) => (
3029
<OAuth2Account
3130
key={`oauth2link-${providerKey}`}

0 commit comments

Comments
 (0)