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 b2d6f10 commit 1518949
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 1518949

Please sign in to comment.