kamiazya/scopes

Enhance Konsist rules for broader test framework support

Offen

#127 geöffnet am 31.08.2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Kotlin (3 Forks)auto 404
enhancementgood first issuehacktoberfest

Repository-Metriken

Stars
 (2 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Background

CodeRabbit review of PR #119 suggested several enhancements to our Konsist architecture validation rules to make them more comprehensive and reduce false positives.

Suggested Improvements

DatabaseTestingRulesTest.kt

  1. Broaden lifecycle hooks detection

    • Support Kotest beforeSpec/afterSpec and JUnit @BeforeAll/@AfterAll
    • Recognize use {} and try/finally patterns for resource closing

  2. Include other SqlDelight drivers

    • Support NativeSqliteDriver and AndroidSqliteDriver
    • Improve in-memory database detection patterns

  3. Better property state detection

    • Use anchored regex for var/val detection
    • Avoid false positives from comments

EventSourcingArchitectureTest.kt

  1. Detect injected clocks

    • Support clock.now() pattern for dependency-injected clocks
    • Not just Clock.System.now()

  2. Allow negative test cases

    • Permit future timestamps when test explicitly asserts failure
    • Detect shouldThrow, shouldBeLeft, etc.

  3. Support virtual time testing

    • Recognize TestClock, FixedClock usage
    • Support coroutine test utilities

General Improvements

  • File-level enforcement instead of per-class for some rules
  • Consider DB-level constraints as valid validation strategy
  • Narrow scope to domain packages to avoid framework classes

Priority

Low - Current rules work well, these are enhancements for edge cases

Related

  • PR #119 (initial Konsist rules implementation)
  • Issue #125 (runTest migration)

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandenen Konsist Regeln in PR #119 und zugehörigen Testdateien. Implementieren Sie breitere Lebenszyklus Hook Muster (Kotest beforeSpec/afterSpec, JUnit @BeforeAll/@AfterAll), unterstützen Sie zusätzliche SqlDelight Treiber, verbessern Sie die Eigenschaftszustandserkennung mit verankerten Regex, unterstützen Sie injizierte Uhren (clock.now()), erlauben Sie negative Testfälle durch Erkennung von shouldThrow und erkennen Sie virtuelle Zeittests mit TestClock/FixedClock. Erkunden Sie die dateibasierte Durchsetzung und die Eingrenzung auf Domain Pakete. Siehe Issue #125 für den Kontext der runTest Migration.
Tech Stack
kotlin
Domain
testing
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
KotlinKonsist
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70