detekt report

Total: 5

complexity: 5

ComplexCondition: 2 Complex conditions should be simplified and extracted into well-named methods if necessary.

semver-core/common/test/kotlin/com/javiersc/semver/VersionTest.kt:103:17This condition is too complex (6). Defined complexity threshold for conditions is set to '4'

 100     fun stage_name_comparator() = runTest {
 101         forAll(versionArbitrary, versionArbitrary) { a: Version, b: Version ->
 102             if (
 103                 (a.major == b.major) &&
 104                     (a.minor == b.minor) &&
 105                     (a.patch == b.patch) &&
 106                     (a.stage?.name != null) &&

semver-core/common/test/kotlin/com/javiersc/semver/VersionTest.kt:132:17This condition is too complex (9). Defined complexity threshold for conditions is set to '4'

 129     fun stage_num_comparator() = runTest {
 130         forAll(versionArbitrary, versionArbitrary) { a: Version, b: Version ->
 131             if (
 132                 (a.major == b.major) &&
 133                     (a.minor == b.minor) &&
 134                     (a.patch == b.patch) &&
 135                     (a.stage?.name != null) &&

CyclomaticComplexMethod: 1 Prefer splitting up complex methods into smaller, easier to test methods.

Documentation

semver-core/common/test/kotlin/com/javiersc/semver/VersionTest.kt:100:9The function stage_name_comparator appears to be too complex based on Cyclomatic Complexity (complexity: 17). Defined complexity threshold for methods is set to '15'

  97     }
  98 
  99     @Test
 100     fun stage_name_comparator() = runTest {
 101         forAll(versionArbitrary, versionArbitrary) { a: Version, b: Version ->
 102             if (
 103                 (a.major == b.major) &&

LargeClass: 1 One class should have one responsibility. Large classes tend to handle many things at once. Split up large classes into smaller classes that are easier to understand.

Documentation

semver-core/common/test/kotlin/com/javiersc/semver/VersionTest.kt:24:7Class VersionTest is too large. Consider splitting it into smaller pieces.

  21 import kotlin.test.Test
  22 import kotlinx.coroutines.test.runTest
  23 
  24 class VersionTest {
  25 
  26     private val major = Arb.positiveInt(11)
  27     private val minor = Arb.positiveInt(11)

LongMethod: 1 One method should have one responsibility. Long methods tend to handle many things at once. Prefer smaller methods to make them easier to understand.

Documentation

semver-core/common/test/kotlin/com/javiersc/semver/VersionTest.kt:515:9The function is_version is too long (156). The maximum length is 60.

 512     }
 513 
 514     @Test
 515     fun is_version() {
 516         Version("1.0.0").apply {
 517             isAlpha.shouldBeFalse()
 518             isNotAlpha.shouldBeTrue()

Metrics

Complexity Report

Findings

complexity: 5