Skip to content

fix(test): Fix expected results after version bump#466

Merged
blanca-pablos merged 1 commit intomainfrom
fix/update-expected-cell-counts
Mar 6, 2026
Merged

fix(test): Fix expected results after version bump#466
blanca-pablos merged 1 commit intomainfrom
fix/update-expected-cell-counts

Conversation

@blanca-pablos
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 6, 2026 10:52
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

Updates test expectations to match outputs after an application version bump, ensuring the E2E/GUI QuPath workflow assertions remain consistent with current result artifacts.

Changes:

  • Update SPOT_0 expected result file sizes and expected classified-cell count for the staging environment.
  • Adjust QuPath GUI test to report the correct annotation count variable in assertion messages.
  • Reformat imports in the QuPath GUI test (currently not Ruff/Black-compliant).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
tests/constants_test.py Updates staging expected result artifact sizes and expected annotation count for SPOT_0.
tests/aignostics/qupath/gui_test.py Tweaks imports formatting and fixes the assertion message to display spot_annotations.

@blanca-pablos blanca-pablos force-pushed the fix/update-expected-cell-counts branch from da606b8 to db5c570 Compare March 6, 2026 10:55
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.
see 8 files with indirect coverage changes

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2026

@blanca-pablos blanca-pablos merged commit 7dfaf52 into main Mar 6, 2026
24 checks passed
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