Cleanup: restore the interrupt flag in AiFailureAnalyzer

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#258 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

@Rishankgupta08 arbeitet bereits daran.

Seit 14.11.2026.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
82/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
java
Bereich
testing

Rechercherichtung

Beginne in src/main/java/com/seleniumboot/ai/AiFailureAnalyzer.java an den beiden genannten Stellen, Zeile 117 und Zeile 219, und suche die catch-Blöcke, die InterruptedException verschlucken. Rufe an jeder dieser Stellen vor der bestehenden Behandlung Thread.currentThread().interrupt() auf. Wo die Blöcke zusammengefasst sind, trenne einen eigenen catch-Block für InterruptedException ab. Fertig ist es, wenn beide Stellen das Flag wiederherstellen, das übrige Verhalten unverändert bleibt und mvn test erfolgreich durchläuft, wobei die Änderung auf dieses Package beschränkt bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

good first issue

Part of the cleanup cycle from #251. SonarCloud rule java:S2142 flags these spots.

These places catch InterruptedException (often inside catch (Exception e)) and carry on without restoring the thread's interrupt flag. Anything further up that checks Thread.interrupted() (a parallel runner shutting down, a timeout) never sees the interrupt.

Where (line numbers as of master 2a07bba):

  • src/main/java/com/seleniumboot/ai/AiFailureAnalyzer.java: lines 117, 219

Fix: restore the flag and keep the existing behaviour. For a combined catch, split it out:

} catch (InterruptedException e) {
    Thread.currentThread().interrupt();
    // then the same handling the old catch did (log / wrap / return)
} catch (Exception e) {
    ...
}

Done when: the listed spots restore the interrupt flag, the behaviour is otherwise unchanged, and mvn test passes. Keep this PR to this package. SonarCloud reruns on the PR.

Vorherrschende Sprache
Java
Sterne
12
Forks
6
Ø Merge
10 Std.
Gemergte PRs (30 T.)
76

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus seleniumboot/selenium-boot

Alle Issues in seleniumboot/selenium-boot

Ähnliche Issues

Weitere Issues zu Java

Neue Issues direkt in Ihr Postfach

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