Issue:
Currently, besides including Kotlin's assert() method in the Assertions category, we also include the require(), requireNotNull(), check(), and checkNotNull() in this category.
Still, these four methods are different than an assertion. The methods return an IllegalStateException or an IllegalArgumentException instead of an AssertionError.
Another difference is that those four methods can not be disabled through the -ea JVM option like an assertion.
Goal:
Review the classification of those four methods - require(), requireNotNull(), check(), and checkNotNull() - as Assertions. Currently, it may be wrongly classified.
Issue:
Currently, besides including Kotlin's
assert()method in the Assertions category, we also include therequire(),requireNotNull(),check(), andcheckNotNull()in this category.Still, these four methods are different than an assertion. The methods return an
IllegalStateExceptionor anIllegalArgumentExceptioninstead of anAssertionError.Another difference is that those four methods can not be disabled through the
-eaJVM option like an assertion.Goal:
Review the classification of those four methods -
require(),requireNotNull(),check(), andcheckNotNull()- as Assertions. Currently, it may be wrongly classified.