Skip to content

Aspire App Host testing#91

Open
frasermclean wants to merge 4 commits into
mainfrom
feature/app-host-testing
Open

Aspire App Host testing#91
frasermclean wants to merge 4 commits into
mainfrom
feature/app-host-testing

Conversation

@frasermclean

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings May 29, 2025 03:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds integration tests for the Aspire App Host health endpoint along with necessary test fixtures and project configuration updates. Key changes include:

  • A new test file for the health endpoint.
  • A dedicated DistributedApplicationFixture to initialize and manage the distributed application.
  • Updates in test project configuration and solution file to include the new test project.

Reviewed Changes

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

Show a summary per file
File Description
backend/tests/AppHost.Tests/HealthEndpointTests.cs Introduces a health endpoint test ensuring an OK status response.
backend/tests/AppHost.Tests/GlobalUsings.cs Adds global using directives for Shouldly and Xunit.
backend/tests/AppHost.Tests/DistributedApplicationFixture.cs Provides test fixture setup for managing distributed application state.
backend/tests/AppHost.Tests/AppHost.Tests.csproj Configures the test project with necessary package references.
backend/RateMyPet.slnx Updates the solution file to include the new AppHost test project.
backend/Directory.Packages.props Adds the Aspire.Hosting.Testing package version update.

@github-actions

github-actions Bot commented May 29, 2025

Copy link
Copy Markdown

Test results

13 tests  ±0   13 ✅ ±0   1s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 3 files   ±0    0 ❌ ±0 

Results for commit 5caf2d4. ± Comparison against base commit 5e0e5a9.

♻️ This comment has been updated with latest results.

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.

2 participants