From fe360f7fadc7173be3ae170778a93ebd01962149 Mon Sep 17 00:00:00 2001 From: Ignat Beresnev Date: Tue, 31 Jan 2023 18:38:09 +0100 Subject: Make implementations of DokkaLogger thread-safe (#2827) --- .../src/main/kotlin/testApi/logger/TestLogger.kt | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) (limited to 'core/test-api/src') diff --git a/core/test-api/src/main/kotlin/testApi/logger/TestLogger.kt b/core/test-api/src/main/kotlin/testApi/logger/TestLogger.kt index 0bc66a2b..e70200a7 100644 --- a/core/test-api/src/main/kotlin/testApi/logger/TestLogger.kt +++ b/core/test-api/src/main/kotlin/testApi/logger/TestLogger.kt @@ -1,24 +1,28 @@ package org.jetbrains.dokka.testApi.logger import org.jetbrains.dokka.utilities.DokkaLogger +import java.util.Collections +/* + * Even in tests it be used in a concurrent environment, so needs to be thread safe + */ class TestLogger(private val logger: DokkaLogger) : DokkaLogger { override var warningsCount: Int by logger::warningsCount override var errorsCount: Int by logger::errorsCount - private var _debugMessages = mutableListOf() + private var _debugMessages = synchronizedMutableListOf() val debugMessages: List get() = _debugMessages.toList() - private var _infoMessages = mutableListOf() + private var _infoMessages = synchronizedMutableListOf() val infoMessages: List get() = _infoMessages.toList() - private var _progressMessages = mutableListOf() + private var _progressMessages = synchronizedMutableListOf() val progressMessages: List get() = _progressMessages.toList() - private var _warnMessages = mutableListOf() + private var _warnMessages = synchronizedMutableListOf() val warnMessages: List get() = _warnMessages.toList() - private var _errorMessages = mutableListOf() + private var _errorMessages = synchronizedMutableListOf() val errorMessages: List get() = _errorMessages.toList() override fun debug(message: String) { @@ -45,4 +49,6 @@ class TestLogger(private val logger: DokkaLogger) : DokkaLogger { _errorMessages.add(message) logger.error(message) } + + private fun synchronizedMutableListOf(): MutableList = Collections.synchronizedList(mutableListOf()) } -- cgit