detekt/detekt

`OutputReportsSpec.kt` tests has really little sense

Open

#8.625 geöffnet am 8. Sept. 2025

Auf GitHub ansehen
 (1 Kommentar) (1 Reaktion) (0 zugewiesene Personen)Kotlin (834 Forks)batch import
corehelp wantedhousekeeping

Repository-Metriken

Stars
 (6.942 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 4T 9h) (65 gemergte PRs in 30 T)

Beschreibung

Reviewing OutputReportsSpec.kt I'm not sure what those tests are really testing.

  • There is not file/class called OutputReports so they aren't testing that
  • OutputReport is the class more near to it. But those tests never execute nothing related with that.
  • It execute ReportsSpec. But that's an interface with no public/internal implementation.
  • It also executes ReportsSpecBuilder but that code is in detekt-tooling and these tests are en detekt-core.

Also, OutputFacadeSpec only has one test and it's to the ones in OutputReportsSpec.

We need to review this part of the code. Remove or move the tests that have no sense and create real tests to check how the reports (ConsoleReport and OutputReport) work.

Contributor Guide

Research-Richtung
Untersuchen Sie OutputReportsSpec.kt und OutputReports.kt, um Testlücken zu identifizieren. Entfernen oder schreiben Sie dann Tests neu, die sich auf ConsoleReport und OutputReport konzentrieren.
Tech Stack
kotlin
Domain
backendtesting
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
KotlinTesting
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70