Simplify BCApps e2e scenario to verify CI/CD and PR Build success

[Copilot]

❔What, Why & How

Rework the BCApps e2e scenario to limit the build to the Apps * projects, reuse the push-triggered CI/CD run, and verify both CI/CD and PR builds conclude successfully with the expected artifacts — plus a couple of supporting e2e-infra fixes.

e2eTests/scenarios/BCApps/runtest.ps1

• Pushing the projects settings change to main already triggers a full CI/CD build; the scenario reuses that push-triggered run (returned by Add-PropertiesToJsonFile -commit -wait) instead of dispatching a second, redundant RunCICD run.
• Both the CI/CD and Pull Request Build runs are re-fetched after waiting and explicitly asserted to conclude with success — WaitWorkflow tolerates cancelled, so a cancelled run no longer passes. WaitWorkflow's global behavior is left unchanged.
• The re-fetch after WaitWorkflow now calls RefreshToken and rebuilds the headers from the refreshed $ENV:GH_TOKEN. WaitWorkflow can outlive the 10-minute GitHub App token window, so the previous stale headers could 401 on a long-running build.
• Retained app/test-app artifact validation for both builds. The selected Apps * projects define testFolders (compiled into .buildartifacts/TestApps and published as *-TestApps-*), so Test-ArtifactsFromRun + *-Apps-*/*-TestApps-* count assertions confirm the builds produced useful output.

.github/workflows/E2E.yaml

• Fixed scenario filtering: $scenariosFilter | ForEach-Object { $scenario -like $_ } returned an array of booleans (always truthy) and did not filter. Now uses @(... | Where-Object { $scenario -like $_ }).Count -gt 0, with results wrapped in @() to guarantee arrays.

e2eTests/e2eTestHelper.psm1

• Pass -silent to the initial invoke-git add / invoke-git commit in CreateAlGoRepository to suppress excessive git output during repo initialization.

✅ Checklist

• Add tests (E2E, unit tests) — updates the BCApps E2E scenario, test run: https://github.com/microsoft/AL-Go/actions/runs/29086533488

[Copilot]

Pull request overview

Simplifies the BCApps E2E scenario to focus on successful CI/CD and PR Build execution.

Changes:

• Limits BCApps builds to application projects and removes artifact-count assertions.
• Improves E2E scenario filtering.
• Suppresses repository initialization Git output.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
e2eTests/scenarios/BCApps/runtest.ps1	Simplifies BCApps workflow validation.
e2eTests/e2eTestHelper.psm1	Silences initialization Git commands.
.github/workflows/E2E.yaml	Corrects filtering and array handling.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.