Either clone or pull, don't do both

Author: jablko

src/testRunner/externalCompileRunner.ts

@@ -57,9 +57,11 @@ abstract class ExternalCompileRunnerBase extends RunnerBase {
                     if (!fs.existsSync(submoduleDir)) {
                         exec("git", ["clone", config.cloneUrl, directoryName], { cwd });
                     }
-                    exec("git", ["reset", "HEAD", "--hard"], { cwd: submoduleDir });
-                    exec("git", ["clean", "-f"], { cwd: submoduleDir });
-                    exec("git", ["pull", "-f"], { cwd: submoduleDir });
+                    else {
+                        exec("git", ["reset", "HEAD", "--hard"], { cwd: submoduleDir });
+                        exec("git", ["clean", "-f"], { cwd: submoduleDir });
+                        exec("git", ["pull", "-f"], { cwd: submoduleDir });
+                    }
 
                     types = config.types;
 

tests/baselines/reference/user/TypeScript-Vue-Starter.log

@@ -0,0 +1,9 @@
+Exit Code: 1
+Standard output:
+src/components/Hello.spec.ts(4,1): error TS2593: Cannot find name 'describe'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.spec.ts(5,3): error TS2593: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.spec.ts(14,5): error TS2304: Cannot find name 'expect'.
+
+
+
+Standard error:

tests/baselines/reference/user/TypeScript-WeChat-Starter.log

@@ -1,12 +1,16 @@
 Exit Code: 1
 Standard output:
-../../../../../node_modules/@types/mocha/index.d.ts(2680,13): error TS2403: Subsequent variable declarations must have the same type.  Variable 'beforeEach' must be of type 'Lifecycle', but here has type 'HookFunction'.
-../../../../../node_modules/@types/mocha/index.d.ts(2698,13): error TS2403: Subsequent variable declarations must have the same type.  Variable 'afterEach' must be of type 'Lifecycle', but here has type 'HookFunction'.
-../../../../../node_modules/@types/mocha/index.d.ts(2714,13): error TS2403: Subsequent variable declarations must have the same type.  Variable 'describe' must be of type 'Describe', but here has type 'SuiteFunction'.
-../../../../../node_modules/@types/mocha/index.d.ts(2735,13): error TS2403: Subsequent variable declarations must have the same type.  Variable 'xdescribe' must be of type 'Describe', but here has type 'PendingSuiteFunction'.
-../../../../../node_modules/@types/mocha/index.d.ts(2749,13): error TS2403: Subsequent variable declarations must have the same type.  Variable 'it' must be of type 'It', but here has type 'TestFunction'.
-../../../../../node_modules/@types/mocha/index.d.ts(2763,13): error TS2403: Subsequent variable declarations must have the same type.  Variable 'test' must be of type 'It', but here has type 'TestFunction'.
-../../../../../node_modules/@types/mocha/index.d.ts(2770,13): error TS2403: Subsequent variable declarations must have the same type.  Variable 'xit' must be of type 'It', but here has type 'PendingTestFunction'.
+src/App.test.tsx(5,1): error TS2593: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.test.tsx(5,1): error TS2593: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.test.tsx(7,3): error TS2304: Cannot find name 'expect'.
+src/components/Hello.test.tsx(10,1): error TS2593: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.test.tsx(12,3): error TS2304: Cannot find name 'expect'.
+src/components/Hello.test.tsx(15,1): error TS2593: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.test.tsx(17,3): error TS2304: Cannot find name 'expect'.
+src/components/Hello.test.tsx(20,1): error TS2593: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.test.tsx(21,3): error TS2304: Cannot find name 'expect'.
+src/components/Hello.test.tsx(26,1): error TS2593: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i @types/jest` or `npm i @types/mocha` and then add `jest` or `mocha` to the types field in your tsconfig.
+src/components/Hello.test.tsx(27,3): error TS2304: Cannot find name 'expect'.
 
 
 

tests/baselines/reference/user/puppeteer.log

@@ -1,6 +1,5 @@
 Exit Code: 1
 Standard output:
-../../../../../node_modules/@types/gulp/index.d.ts(9,27): error TS2307: Cannot find module 'chokidar'.
 lib/Accessibility.js(71,12): error TS2339: Property 'nodes' does not exist on type 'Promise<any>'.
 lib/Accessibility.js(74,14): error TS2339: Property 'node' does not exist on type 'Promise<any>'.
 lib/Accessibility.js(134,15): error TS2503: Cannot find namespace 'Protocol'.
@@ -204,7 +203,7 @@ lib/Page.js(384,9): error TS2339: Property 'cookies' does not exist on type 'Pro
 lib/Page.js(388,20): error TS2503: Cannot find namespace 'Protocol'.
 lib/Page.js(424,5): error TS2322: Type 'Promise<ElementHandle>' is not assignable to type 'ElementHandle'.
 lib/Page.js(432,5): error TS2322: Type 'Promise<ElementHandle>' is not assignable to type 'ElementHandle'.
-lib/Page.js(451,7): error TS2740: Type '(...args: any[]) => Promise<any>' is missing the following properties from type 'Window': applicationCache, caches, clientInformation, closed, and 242 more.
+lib/Page.js(451,7): error TS2740: Type '(...args: any[]) => Promise<any>' is missing the following properties from type 'Window': applicationCache, caches, clientInformation, closed, and 223 more.
 lib/Page.js(461,9): error TS2349: This expression is not callable.
   Type 'Window' has no call signatures.
 lib/Page.js(493,46): error TS2339: Property 'metrics' does not exist on type 'Promise<any>'.