From ee95d16de527f625ae7eadd34c3f7de6fa4bb046 Mon Sep 17 00:00:00 2001 From: Ignat Beresnev Date: Thu, 9 Feb 2023 21:29:33 +0100 Subject: [PATCH] Disable caching of Dokka's integration tests (#2826) --- integration-tests/build.gradle.kts | 23 ++++++++++++++++++++++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/integration-tests/build.gradle.kts b/integration-tests/build.gradle.kts index 45e1609766..226b4b6f4c 100644 --- a/integration-tests/build.gradle.kts +++ b/integration-tests/build.gradle.kts @@ -21,7 +21,28 @@ subprojects { implementation(project(":integration-tests")) } - val integrationTest by tasks.register("integrationTest") { + /** + * Dokka's integration test task is not cacheable because the HTML outputs + * it produces when running the tests are used for showcasing resulting documentation, + * which does not work well with caching. + * + * At the moment there are two problems that do not allow to make it cacheable: + * + * 1. The task's inputs are such that changes in Dokka's code do not invalidate the cache, + * because it is run with the same version of Dokka ("DOKKA_VERSION") on the same + * test project inputs. + * 2. The tests generate HTML output which is then used to showcase documentation. + * The outputs are usually copied to a location from which it will be served. + * However, if the test is cacheable, it produces no outputs, so no documentation + * to showcase. It needs to be broken into two separate tasks: one cacheable for running + * the tests and producing HTML output, and another non-cacheable for copying the output. + * + * @see [org.jetbrains.dokka.it.TestOutputCopier] for more details on showcasing documentation + */ + @DisableCachingByDefault(because = "Contains incorrect inputs/outputs configuration, see the KDoc for details") + abstract class NonCacheableIntegrationTest : Test() + + val integrationTest by tasks.register("integrationTest") { maxHeapSize = "2G" description = "Runs integration tests." group = "verification"