kafka icon indicating copy to clipboard operation
kafka copied to clipboard

KAFKA-19947 Add Unit test coverage for KafkaCSVMetricsReporter

Open devtrace404 opened this issue 1 month ago • 1 comments

This PR adds comprehensive unit test coverage for the KafkaCSVMetricsReporter class, which previously had zero test coverage.

Test Results

% ./gradlew :core:test --tests kafka.metrics.KafkaCSVMetricsReporterTest
> Task :core:test

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testCsvFileFormat() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testMultipleStopsAreSafe() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testReporterDoesNotStartWhenDisabled() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testReporterStartsWhenEnabled() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testStopAndRestart() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testManualStart() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testMultipleStartsAreSafe() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testCustomPollingInterval() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testOldDirectoryIsDeleted() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testDirectoryCreation() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testDirectoryCreationWithDefaultPath() PASSED

Gradle Test Run :core:test > Gradle Test Executor 3 > KafkaCSVMetricsReporterTest > testReporterCreationAndMBeanName() PASSED
BUILD SUCCESSFUL in 2m 34s
86 actionable tasks: 86 executed

devtrace404 avatar Dec 02 '25 01:12 devtrace404

A label of 'needs-attention' was automatically added to this PR in order to raise the attention of the committers. Once this issue has been triaged, the triage label should be removed to prevent this automation from happening again.

github-actions[bot] avatar Dec 09 '25 03:12 github-actions[bot]