Cleanup: restore the interrupt flag in AiFailureAnalyzer

Aperta Adatta ai principianti
  • Qualcosa non va?
#258 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@Rishankgupta08 ci sta già lavorando.

Dal 14/11/2026.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
java
Ambito
testing

Direzione di ricerca

Inizia da src/main/java/com/seleniumboot/ai/AiFailureAnalyzer.java nei due punti indicati, alle righe 117 e 219, e individua i blocchi catch che ingoiano InterruptedException. In ciascuno, chiama Thread.currentThread().interrupt() prima della gestione esistente, separando un catch dedicato a InterruptedException dove i blocchi sono combinati. È completato quando entrambi i punti ripristinano il flag, il resto del comportamento resta invariato e mvn test passa, con la modifica limitata a questo package.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Java
Stelle
12
Fork
6
Merge medio
10h
PR unite (30g)
76

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di seleniumboot/selenium-boot

Tutte le issue di seleniumboot/selenium-boot

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.