Skip to content

Harden Windows E2E app install flow#635

Merged
StuartFerguson merged 9 commits into
mainfrom
task/#610_update_shared_nugets
Jun 16, 2026
Merged

Harden Windows E2E app install flow#635
StuartFerguson merged 9 commits into
mainfrom
task/#610_update_shared_nugets

Conversation

@StuartFerguson

@StuartFerguson StuartFerguson commented Jun 15, 2026

Copy link
Copy Markdown
Member

The Windows E2E workflow was hard-coded to a single generated Install.ps1 path, which made it brittle when package output location/name changed. This updates the install step to match the navigation workflow’s resilient package discovery and installation behavior.

  • Dynamic package discovery
    • Locates AppPackages from the two expected build outputs instead of assuming one fixed path.
  • Script-first install
    • Runs Install.ps1 when present, preserving the existing packaged install flow.
  • MSIX fallback
    • Falls back to Add-AppxPackage when only the .msix is available.
$candidatePaths = @(
  (Join-Path $env:GITHUB_WORKSPACE "TransactionProcessor.Mobile\AppPackages"),
  (Join-Path $env:GITHUB_WORKSPACE "TransactionProcessor.Mobile\bin\Release\net10.0-windows10.0.19041.0\win-x64\AppPackages")
)

$installScript = Get-ChildItem -Path $appPackagesPath -Recurse -Filter Install.ps1 | Select-Object -First 1
if ($null -ne $installScript) {
  & $installScript.FullName -Force
} else {
  $msixPackage = Get-ChildItem -Path $appPackagesPath -Recurse -Filter *.msix | Select-Object -First 1
  Add-AppxPackage -Path $msixPackage.FullName -ForceApplicationShutdown
}

Migrated all NuGet package versioning to Directory.Packages.props for centralized management. Removed explicit version numbers from .csproj files. Updated tooltip label formatter in ReportsSalesAnalysisPageViewModel to use StackedValue. Commented out TooltipFindingStrategy binding in ReportsBalanceAnalysisPage.xaml for clarity.
@codacy-production

codacy-production Bot commented Jun 15, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 21 complexity · 0 duplication

Metric Results
Complexity 21
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Changed the assertion in ThenTheApplicationCloses to check for RunningInBackground instead of NotRunning. This aligns the test with the current app behavior on the Android platform.
Copilot AI changed the title Centralize NuGet package versions and minor UI fixes Add Windows AppPackages debug listing in navigation workflow Jun 16, 2026
Copilot AI changed the title Add Windows AppPackages debug listing in navigation workflow Fix Windows navigation AppPackages path resolution Jun 16, 2026
Copilot AI changed the title Fix Windows navigation AppPackages path resolution Harden Windows E2E app install flow Jun 16, 2026
This was linked to issues Jun 16, 2026
@StuartFerguson StuartFerguson merged commit a57c713 into main Jun 16, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move Xunit from v2 to v3 Update Shared Nugets

2 participants