Deleting unnecessary files. Rename projects#16
Conversation
There was a problem hiding this comment.
Pull request overview
This PR cleans up the repository by removing an entire legacy Blazor Server + REST API architecture (Client, Application, Infrastructure, Domain, Shared projects) that was previously part of the solution. In its place, it restructures the test infrastructure for the remaining WinUI desktop app (FIP.App/FIP.Core), renaming old test projects and adding a new WinUI UI test project, along with a new CI pipeline configuration.
Changes:
- Deletes all legacy web-stack projects (Client, Application, Infrastructure, Domain, Shared, Client.Infrastructure) and their associated files (migrations, controllers, services, etc.)
- Renames/restructures test projects:
FIP.Core.UnitTests→FIP.UnitTests(with many newFIPColorTestsxUnit tests) andFIP.Core.xUnitTests→FIP.UITests(new WinUI-based MSTest UI test project) - Replaces the old
azure-pipelines.ymlwith a new.github/workflows/azure-pipelines.ymland updates the solution file accordingly
Reviewed changes
Copilot reviewed 109 out of 182 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
src/FIP.UnitTests/FIPColorTests.cs |
New comprehensive xUnit tests for FIPColor model |
src/FIP.UnitTests/FIP.UnitTests.csproj |
Fixed project reference path from src\FIP.Core to FIP.Core |
src/FIP.UITests/* |
New WinUI-based UI test project with MSTest framework, assets, and manifests |
.github/workflows/azure-pipelines.yml |
New CI pipeline replacing the deleted root-level azure-pipelines.yml |
FolderIconPainter.sln |
Solution updated to replace old test project GUIDs with new renamed projects |
.gitignore |
Removed [Pp]ublishProfiles/ exclusion to allow publish profiles to be tracked |
| All deleted legacy files | Removal of entire web/server-side stack no longer needed for this desktop app |
Files not reviewed (2)
- src/Infrastructure/Migrations/20220222172859_Initial.Designer.cs: Language not supported
- src/Infrastructure/Migrations/20221006160054_UpdateConfigurations.Designer.cs: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 109 out of 182 changed files in this pull request and generated 1 comment.
Files not reviewed (2)
- src/Infrastructure/Migrations/20220222172859_Initial.Designer.cs: Language not supported
- src/Infrastructure/Migrations/20221006160054_UpdateConfigurations.Designer.cs: Language not supported
Comments suppressed due to low confidence (1)
src/FIP.UnitTests/FIP.UnitTests.csproj:25
- The
FIP.UnitTestsproject'sFIP.UnitTests.csprojhad its project reference path updated from..\src\FIP.Core\FIP.Core.csprojto..\FIP.Core\FIP.Core.csproj. This is correct since the project is now located atsrc/FIP.UnitTests/, so..already points tosrc/.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.