Skip to content

Improve bUnit test reliability#88

Closed
Copilot wants to merge 2 commits into
mainfrom
copilot/improve-bunit-test-reliability
Closed

Improve bUnit test reliability#88
Copilot wants to merge 2 commits into
mainfrom
copilot/improve-bunit-test-reliability

Conversation

Copilot AI commented May 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • give the shared bUnit test context a longer default wait timeout for slower CI runners
  • replace immediate post-click assertions in the category management tests with wait-based assertions
  • give the day-switching persistence test explicit longer waits around async rerenders

Validation

  • dotnet workload restore JournalApp.sln
  • dotnet test JournalApp.Tests/JournalApp.Tests.csproj --no-restore --filter "FullyQualifiedName~JournalApp.Tests.ManageCategoriesTests|FullyQualifiedName~JournalApp.Tests.IndexTests" (10 runs)
  • dotnet test JournalApp.sln --no-restore
  • parallel_validation (code review + CodeQL)

Agent-Logs-Url: https://github.com/danielchalmers/JournalApp/sessions/a94b06ad-ecec-4243-8546-0a5397c67402

Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>
@danielchalmers

Copy link
Copy Markdown
Owner

@copilot remove DefaultUiWaitTimeout and just improve reliability using default WaitForAssertion

Agent-Logs-Url: https://github.com/danielchalmers/JournalApp/sessions/b4b132f8-62be-47f3-bf39-6f8d490d7273

Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>

Copilot AI commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

@copilot remove DefaultUiWaitTimeout and just improve reliability using default WaitForAssertion

Updated in 1c12e85. I removed the shared DefaultUiWaitTimeout override and kept the reliability fix by using the default WaitForAssertion behavior in the affected tests only. No UI change in this update.

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