diffplug / diffplug/selfie

Ignore comments within String constants

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

Beschreibung

This code is too simple.

https://github.com/diffplug/selfie/blob/6c3d0ee489152ff87aff5251b05e75314491ad3c/selfie-lib/src/commonMain/kotlin/com/diffplug/selfie/guts/CommentTracker.kt#L51-L54

https://github.com/diffplug/selfie/blob/6c3d0ee489152ff87aff5251b05e75314491ad3c/selfie-lib/src/commonMain/kotlin/com/diffplug/selfie/guts/SourceFile.kt#L83-L87

For example:

```java
class Test {
@Test public void example() {
expectSelfie("underTest").toBe("""
This is a string constant
// selfieonce
The comment above is not really a comment.
We effectively have a "forbidden value" that can't be inside inline snapshots.
""")
}
}
```

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit den Zeilen 51-54 in CommentTracker.kt und den Zeilen 83-87 in SourceFile.kt und reproduziere dann das Java-Textblock-Beispiel, in dem // selfieonce innerhalb einer String-Konstanten erscheint. Verfolge, wie Kommentare erkannt werden, und überprüfe, dass kommentarähnlicher Text innerhalb von String-Konstanten weder als Kommentar noch als unzulässiger Snapshot-Wert behandelt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java, kotlin
Bereich
testing
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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