Skip to content

[IGNORE] playground test #9201

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from
9 changes: 9 additions & 0 deletions .github/workflows/run-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,15 @@ jobs:
echo "TEST_PROJECT_PATH=$projectPath" >> $env:GITHUB_ENV
echo "TEST_ASSEMBLY_NAME=$filenameWithoutExtension" >> $env:GITHUB_ENV

- name: Check Playground manifests
if: ${{ inputs.os == 'ubuntu-latest' && inputs.testShortName == 'Playground' }}
env:
CI: false
run: |
./build.sh -restore -testnobuild &&
for f in `find playground -name '*.AppHost.csproj'`; do echo "Generating manifest for $f"; ./dotnet.sh run --project $f --no-build --publisher manifest --output-path $PWD/$(dirname $f)/aspire-manifest.json; done &&
git diff --exit-code -- playground

- name: Build test project
if: ${{ ! inputs.requiresNugets }}
env:
Expand Down
13 changes: 11 additions & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ jobs:
- name: Build with packages
env:
CI: false
run: ./build.sh -restore -build -ci -pack /bl /p:InstallBrowsersForPlaywright=false /p:SkipTestProjects=true
run: ./build.sh -restore -build -ci -pack -testnobuild /bl /p:InstallBrowsersForPlaywright=false /p:SkipPlaygroundAppBuilds=true

- name: Upload built NuGets
uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
Expand Down Expand Up @@ -134,9 +134,18 @@ jobs:
testShortName: EndToEnd
# EndToEnd is not run on Windows due to missing Docker support
os: ubuntu-latest
testProjectPath: tests/Aspire.EndToEnd.Tests/Aspire.EndToEnd.Tests.csproj
requiresNugets: true

#playground_tests:
#name: Playground ${{ matrix.os }}
#uses: ./.github/workflows/run-tests.yml
#needs: build_packages
#with:
#testShortName: Playground
## Playground is not run on Windows due to missing Docker support
#os: ubuntu-latest
#requiresNugets: true

results: # This job is used for branch protection. It ensures all the above tests passed
if: ${{ always() && github.repository_owner == 'dotnet' }}
runs-on: ubuntu-latest
Expand Down
3 changes: 1 addition & 2 deletions eng/Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@
<ProjectToBuild Include="$(RepoRoot)src\**\*.csproj" Exclude="$(RepoRoot)src\Aspire.ProjectTemplates\templates\**\*.csproj" />
<ProjectToBuild Include="$(RepoRoot)eng\dcppack\**\*.csproj" />
<ProjectToBuild Include="$(RepoRoot)eng\dashboardpack\**\*.csproj" />
<ProjectToBuild Include="$(RepoRoot)playground\**\*.csproj" />

<ProjectToBuild Include="$(RepoRoot)playground\**\*.csproj" Condition="'$(SkipPlaygroundAppBuilds)' != 'true'" />
<!-- `$(SkipTestProjects)` allows skipping test projects from being
included in the build at all. This is useful for cases like when we are
just building the packages, and don't need to build the test projects. -->
Expand Down
Loading