Skip to content

Commit 3f2ae34

Browse files
dfa1claude
andcommitted
test(core): assert list size before allSatisfy in DTypeIsUnsignedTest (S5841)
allSatisfy passes vacuously on an empty list; assert hasSize(7) first so the test cannot silently pass if the fixture list is ever emptied. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent a1877af commit 3f2ae34

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

core/src/test/java/io/github/dfa1/vortex/core/model/DTypeIsUnsignedTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,6 @@ void nonPrimitiveTypes_areNotUnsigned() {
3333
new DType.Struct(List.of("u"), List.of(DType.U64), false));
3434

3535
// When / Then
36-
assertThat(types).allSatisfy(t -> assertThat(t.isUnsigned()).isFalse());
36+
assertThat(types).hasSize(7).allSatisfy(t -> assertThat(t.isUnsigned()).isFalse());
3737
}
3838
}

0 commit comments

Comments
 (0)