fix: for set alias

Author: maggiben

src/commands/force/org/status.ts

@@ -73,8 +73,8 @@ export class OrgStatusCommand extends SfdxCommand {
       });
       if (results.sandboxRes?.authUserName) {
         if (this.flags.setalias) {
-          const alias = await Aliases.create({});
-          const result = alias.set(this.flags.setalias, results.sandboxRes.authUserName);
+          const aliases = await Aliases.create(Aliases.getDefaultOptions());
+          const result = await aliases.updateValue(this.flags.setalias, results.sandboxRes.authUserName);
           this.logger.debug('Set Alias: %s result: %s', this.flags.setalias, result);
         }
         if (this.flags.setdefaultusername) {

test/commands/force/org/status.test.ts

@@ -48,10 +48,10 @@ describe('org:status', () => {
   // stubs
   let uxTableStub: sinon.SinonStub;
   let cmd: TestOrgStatusCommand;
-  let aliasSetStub: sinon.SinonStub;
   let configSetStub: sinon.SinonStub;
   let configWriteStub: sinon.SinonStub;
   let onStub: sinon.SinonStub;
+  let updateValueStub: sinon.SinonStub;
   let configAggregatorStub;
 
   class TestOrgStatusCommand extends OrgStatusCommand {
@@ -92,37 +92,42 @@ describe('org:status', () => {
     stubMethod(sandbox, Lifecycle, 'getInstance').returns({
       on: onStub,
     });
-    aliasSetStub = stubMethod(sandbox, Aliases.prototype, 'set').returns(sandboxalias);
     uxTableStub = stubMethod(sandbox, UX.prototype, 'table');
+    updateValueStub = stubMethod(sandbox, Aliases.prototype, 'updateValue');
     return cmd.runIt();
   };
 
   it('will return sandbox process object', async () => {
     const res = await runStatusCommand(['--sandboxname', sanboxname]);
     expect(uxTableStub.firstCall.args[0].length).to.equal(12);
-    expect(aliasSetStub.callCount).to.be.equal(0);
+    expect(updateValueStub.callCount).to.be.equal(0);
     expect(configSetStub.callCount).to.be.equal(0);
     expect(configWriteStub.callCount).to.be.equal(0);
     expect(onStub.callCount).to.be.equal(2);
     expect(res).to.deep.equal(sandboxProcessObj);
   });
 
-  it('will set alias and default username', async () => {
+  it('will set alias', async () => {
     const res = await runStatusCommand([
       '--sandboxname',
       sanboxname,
       '--setalias',
       sandboxalias,
       '--setdefaultusername',
     ]);
-    expect(aliasSetStub.callCount).to.be.equal(1);
-    expect(aliasSetStub.firstCall.args[0]).to.be.equal(sandboxalias);
-    expect(aliasSetStub.firstCall.args[1]).to.be.equal(authUserName);
+    expect(updateValueStub.firstCall.args).to.deep.equal([sandboxalias, authUserName]);
+    expect(onStub.secondCall.firstArg).to.be.equal(SandboxEvents.EVENT_RESULT);
+    expect(onStub.callCount).to.be.equal(2);
+    expect(res).to.deep.equal(sandboxProcessObj);
+  });
+
+  it('will set default username', async () => {
+    const res = await runStatusCommand(['--sandboxname', sanboxname, '--setdefaultusername']);
     expect(configSetStub.firstCall.args[0]).to.be.equal(Config.DEFAULT_USERNAME);
     expect(configSetStub.firstCall.args[1]).to.be.equal(authUserName);
+    expect(configWriteStub.calledOnce).to.be.true;
     expect(onStub.secondCall.firstArg).to.be.equal(SandboxEvents.EVENT_RESULT);
     expect(onStub.callCount).to.be.equal(2);
-    expect(configWriteStub.calledOnce).to.be.true;
     expect(res).to.deep.equal(sandboxProcessObj);
   });