Skip to content

Commit

Permalink
fix: documentation and jacoco reports
Browse files Browse the repository at this point in the history
* fix: adopt documentation
* fix: fix jacoco report with additional jars
  • Loading branch information
triplem authored Mar 9, 2021
1 parent e33b9e4 commit b80434a
Show file tree
Hide file tree
Showing 7 changed files with 226 additions and 98 deletions.
2 changes: 2 additions & 0 deletions README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,5 @@ endif::[]
image:{img-semantic-release}[Semantic Release Badge,link={uri-semantic-release}]
image:{img-sonarcloud}[Sonarcloud Status,link={uri-sonarcloud}]
image:{img-actions}[CI Workflow]

link:https://github.com/users/triplem/packages/container/package/gradle-by-example[docker package]
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ dependencies {
implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8")

// Add additonal dependencies useful for development
//implementation("io.github.microutils:kotlin-logging:2.0.4")
implementation("io.github.microutils:kotlin-logging:2.0.4")
testImplementation("com.willowtreeapps.assertk:assertk-jvm:0.23")
testImplementation(kotlin("test"))
testImplementation(kotlin("test-junit5"))

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ plugins {
}

repositories {
jcenter()
mavenCentral()
}

val revDate = System.getenv()["revdate"] ?: LocalDateTime.now().format(DateTimeFormatter.ofPattern("yyyy-MM-dd"))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,9 @@ val coverageClassPath: Configuration by configurations.creating {
val aggregateJacocoTestReport by tasks.registering(JacocoReport::class) {
additionalClassDirs(coverageClassPath.incoming.artifactView { lenient(true) }.files)
additionalSourceDirs(sourcesPath.incoming.artifactView { lenient(true) }.files)
executionData(coverageDataPath.incoming.artifactView { lenient(true) }.files.filter { it.exists() })
executionData(coverageDataPath.incoming.artifactView { lenient(true) }
.files.filter { it.exists() }
.filter { it.extension == "exec" })

reports {
// xml is usually used to integrate code coverage with
Expand Down
306 changes: 212 additions & 94 deletions documentation/docs/index.adoc

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion modules/list/list.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,5 @@ plugins {
}

dependencies {
testImplementation("com.willowtreeapps.assertk:assertk-jvm:0.23")

}
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,11 @@ package org.javafreedom.list

import assertk.assertThat
import assertk.assertions.*
import mu.KotlinLogging
import kotlin.test.Test

private val logger = KotlinLogging.logger {}

class LinkedListTest {

@Test fun testConstructor() {
Expand All @@ -17,6 +20,8 @@ class LinkedListTest {
@Test fun testAdd() {
val list = LinkedList()

logger.debug { "Just a test for log messages" }

list.add("one")
assertThat(list.size()).isEqualTo(1)
assertThat(list.get(0)).isEqualTo("one")
Expand Down

0 comments on commit b80434a

Please sign in to comment.