diffplug / diffplug/selfie

When env vars have different value depending on case, throw error

Offen
#543 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug jvm py
Vorherrschende Sprache
Kotlin
Sterne
101
Forks
18
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

This has the confusing behavior that variables with a lowercase key trump an uppercase key

https://github.com/diffplug/selfie/blob/e6fb9e6a33b650c72aab3d5459ae1c22ab865cda/jvm/selfie-runner-junit5/src/main/kotlin/com/diffplug/selfie/junit5/SelfieSettingsAPI.kt#L31-L38

I think it's good to be case-insensitive, but if both variables exist and disagree it should generate an easy-to-understand error.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in jvm/selfie-runner-junit5/src/main/kotlin/com/diffplug/selfie/junit5/SelfieSettingsAPI.kt bei den Zeilen 31-38 und untersuche, wie Umgebungsvariablen ausgewählt werden. Stelle den Fall nach, in dem sich Variablen in Klein- und Großschreibung widersprechen, füge dann Tests für den erwarteten Fehler hinzu und führe die relevanten JVM-Tests aus. Erledigt ist dies, wenn widersprüchliche Werte einen leicht verständlichen Fehler erzeugen, statt dass einer davon stillschweigend Vorrang erhält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
kotlin
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.