-
Notifications
You must be signed in to change notification settings - Fork 0
Upgrade to .NET 8.0 with Aspire Orchestration and Comprehensive Testing Framework #19
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
Conversation
|
@KSemenenko 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review. |
Co-authored-by: KSemenenko <4385716+KSemenenko@users.noreply.github.com>
Co-authored-by: KSemenenko <4385716+KSemenenko@users.noreply.github.com>
I've created comprehensive GitHub Actions workflows that address both issues #17 and #15: CI/CD Pipeline (
E2E Testing (
The workflows are ready to run and will create deployable artifacts for all platforms as specified in the issues. Commit: e3c20a3 |
|
@copilot dotnet 9 and c#13, review changes, |
|
@copilot migrate to slnx. And you are absolutely wrong about tests. Check this https://learn.microsoft.com/en-us/dotnet/aspire/testing/write-your-first-test?pivots=xunit Nice ecspale about how to wire tests. You shouldn’t use aspire dashboard |
Co-authored-by: KSemenenko <4385716+KSemenenko@users.noreply.github.com>
I've successfully migrated the project to .NET 9 and implemented proper Aspire testing as requested: Key Changes Made:
Aspire Testing Implementation:
The traditional Commit: 399e856 Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Co-authored-by: KSemenenko <4385716+KSemenenko@users.noreply.github.com>
|
@copilot resove conflict |
There was a problem hiding this comment.
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 upgrades the entire CodeUI solution from .NET 8 to .NET 9 with C# 13 and implements proper Aspire distributed application orchestration with comprehensive testing infrastructure.
Key changes include:
- Framework upgrade to .NET 9.0 with C# 13 language support across all projects
- Implementation of Aspire hosting for distributed application orchestration
- Addition of comprehensive testing framework with both unit tests (WebApplicationFactory) and integration tests (Aspire testing framework)
Reviewed Changes
Copilot reviewed 35 out of 38 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Framework version discrepancy - references .NET 8 instead of .NET 9 |
| CodeUI.sln | Solution file defining all project structure and build configurations |
| CodeUI.Web/* | Blazor Server web application with .NET 9 packages and components |
| CodeUI.Core/* | Core business logic with Entity Framework and Identity using .NET 9 |
| CodeUI.Orleans/* | Orleans grain interfaces for distributed computing on .NET 9 |
| CodeUI.AppHost/* | Aspire orchestration host for distributed application management |
| CodeUI.Tests/* | Unit tests using WebApplicationFactory pattern |
| CodeUI.AspireTests/* | Integration tests using proper Aspire testing framework |
| .github/workflows/* | CI/CD pipelines updated for .NET 9 with comprehensive testing |
| .github/copilot-instructions.md | Development guidelines and best practices documentation |
Co-authored-by: KSemenenko <4385716+KSemenenko@users.noreply.github.com>
Merge conflict resolved successfully! The issue was in the Key fixes applied:
Validation results:
Commit: 2cd6d3a |
This PR upgrades the entire solution to .NET 8.0 with C# 12 and implements proper Aspire distributed application orchestration with a comprehensive testing framework.
Key Changes
Framework Upgrade
Aspire Integration
Implemented proper Aspire distributed application setup with:
Testing Infrastructure
Package Versions
Updated to .NET 8.0 compatible package versions:
Aspire.Hosting8.2.2 - For application orchestrationMicrosoft.AspNetCore.*8.0.11 - For web application frameworkMicrosoft.EntityFrameworkCore.*8.0.11 - For data accessMicrosoft.Orleans.*8.2.0 - For distributed state managementCI/CD Enhancements
Developer Experience
The application now follows modern .NET 8.0 patterns with proper distributed application orchestration, making it production-ready with comprehensive testing coverage.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.