Move TestName warning into script

Author: rkeithhill

InvokePesterStub.ps1

@@ -39,10 +39,20 @@ param(
     [Parameter()]
     [ValidatePattern('\d*')]
     [string]
-    $LineNumber
+    $LineNumber,
+
+    # If specified, executes all the tests in the specified test script.
+    [Parameter()]
+    [switch]
+    $All
 )
 
 try {
+    if ($All) {
+        Invoke-Pester -Script $ScriptPath -PesterOption @{IncludeVSCodeMarker=$true}
+        return
+    }
+
     $pesterVersion = (Microsoft.PowerShell.Core\Get-Command Invoke-Pester -ErrorAction Stop).Version
     if (($pesterVersion -ge '4.6.0') -and ($LineNumber -match '\d+')) {
         $pesterOption = New-PesterOption -ScriptBlockFilter @(
@@ -53,6 +63,11 @@ try {
         Invoke-Pester -Script $ScriptPath -PesterOption @{ IncludeVSCodeMarker=$true } -TestName $TestName
     }
     else {
+        # We get here when PSES couldn't parse the TestName
+        Write-Warning "The Describe block's TestName cannot be evaluated. ALL TESTS will be executed."
+        Write-Warning "Either try again with Pester 4.6.0 or higher, or remove any variables or"
+        Write-Warning "sub-expressions in the Describe block's TestName."
+
         Invoke-Pester -Script $ScriptPath -PesterOption @{IncludeVSCodeMarker=$true}
     }
 }

src/features/PesterTests.ts

@@ -54,24 +54,18 @@ export class PesterTestsFeature implements IFeature {
     private launchAllTestsInActiveEditor(launchType: LaunchType) {
         const uriString = vscode.window.activeTextEditor.document.uri.toString();
         const launchConfig = this.createLaunchConfig(uriString, launchType);
+        launchConfig.args.push("-All");
         this.launch(launchConfig);
     }
 
-    private async launchTests(uriString: string, runInDebugger: boolean, describeBlockName?: string, lineNum?: number) {
-        // PSES passes null for the describeBlockName to signal that it can't evaluate the TestName.
-        if (!describeBlockName) {
-            const answer = await vscode.window.showErrorMessage(
-                "This Describe block's TestName parameter cannot be evaluated. " +
-                `Would you like to ${runInDebugger ? "debug" : "run"} all the tests in this file?`,
-                "Yes", "No");
-
-            if (answer !== "Yes") {
-                return;
-            }
-        }
+    private async launchTests(
+        uriString: string,
+        runInDebugger: boolean,
+        describeBlockName?: string,
+        describeBlockLineNumber?: number) {
 
         const launchType = runInDebugger ? LaunchType.Debug : LaunchType.Run;
-        const launchConfig = this.createLaunchConfig(uriString, launchType, describeBlockName, lineNum);
+        const launchConfig = this.createLaunchConfig(uriString, launchType, describeBlockName, describeBlockLineNumber);
         this.launch(launchConfig);
     }