Skip to content

Conversation

egonw
Copy link
Member

@egonw egonw commented Mar 16, 2025

No description provided.

@egonw egonw added the cdk2024 label Mar 16, 2025
@egonw egonw requested a review from johnmay March 16, 2025 13:59
@@ -191,6 +191,14 @@ void testDestroyBondOrder_IBond_Order() {
Assertions.assertEquals(4.0, BondManipulator.destroyBondOrder(Order.QUADRUPLE), 0.00001);
Assertions.assertEquals(5.0, BondManipulator.destroyBondOrder(Order.QUINTUPLE), 0.00001);
Assertions.assertEquals(6.0, BondManipulator.destroyBondOrder(Order.SEXTUPLE), 0.00001);

// above code is deprecated in favor of this:
Assertions.assertEquals(1.0, Order.SINGLE.numeric(), 0.00001);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't these return integers?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If not now this is likely to flag something at some point

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, good point. Sorry about that; still very tired it seems :( I will fix this!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in 7a55e8c

@egonw egonw assigned egonw and unassigned johnmay Mar 16, 2025
Copy link

@johnmay johnmay merged commit f286e00 into main Mar 17, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants