fix: add package:beta:update, NUT

Author: WillieRuemmele

messages/package_update.md

@@ -35,6 +35,6 @@ new package description
 
 New description of the package.
 
-# humanSuccess
+# success
 
-Successfully updated the package.
+Successfully updated the package. %s

src/commands/force/package/beta/update.ts

@@ -8,6 +8,7 @@
 import * as os from 'os';
 import { flags, FlagsConfig, SfdxCommand } from '@salesforce/command';
 import { Messages } from '@salesforce/core';
+import { BY_LABEL, getPackageIdFromAlias, Package, PackageSaveResult, validateId } from '@salesforce/packaging';
 
 Messages.importMessagesDirectory(__dirname);
 const messages = Messages.loadMessages('@salesforce/plugin-packaging', 'package_update');
@@ -42,8 +43,20 @@ export class PackageUpdateCommand extends SfdxCommand {
     }),
   };
 
-  // eslint-disable-next-line @typescript-eslint/require-await
-  public async run(): Promise<unknown> {
-    throw new Error('Beta command not yet implemented');
+  public async run(): Promise<PackageSaveResult> {
+    const pkg = new Package({ connection: this.hubOrg.getConnection() });
+    const id = getPackageIdFromAlias(this.flags.package, this.project);
+    validateId(BY_LABEL.PACKAGE_ID, id);
+
+    const result = await pkg.update({
+      Id: id,
+      Name: this.flags.name as string,
+      Description: this.flags.description as string,
+      PackageErrorUsername: this.flags.errornotificationusername as string,
+    });
+
+    this.ux.log(messages.getMessage('success', [id]));
+
+    return result;
   }
 }

test/commands/force/package/packageCreateAndDelete.nut.ts

@@ -10,7 +10,7 @@ import { execCmd, TestSession } from '@salesforce/cli-plugins-testkit';
 import { OrgConfigProperties } from '@salesforce/core';
 import { expect } from 'chai';
 
-describe('package create and delete', () => {
+describe('package create/update/delete', () => {
   let session: TestSession;
   let devHubUsernameOrAlias: string;
   let pkgName: string;
@@ -29,7 +29,7 @@ describe('package create and delete', () => {
   after(async () => {
     await session?.clean();
   });
-  describe('create/delete - human results', () => {
+  describe('create/update/delete - human results', () => {
     before(async () => {
       pkgName = `test-pkg-${Date.now()}`;
     });
@@ -40,14 +40,20 @@ describe('package create and delete', () => {
       expect(output).to.contain('=== Ids');
       expect(output).to.match(/Package Id\s+?0Ho/);
     });
+    it('should update a package - human readable results', function () {
+      const command = `force:package:beta:update --package ${pkgName} --description "My new description" -v ${devHubUsernameOrAlias}`;
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+      const output = execCmd(command, { ensureExitCode: 0 }).shellOutput.stdout as string;
+      expect(output).to.match(/Successfully updated the package\.\s+0Ho/);
+    });
     it('should delete a package - human readable results', function () {
       const command = `force:package:beta:delete -p ${pkgName} -v ${devHubUsernameOrAlias} --noprompt`;
       // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
       const output = execCmd(command, { ensureExitCode: 0 }).shellOutput.stdout as string;
       expect(output).to.contain('Successfully deleted the package. 0Ho');
     });
   });
-  describe('create/delete - json results', () => {
+  describe('create/update/delete - json results', () => {
     before(async () => {
       pkgName = `test-pkg-${Date.now()}`;
     });
@@ -59,6 +65,14 @@ describe('package create and delete', () => {
       expect(output.result).to.have.property('Id');
       expect(output.result.Id).to.match(/0Ho.{12,15}/);
     });
+    it('should update a package - json results', function () {
+      const command = `force:package:beta:update --package ${pkgName} --description "My new description" -v ${devHubUsernameOrAlias} --json`;
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+      const output = execCmd<{ id: string }>(command, { ensureExitCode: 0 }).jsonOutput;
+      expect(output.status).to.equal(0);
+      expect(output.result).to.have.property('id');
+      expect(output.result.id).to.match(/0Ho.{12,15}/);
+    });
     it('should delete a package - json results', function () {
       const command = `force:package:beta:delete -p ${pkgName} -v ${devHubUsernameOrAlias} --json`;
       // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access