arturobernalg commented on a change in pull request #104:
URL: https://github.com/apache/commons-geometry/pull/104#discussion_r536883746
##########
File path:
commons-geometry-core/src/test/java/org/apache/commons/geometry/core/GeometryTestUtils.java
##########
@@ -68,15 +68,15 @@ public static void assertThrows(final Runnable r, final
Class<?> exceptionType)
public static void assertThrows(final Runnable r, final Class<?>
exceptionType, final String message) {
try {
r.run();
- Assert.fail("Operation should have thrown an exception");
+ Assertions.fail("Operation should have thrown an exception");
Review comment:
Hi @darkma773r ,
I completely agree with you. but, what do you think to do this in a separate
jira? as a upgrade?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
