Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: M Schalk <30376729+schalkms@users.noreply.github.com>
  • Loading branch information
BraisGabin and schalkms committed Dec 26, 2021
1 parent 00a2e2e commit c6b0b1b
Showing 1 changed file with 4 additions and 4 deletions.
Expand Up @@ -14,15 +14,15 @@ internal class DefaultConfigProviderSpec : Spek({
createNullLoggingSpec {}
}

it("get") {
it("gets") {
val config = DefaultConfigProvider().apply { init(spec) }.get()

assertThat(config.parentPath).isNull()
assertThat(config.subConfig("build").valueOrNull<Int>("maxIssues")).isEqualTo(0)
assertThat(config.valueOrNull<Any>("sample")).isNull()
}

it("copy") {
it("copies") {
val path = createTempFileForTest("test", "test")
DefaultConfigProvider().apply { init(spec) }.copy(path)

Expand All @@ -40,15 +40,15 @@ internal class DefaultConfigProviderSpec : Spek({
}
}

it("get") {
it("gets") {
val config = DefaultConfigProvider().apply { init(spec) }.get()

assertThat(config.parentPath).isNull()
assertThat(config.subConfig("build").valueOrNull<Int>("maxIssues")).isEqualTo(0)
assertThat(config.valueOrNull<Any>("sample")).isNotNull()
}

it("copy") {
it("copies") {
val path = createTempFileForTest("test", "test")
DefaultConfigProvider().apply { init(spec) }.copy(path)

Expand Down

0 comments on commit c6b0b1b

Please sign in to comment.