Skip to content

Updated classify single functional test for intended behaviour#130

Merged
corpo-iwillspeak merged 2 commits into
mainfrom
feature/classification-short-circuit
Jun 25, 2026
Merged

Updated classify single functional test for intended behaviour#130
corpo-iwillspeak merged 2 commits into
mainfrom
feature/classification-short-circuit

Conversation

@carlpatchett-crispthinking

@carlpatchett-crispthinking carlpatchett-crispthinking commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

This pull request updates the test logic in test_classify_single.py to account for the non-deterministic presence of the WebIQ hash check due to its 250ms execution budget. The test now only asserts the correctness of the classification when a "KnownCSAM-" label is present, rather than requiring its presence.

Test logic improvements:

  • Updated the test to remove the requirement that a "KnownCSAM-" classification must always be present, reflecting that the hash check may be skipped if it exceeds its time budget. Now, the test only asserts that if a "KnownCSAM-" classification is present, its weight must be 0.0.

@tcarroll-kroll

Copy link
Copy Markdown
Member

Why have you changed uv.lock?

@corpo-iwillspeak corpo-iwillspeak merged commit 358aa90 into main Jun 25, 2026
16 checks passed
@corpo-iwillspeak corpo-iwillspeak deleted the feature/classification-short-circuit branch June 25, 2026 08:02
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.

5 participants