fixing #1145 (#1146)

Author: connectdotz

src/test-provider/jest-test-run.ts

@@ -68,7 +68,13 @@ export class JestTestRun implements JestExtOutput, TestRunProtocol {
     if (!this._run && !this.isCancelled) {
       const runName = `${this.name} (${this.runCount++})`;
 
-      this._run = this.createRun(this.request, runName);
+      // vscode seems to identify run by the request instance, so we need to create a new request for each run
+      const request = new vscode.TestRunRequest(
+        this.request.include,
+        this.request.exclude,
+        this.request.profile
+      );
+      this._run = this.createRun(request, runName);
       this._run.appendOutput(`\r\nTestRun "${runName}" started\r\n`);
 
       // ignore skipped tests if there are more than one test to run

tests/test-provider/jest-test-runt.test.ts

@@ -44,6 +44,8 @@ describe('JestTestRun', () => {
       }));
 
     mockRequest = {};
+    (vscode.TestRunRequest as jest.Mocked<any>).mockClear();
+    (vscode.TestRunRequest as jest.Mocked<any>).mockImplementation(() => mockRequest);
     jestRun = new JestTestRun('test', mockContext, mockRequest, mockCreateTestRun);
   });
 
@@ -365,10 +367,36 @@ describe('JestTestRun', () => {
       expect(run1.end).toHaveBeenCalled();
 
       const newRequest: any = { include: ['test1'] };
+      (vscode.TestRunRequest as jest.Mocked<any>).mockImplementation(() => newRequest);
       jestRun.updateRequest(newRequest);
       jestRun.started({} as any);
       expect(mockCreateTestRun).toHaveBeenCalledTimes(2);
-      expect(mockCreateTestRun.mock.calls[1][0]).toBe(newRequest);
+      expect(mockCreateTestRun.mock.calls[1][0]).toEqual(newRequest);
+      expect(vscode.TestRunRequest).toHaveBeenCalledTimes(2);
+    });
+  });
+
+  describe('supports continuous test run', () => {
+    it('by start/stop underlying TestRun per continuous run session', () => {
+      jestRun = new JestTestRun('test', mockContext, mockRequest, mockCreateTestRun);
+
+      // first run
+      jestRun.started({} as any);
+      expect(mockCreateTestRun).toHaveBeenCalledTimes(1);
+      const run1 = mockCreateTestRun.mock.results[0].value;
+      expect(run1.started).toHaveBeenCalled();
+      jestRun.end();
+      expect(run1.end).toHaveBeenCalled();
+      expect(vscode.TestRunRequest).toHaveBeenCalledTimes(1);
+
+      // 2nd run
+      jestRun.started({} as any);
+      expect(mockCreateTestRun).toHaveBeenCalledTimes(2);
+      const run2 = mockCreateTestRun.mock.results[1].value;
+      expect(run2.started).toHaveBeenCalled();
+      jestRun.end();
+      expect(run2.end).toHaveBeenCalled();
+      expect(vscode.TestRunRequest).toHaveBeenCalledTimes(2);
     });
   });
   describe('cancel', () => {