Merge pull request #124 from salesforcecli/sm/fix-multiple-alias-9351642

fix: use most recent alias

Author: maggiben

src/shared/utils.ts

@@ -11,7 +11,8 @@ import * as open from 'open';
 export const getAliasByUsername = async (username: string): Promise<string> => {
   const alias = await Aliases.create(Aliases.getDefaultOptions());
   const keys = alias.getKeysByValue(username);
-  return keys?.length ? keys[0] : undefined;
+  // use the most recently added alias for that username
+  return keys?.length ? keys[keys.length - 1] : undefined;
 };
 
 export const openUrl = async (url: string): Promise<ChildProcess> => {

test/shared/utils.test.ts

@@ -27,8 +27,8 @@ describe('getAliasByUsername', () => {
     expect(await getAliasByUsername('username1')).to.equal('alias1');
   });
 
-  it('returns first alias for a username that has multiple aliases', async () => {
-    expect(await getAliasByUsername('username2')).to.equal('alias2');
+  it('returns most recent alias for a username that has multiple aliases', async () => {
+    expect(await getAliasByUsername('username2')).to.equal('alias2b');
   });
 
   it('returns undefined when no matching username is found', async () => {