feat(e2e): pass compile-time suite flags into detect-changes script

The script now accepts -EnableBrowser/Node/React/Angular params (baked in
by YAML compile-time expansion) and reports each suite's status clearly:

  msal-browser: skipped (disabled by parameter)
  msal-node:    skipped (no affected changes)
  msal-react:   will run
  ...

This makes manual runs with some suites disabled self-documenting in logs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: hectormmg

.pipelines/3p-e2e.yml

@@ -80,6 +80,11 @@ extends:
                       inputs:
                         targetType: filePath
                         filePath: .pipelines/scripts/detect-changes.ps1
+                        arguments: >-
+                          -EnableBrowser "${{ parameters.runBrowserTests }}"
+                          -EnableNode "${{ parameters.runNodeTests }}"
+                          -EnableReact "${{ parameters.runReactTests }}"
+                          -EnableAngular "${{ parameters.runAngularTests }}"
                         pwsh: true
 
                 - ${{ if eq(parameters.runBrowserTests, true) }}:

.pipelines/scripts/detect-changes.ps1

@@ -3,15 +3,32 @@
 # changes default all flags to true (fail-open).
 #
 # Parameters:
-#   -Repo3pPath  Path to the 3P repo root. Defaults to the current directory.
-#   -Repo1pPath  Path to the 1P repo root. Optional; provide when the 1P repo
-#                is checked out (i.e. in 1p-e2e.yml) to also gate msal-browser-1p.
+#   -Repo3pPath      Path to the 3P repo root. Defaults to the current directory.
+#   -Repo1pPath      Path to the 1P repo root. Optional; provide when the 1P repo
+#                    is checked out (i.e. in 1p-e2e.yml) to also gate msal-browser-1p.
+#   -EnableBrowser   Whether msal-browser tests are enabled at all (compile-time param).
+#   -EnableNode      Whether msal-node tests are enabled at all (compile-time param).
+#   -EnableReact     Whether msal-react tests are enabled at all (compile-time param).
+#   -EnableAngular   Whether msal-angular tests are enabled at all (compile-time param).
+#   -Enable1p        Whether msal-browser-1p tests are enabled at all (compile-time param).
 
 param(
     [string]$Repo3pPath = $PWD,
-    [string]$Repo1pPath = ""
+    [string]$Repo1pPath = "",
+    [string]$EnableBrowser = "true",
+    [string]$EnableNode    = "true",
+    [string]$EnableReact   = "true",
+    [string]$EnableAngular = "true",
+    [string]$Enable1p      = "true"
 )
 
+# Convert string flags passed from YAML compile-time expansion to booleans
+$enabledBrowser = $EnableBrowser -ne "false"
+$enabledNode    = $EnableNode    -ne "false"
+$enabledReact   = $EnableReact   -ne "false"
+$enabledAngular = $EnableAngular -ne "false"
+$enabled1p      = $Enable1p      -ne "false"
+
 $runBrowser = $runNode = $runReact = $runAngular = $run1p = $true
 
 if ($env:SYSTEM_PULLREQUEST_TARGETBRANCH) {
@@ -63,7 +80,26 @@ if ($env:SYSTEM_PULLREQUEST_TARGETBRANCH) {
     Write-Host "Non-PR build — running all test suites"
 }
 
-Write-Host "Run: browser=$runBrowser node=$runNode react=$runReact angular=$runAngular browser-1p=$run1p"
+# AND detection result with compile-time enabled flags and report reason
+function Report-Suite([string]$name, [bool]$detected, [bool]$enabled) {
+    if (-not $enabled)   { Write-Host "  $name`: skipped (disabled by parameter)" }
+    elseif (-not $detected) { Write-Host "  $name`: skipped (no affected changes)" }
+    else                 { Write-Host "  $name`: will run" }
+}
+
+Write-Host "Suite status:"
+Report-Suite "msal-browser"    $runBrowser $enabledBrowser
+Report-Suite "msal-node"       $runNode    $enabledNode
+Report-Suite "msal-react"      $runReact   $enabledReact
+Report-Suite "msal-angular"    $runAngular $enabledAngular
+Report-Suite "msal-browser-1p" $run1p      $enabled1p
+
+$runBrowser = $runBrowser -and $enabledBrowser
+$runNode    = $runNode    -and $enabledNode
+$runReact   = $runReact   -and $enabledReact
+$runAngular = $runAngular -and $enabledAngular
+$run1p      = $run1p      -and $enabled1p
+
 Write-Host "##vso[task.setvariable variable=runMsalBrowser;isOutput=true]$($runBrowser.ToString().ToLower())"
 Write-Host "##vso[task.setvariable variable=runMsalNode;isOutput=true]$($runNode.ToString().ToLower())"
 Write-Host "##vso[task.setvariable variable=runMsalReact;isOutput=true]$($runReact.ToString().ToLower())"