Found 2026-08-03 while adding regression tests for BRISK/AKAZE (see the opencv5 migration issue, #44). tests/test_image_point_features.py's test_sift, test_surf, test_orb, test_corners, and test_feature_properties all wrap the actual detector call in a bare try: ... except: pass. This means a completely broken detector (e.g. the exact AttributeError: BRISK_create not found bug fixed in #44) would make these tests pass silently instead of failing -- confirmed by construction while writing test_brisk/test_akaze without that pattern: reverting the BRISK/AKAZE fix makes the new tests fail loudly (as expected), but the same revert applied to SIFT/ORB would not be caught by the existing tests at all.
Fix
Remove the bare except: pass from these five tests; assert the real expected outcome directly (matching the pattern used for the new test_brisk/test_akaze). If a specific environment genuinely can't run one of these (e.g. SURF requiring a non-free build flag), skip explicitly with self.skipTest(reason) rather than swallowing all exceptions.
Found 2026-08-03 while adding regression tests for BRISK/AKAZE (see the opencv5 migration issue, #44).
tests/test_image_point_features.py'stest_sift,test_surf,test_orb,test_corners, andtest_feature_propertiesall wrap the actual detector call in a baretry: ... except: pass. This means a completely broken detector (e.g. the exactAttributeError: BRISK_create not foundbug fixed in #44) would make these tests pass silently instead of failing -- confirmed by construction while writingtest_brisk/test_akazewithout that pattern: reverting the BRISK/AKAZE fix makes the new tests fail loudly (as expected), but the same revert applied to SIFT/ORB would not be caught by the existing tests at all.Fix
Remove the bare
except: passfrom these five tests; assert the real expected outcome directly (matching the pattern used for the newtest_brisk/test_akaze). If a specific environment genuinely can't run one of these (e.g. SURF requiring a non-free build flag), skip explicitly withself.skipTest(reason)rather than swallowing all exceptions.