Skip to content

Conversation

@xnodeoncode
Copy link
Owner

No description provided.

Copy link

Copilot AI left a 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 pull request updates the test project path in the CI workflow from Aquiis.SimpleStart.Tests to 6-Tests/Aquiis.Application.Tests, suggesting a reorganization of the test project structure.

Changes:

  • Updated the test command to point to a new test project location with a numeric prefix directory structure

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


- name: Run focused tests
run: dotnet test Aquiis.SimpleStart.Tests/Aquiis.SimpleStart.Tests.csproj --no-build --configuration Release --verbosity normal
run: dotnet test 6-Tests/Aquiis.Application.Tests/Aquiis.Application.Tests.csproj --no-build --configuration Release --verbosity normal
Copy link

Copilot AI Jan 10, 2026

Choose a reason for hiding this comment

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

The test project path has been updated to reference 6-Tests/Aquiis.Application.Tests/Aquiis.Application.Tests.csproj, but this directory structure doesn't exist in the repository. The current codebase shows Aquiis.SimpleStart.Tests/ exists in the root directory, and the solution file (Aquiis.sln) only references Aquiis.SimpleStart.Tests, not the new Aquiis.Application.Tests project.

Since this step uses the --no-build flag, it depends on the previous build step which builds Aquiis.sln. If the new test project isn't included in the solution file, it won't be built, and this test command will fail with a "project not found" or "no binaries to test" error.

Either the new test project path needs to be added to the solution file, or this CI workflow update is premature and should be reverted to the original path until the project restructuring is complete.

Suggested change
run: dotnet test 6-Tests/Aquiis.Application.Tests/Aquiis.Application.Tests.csproj --no-build --configuration Release --verbosity normal
run: dotnet test Aquiis.SimpleStart.Tests/Aquiis.SimpleStart.Tests.csproj --no-build --configuration Release --verbosity normal

Copilot uses AI. Check for mistakes.
@xnodeoncode
Copy link
Owner Author

Closing PR no longer needed. Changes included in Shared UI Implementation PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants