Skip to content

Commit 8685938

Browse files
committed
tests: apply AI comments
1 parent 831e94f commit 8685938

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

math/fixed_point/tests/sd29x9_tests/arithmetic_tests.move

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,9 @@ fun sub_handles_edge_cases() {
5555
expect(zero.sub(min_plus_epsilon), max);
5656
expect(zero.sub(max), min_plus_epsilon);
5757

58-
expect(max.negate().add(epsilon.negate()), min);
59-
expect(max.sub(epsilon).add(epsilon), max);
60-
expect(min.add(epsilon).add(epsilon).negate().add(epsilon), max);
58+
expect(max.negate().sub(epsilon), min);
59+
expect(max.sub(epsilon).sub(epsilon.negate()), max);
60+
expect(min.add(epsilon).add(epsilon).negate().sub(epsilon.negate()), max);
6161
}
6262

6363
#[test, expected_failure(abort_code = sd29x9_base::EOverflow)]

math/fixed_point/tests/sd29x9_tests/mul_tests.move

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ fun mul_handles_multiplication_by_zero() {
2121
neg(5 * SCALE + 250_000_000),
2222
pos(500_000_000_000_000_000),
2323
neg(500_000_000_000_000_000),
24+
pos(MAX_POSITIVE_VALUE),
25+
neg(MIN_NEGATIVE_VALUE),
2426
];
2527
values.destroy!(|val| {
2628
expect(val.mul(zero), zero);

0 commit comments

Comments
 (0)