From 66613f791ec2817b3de1b924119d582313f528ea Mon Sep 17 00:00:00 2001
From: Chris Ryan <74329397+chris-ryan-square@users.noreply.github.com>
Date: Tue, 20 Jul 2021 11:01:36 +1000
Subject: [PATCH] WIP: Kotlin 1.5 with necessary dokka upgrade
 (cashapp/misk#2055)

* Upgrade mvn publish pluging; fix dokka setup for change

* Attempt to get dokkaGfm to work properly - OOM/Metaspace problem, see: https://github.com/Kotlin/dokka/issues/1405

* Upgrade to Kotlin 1.5

* Upgrade to kotlin 1.5; fix misk-admin config; use --no-daemon --> still OOM/Metaspace :(

* Do Dokka tasks in separate daemons and separate modules; reduce memeory required

* Remove non-existent dir call

* Only do the publish docs when building master
---
 wisp-aws-environment/build.gradle.kts                     | 7 -------
 wisp-client/build.gradle.kts                              | 7 -------
 wisp-config/build.gradle.kts                              | 7 -------
 wisp-config/src/test/kotlin/wisp/config/WispConfigTest.kt | 8 ++++----
 wisp-containers-testing/build.gradle.kts                  | 7 -------
 wisp-deployment-testing/build.gradle.kts                  | 7 -------
 wisp-deployment/build.gradle.kts                          | 7 -------
 wisp-feature-testing/build.gradle.kts                     | 7 -------
 wisp-feature/build.gradle.kts                             | 7 -------
 wisp-logging-testing/build.gradle.kts                     | 7 -------
 wisp-logging/build.gradle.kts                             | 7 -------
 wisp-resource-loader-testing/build.gradle.kts             | 7 -------
 wisp-resource-loader/build.gradle.kts                     | 7 -------
 wisp-ssl/build.gradle.kts                                 | 7 -------
 14 files changed, 4 insertions(+), 95 deletions(-)

diff --git a/wisp-aws-environment/build.gradle.kts b/wisp-aws-environment/build.gradle.kts
index db9d5a0..769d2c9 100644
--- a/wisp-aws-environment/build.gradle.kts
+++ b/wisp-aws-environment/build.gradle.kts
@@ -9,11 +9,4 @@ dependencies {
 
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-client/build.gradle.kts b/wisp-client/build.gradle.kts
index c1a5ccc..8951103 100644
--- a/wisp-client/build.gradle.kts
+++ b/wisp-client/build.gradle.kts
@@ -20,11 +20,4 @@ dependencies {
   implementation(Dependencies.retrofitWire)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-config/build.gradle.kts b/wisp-config/build.gradle.kts
index d5d4246..1ce4bca 100644
--- a/wisp-config/build.gradle.kts
+++ b/wisp-config/build.gradle.kts
@@ -11,11 +11,4 @@ dependencies {
   testImplementation(Dependencies.kotlinTest)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-config/src/test/kotlin/wisp/config/WispConfigTest.kt b/wisp-config/src/test/kotlin/wisp/config/WispConfigTest.kt
index 09b051c..29ece0a 100644
--- a/wisp-config/src/test/kotlin/wisp/config/WispConfigTest.kt
+++ b/wisp-config/src/test/kotlin/wisp/config/WispConfigTest.kt
@@ -107,8 +107,8 @@ internal class WispConfigTest {
     assertEquals("RRRRRRRRRRRRRRRRR", myConfig.aws.AWS_SECRET_ACCESS_KEY.value)
 
     // values should me masked
-    assertNotEquals("AAAAAAAAAAAAAAAA", myConfig.aws.AWS_ACCESS_KEY_ID)
-    assertNotEquals("RRRRRRRRRRRRRRRRR", myConfig.aws.AWS_SECRET_ACCESS_KEY)
+    assertNotEquals("AAAAAAAAAAAAAAAA", myConfig.aws.AWS_ACCESS_KEY_ID.toString())
+    assertNotEquals("RRRRRRRRRRRRRRRRR", myConfig.aws.AWS_SECRET_ACCESS_KEY.toString())
   }
 
   @Test
@@ -125,8 +125,8 @@ internal class WispConfigTest {
     assertEquals("RRRRRRRRRRRRRRRRR", myConfig.AWS_SECRET_ACCESS_KEY.value)
 
     // values should me masked
-    assertNotEquals("AAAAAAAAAAAAAAAA", myConfig.AWS_ACCESS_KEY_ID)
-    assertNotEquals("RRRRRRRRRRRRRRRRR", myConfig.AWS_SECRET_ACCESS_KEY)
+    assertNotEquals("AAAAAAAAAAAAAAAA", myConfig.AWS_ACCESS_KEY_ID.toString())
+    assertNotEquals("RRRRRRRRRRRRRRRRR", myConfig.AWS_SECRET_ACCESS_KEY.toString())
   }
 
   // TODO(chrisryan): add tests to support other formats
diff --git a/wisp-containers-testing/build.gradle.kts b/wisp-containers-testing/build.gradle.kts
index 04e6c52..049566f 100644
--- a/wisp-containers-testing/build.gradle.kts
+++ b/wisp-containers-testing/build.gradle.kts
@@ -13,11 +13,4 @@ dependencies {
   api(project(":wisp-logging"))
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-deployment-testing/build.gradle.kts b/wisp-deployment-testing/build.gradle.kts
index 3a810e2..1ba068a 100644
--- a/wisp-deployment-testing/build.gradle.kts
+++ b/wisp-deployment-testing/build.gradle.kts
@@ -6,11 +6,4 @@ dependencies {
   testImplementation(Dependencies.kotlinTest)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-deployment/build.gradle.kts b/wisp-deployment/build.gradle.kts
index 8892efe..efcf556 100644
--- a/wisp-deployment/build.gradle.kts
+++ b/wisp-deployment/build.gradle.kts
@@ -6,11 +6,4 @@ dependencies {
   testImplementation(project(":wisp-deployment-testing"))
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-feature-testing/build.gradle.kts b/wisp-feature-testing/build.gradle.kts
index e452658..e3a65e6 100644
--- a/wisp-feature-testing/build.gradle.kts
+++ b/wisp-feature-testing/build.gradle.kts
@@ -16,11 +16,4 @@ dependencies {
   testImplementation(Dependencies.kotlinTest)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-feature/build.gradle.kts b/wisp-feature/build.gradle.kts
index dba616b..f8261ec 100644
--- a/wisp-feature/build.gradle.kts
+++ b/wisp-feature/build.gradle.kts
@@ -4,11 +4,4 @@ dependencies {
   implementation(Dependencies.loggingApi)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-logging-testing/build.gradle.kts b/wisp-logging-testing/build.gradle.kts
index 4cea580..8cfe558 100644
--- a/wisp-logging-testing/build.gradle.kts
+++ b/wisp-logging-testing/build.gradle.kts
@@ -11,11 +11,4 @@ dependencies {
   testImplementation(project(":wisp-logging"))
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-logging/build.gradle.kts b/wisp-logging/build.gradle.kts
index a6a4be8..380bd4e 100644
--- a/wisp-logging/build.gradle.kts
+++ b/wisp-logging/build.gradle.kts
@@ -11,11 +11,4 @@ dependencies {
   testImplementation(project(":wisp-logging-testing"))
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-resource-loader-testing/build.gradle.kts b/wisp-resource-loader-testing/build.gradle.kts
index 6779875..654cd31 100644
--- a/wisp-resource-loader-testing/build.gradle.kts
+++ b/wisp-resource-loader-testing/build.gradle.kts
@@ -10,11 +10,4 @@ dependencies {
   testImplementation(Dependencies.kotlinTest)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-resource-loader/build.gradle.kts b/wisp-resource-loader/build.gradle.kts
index 5d80b79..8af2508 100644
--- a/wisp-resource-loader/build.gradle.kts
+++ b/wisp-resource-loader/build.gradle.kts
@@ -9,11 +9,4 @@ dependencies {
   testImplementation(Dependencies.kotlinTest)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")
diff --git a/wisp-ssl/build.gradle.kts b/wisp-ssl/build.gradle.kts
index 28d7247..f23671c 100644
--- a/wisp-ssl/build.gradle.kts
+++ b/wisp-ssl/build.gradle.kts
@@ -10,11 +10,4 @@ dependencies {
   testImplementation(Dependencies.kotlinTest)
 }
 
-afterEvaluate {
-  project.tasks.dokka {
-    outputDirectory = "$rootDir/docs/0.x"
-    outputFormat = "gfm"
-  }
-}
-
 apply(from = "$rootDir/gradle-mvn-publish.gradle")