Skip to content

Commit 4bdaeb6

Browse files
committed
fix tests on ubunto
1 parent 3bbced1 commit 4bdaeb6

1 file changed

Lines changed: 16 additions & 2 deletions

File tree

.github/workflows/CI.yml

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ jobs:
4545

4646
# Pack so we can validate there are no warnings/errors
4747
# We must explicitly set the configuration parameter otherwise it defaults to Release
48-
# Packages will include all target frameworks (net8.0, net10.0 for Persistence projects)
48+
# Packages will include all target frameworks (net48, net8.0, net10.0 for Persistence projects)
49+
# Note: On Windows, net48 binaries are included; on Ubuntu, net48 build is skipped but pack succeeds
4950
- name: Pack - Formulas.Tools
5051
run: dotnet pack --no-build -c ${{ env.Configuration }} src/PAModel/Microsoft.PowerPlatform.Formulas.Tools.csproj
5152

@@ -56,5 +57,18 @@ jobs:
5657
run: dotnet pack --no-build -c ${{ env.Configuration }} src/Persistence.Testing/Microsoft.PowerPlatform.PowerApps.Persistence.Testing.csproj
5758

5859
# Run tests for all target frameworks
59-
- name: Test - PASopa.sln
60+
# On Windows: run all projects and frameworks together
61+
- name: Test - PASopa.sln (Windows - all frameworks)
62+
if: runner.os == 'Windows'
6063
run: dotnet test --no-build --solution src/PASopa.sln
64+
65+
# On non-Windows: run test projects individually, since they don't all support the same TFMs
66+
- name: Test - Persistence.Tests
67+
if: runner.os != 'Windows'
68+
run: dotnet test --no-build --project src/Persistence.Tests/Persistence.Tests.csproj
69+
- name: Test - PAModelTests
70+
if: runner.os != 'Windows'
71+
run: dotnet test --no-build --project src/PAModelTests/PAModelTests.csproj
72+
- name: Test - YamlValidator.Tests
73+
if: runner.os != 'Windows'
74+
run: dotnet test --no-build --project src/YamlValidator.Tests/YamlValidator.Tests.csproj

0 commit comments

Comments
 (0)