Skip to content

Commit e6ca15c

Browse files
test: update empty-table test to expect not-equivalent after vacuous guard
The test previously expected empty databases to return equivalent=True, which was the exact bug being fixed. Updated to assert equivalent=False. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 9bb7722 commit e6ca15c

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/test_languages/test_java/test_comparator.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1153,7 +1153,7 @@ def test_comparator_nan_handling(
11531153
def test_comparator_empty_table(
11541154
self, tmp_path: Path, create_test_results_db
11551155
):
1156-
"""Empty test_results tables should result in equivalent=True."""
1156+
"""Empty test_results tables should result in equivalent=False (vacuous equivalence guard)."""
11571157
original_path = tmp_path / "original.db"
11581158
candidate_path = tmp_path / "candidate.db"
11591159

@@ -1163,8 +1163,8 @@ def test_comparator_empty_table(
11631163

11641164
equivalent, diffs = compare_test_results(original_path, candidate_path)
11651165

1166-
# No rows to compare, so they should be equivalent
1167-
assert equivalent is True
1166+
# No rows means no actual comparisons were performed — reject as not equivalent
1167+
assert equivalent is False
11681168
assert len(diffs) == 0
11691169

11701170
def test_comparator_infinity_handling(

0 commit comments

Comments
 (0)