Merge pull request #70 from salesforcecli/phale/W-12679981

fix: default result format human + coverage depends on result format

Author: shetzel

messages/runtest.md

@@ -91,10 +91,6 @@ Runs test methods from a single Apex class synchronously; if not specified, test
 
 Display detailed code coverage per test.
 
-# missingReporterErr
-
-Select a result format when specifying code coverage
-
 # runTestReportCommand
 
 Run "%s apex get test -i %s -o %s" to retrieve test results

src/commands/apex/get/test.ts

@@ -57,6 +57,7 @@ export default class Test extends SfCommand<RunResult> {
       char: 'r',
       summary: messages.getMessage('flags.result-format.summary'),
       options: resultFormat,
+      default: 'human',
     }),
   };
 

src/commands/apex/run/test.ts

@@ -68,6 +68,7 @@ export default class Test extends SfCommand<RunCommandResult> {
       char: 'r',
       summary: messages.getMessage('flags.result-format.summary'),
       options: resultFormat,
+      default: 'human',
     }),
     'suite-names': Flags.string({
       deprecateAliases: true,
@@ -174,10 +175,6 @@ export default class Test extends SfCommand<RunCommandResult> {
     synchronous?: boolean,
     testLevel?: TestLevel
   ): Promise<TestLevel> {
-    if (codeCoverage && !resultFormatFlag) {
-      return Promise.reject(new Error(messages.getMessage('missingReporterErr')));
-    }
-
     if ((classNames && (suiteNames || tests)) || (suiteNames && tests)) {
       return Promise.reject(new Error(messages.getMessage('classSuiteTestErr')));
     }

test/commands/apex/run/test.test.ts

@@ -368,15 +368,6 @@ describe('apex:test:run', () => {
   });
 
   describe('validateFlags', () => {
-    // TODO: move this method to oclif flag validation
-    it('rejects codecoverage without resultformat', async () => {
-      try {
-        await new Test(['--code-coverage'], config).run();
-      } catch (e) {
-        expect((e as Error).message).to.equal(messages.getMessage('missingReporterErr'));
-      }
-    });
-
     it('rejects tests/classnames/suitenames and testlevels', async () => {
       try {
         await new Test(['--tests', 'mytest', '--test-level', 'RunAllTestsInOrg'], config).run();

yarn.lock

@@ -1369,14 +1369,6 @@
     "@typescript-eslint/types" "5.54.1"
     "@typescript-eslint/visitor-keys" "5.54.1"
 
-"@typescript-eslint/scope-manager@5.54.1":
-  version "5.54.1"
-  resolved "https://registry.yarnpkg.com/@typescript-eslint/scope-manager/-/scope-manager-5.54.1.tgz#6d864b4915741c608a58ce9912edf5a02bb58735"
-  integrity sha512-zWKuGliXxvuxyM71UA/EcPxaviw39dB2504LqAmFDjmkpO8qNLHcmzlh6pbHs1h/7YQ9bnsO8CCcYCSA8sykUg==
-  dependencies:
-    "@typescript-eslint/types" "5.54.1"
-    "@typescript-eslint/visitor-keys" "5.54.1"
-
 "@typescript-eslint/type-utils@5.53.0":
   version "5.53.0"
   resolved "https://registry.yarnpkg.com/@typescript-eslint/type-utils/-/type-utils-5.53.0.tgz#41665449935ba9b4e6a1ba6e2a3f4b2c31d6cf97"
@@ -1397,11 +1389,6 @@
   resolved "https://registry.yarnpkg.com/@typescript-eslint/types/-/types-5.54.1.tgz#29fbac29a716d0f08c62fe5de70c9b6735de215c"
   integrity sha512-G9+1vVazrfAfbtmCapJX8jRo2E4MDXxgm/IMOF4oGh3kq7XuK3JRkOg6y2Qu1VsTRmWETyTkWt1wxy7X7/yLkw==
 
-"@typescript-eslint/types@5.54.1":
-  version "5.54.1"
-  resolved "https://registry.yarnpkg.com/@typescript-eslint/types/-/types-5.54.1.tgz#29fbac29a716d0f08c62fe5de70c9b6735de215c"
-  integrity sha512-G9+1vVazrfAfbtmCapJX8jRo2E4MDXxgm/IMOF4oGh3kq7XuK3JRkOg6y2Qu1VsTRmWETyTkWt1wxy7X7/yLkw==
-
 "@typescript-eslint/typescript-estree@5.53.0":
   version "5.53.0"
   resolved "https://registry.yarnpkg.com/@typescript-eslint/typescript-estree/-/typescript-estree-5.53.0.tgz#bc651dc28cf18ab248ecd18a4c886c744aebd690"
@@ -1428,19 +1415,6 @@
     semver "^7.3.7"
     tsutils "^3.21.0"
 
-"@typescript-eslint/typescript-estree@5.54.1":
-  version "5.54.1"
-  resolved "https://registry.yarnpkg.com/@typescript-eslint/typescript-estree/-/typescript-estree-5.54.1.tgz#df7b6ae05fd8fef724a87afa7e2f57fa4a599be1"
-  integrity sha512-bjK5t+S6ffHnVwA0qRPTZrxKSaFYocwFIkZx5k7pvWfsB1I57pO/0M0Skatzzw1sCkjJ83AfGTL0oFIFiDX3bg==
-  dependencies:
-    "@typescript-eslint/types" "5.54.1"
-    "@typescript-eslint/visitor-keys" "5.54.1"
-    debug "^4.3.4"
-    globby "^11.1.0"
-    is-glob "^4.0.3"
-    semver "^7.3.7"
-    tsutils "^3.21.0"
-
 "@typescript-eslint/utils@5.53.0":
   version "5.53.0"
   resolved "https://registry.yarnpkg.com/@typescript-eslint/utils/-/utils-5.53.0.tgz#e55eaad9d6fffa120575ffaa530c7e802f13bce8"
@@ -1485,14 +1459,6 @@
     "@typescript-eslint/types" "5.54.1"
     eslint-visitor-keys "^3.3.0"
 
-"@typescript-eslint/visitor-keys@5.54.1":
-  version "5.54.1"
-  resolved "https://registry.yarnpkg.com/@typescript-eslint/visitor-keys/-/visitor-keys-5.54.1.tgz#d7a8a0f7181d6ac748f4d47b2306e0513b98bf8b"
-  integrity sha512-q8iSoHTgwCfgcRJ2l2x+xCbu8nBlRAlsQ33k24Adj8eoVBE0f8dUeI+bAa8F84Mv05UGbAx57g2zrRsYIooqQg==
-  dependencies:
-    "@typescript-eslint/types" "5.54.1"
-    eslint-visitor-keys "^3.3.0"
-
 "@ungap/promise-all-settled@1.1.2":
   version "1.1.2"
   resolved "https://registry.yarnpkg.com/@ungap/promise-all-settled/-/promise-all-settled-1.1.2.tgz#aa58042711d6e3275dd37dc597e5d31e8c290a44"