fix: follow up to support running tests with class id specified (#165)

#155, @W-8991173@
forcedotcom · Mar 18, 2021 · 9e8a00e · 9e8a00e
1 parent 135b0ab
commit 9e8a00e
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/packages/apex-node/src/tests/testService.ts b/packages/apex-node/src/tests/testService.ts
@@ -131,7 +131,8 @@ export class TestService {
           }
         }
       } else {
-        testItems.push({ className: test });
+        const prop = isValidApexClassID(test) ? 'classId' : 'className';
+        testItems.push({ [prop]: test });
       }
     }
 

diff --git a/packages/apex-node/test/tests/asyncTests.test.ts b/packages/apex-node/test/tests/asyncTests.test.ts
@@ -1261,6 +1261,22 @@ describe('Run Apex tests asynchronously', () => {
       expect(namespaceStub.notCalled).to.be.true;
     });
 
+    it('should build async payload for tests with only classid', async () => {
+      const namespaceStub = sandboxStub
+        .stub(TestService.prototype, 'queryNamespaces')
+        .resolves(new Set(['myNamespace']));
+      const testSrv = new TestService(mockConnection);
+      const payload = await testSrv.buildAsyncPayload(
+        TestLevel.RunSpecifiedTests,
+        '01p4x00000KWt3T'
+      );
+      expect(payload).to.deep.equal({
+        tests: [{ classId: '01p4x00000KWt3T' }],
+        testLevel: TestLevel.RunSpecifiedTests
+      });
+      expect(namespaceStub.notCalled).to.be.true;
+    });
+
     it('should build async payload for class with only classname', async () => {
       const namespaceStub = sandboxStub
         .stub(TestService.prototype, 'queryNamespaces')