Cleanup: restore the interrupt flag in TestRail / Xray clients

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#256 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
84/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
java
Domaine
testing-qa

Piste de recherche

Commence dans src/main/java/com/seleniumboot/testmanagement/TestRailClient.java vers la ligne 79, puis dans XrayClient.java aux lignes 90 et 110, où InterruptedException est interceptée et ignorée. À chacun de ces endroits, ajoute un catch séparé pour InterruptedException qui appelle Thread.currentThread().interrupt() et conserve le traitement existant. C'est terminé lorsque ces endroits rétablissent le drapeau d'interruption, que rien d'autre n'a changé et que mvn test réussit.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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/testmanagement/TestRailClient.java: line 79
  • src/main/java/com/seleniumboot/testmanagement/XrayClient.java: lines 90, 110

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.

Langage dominant
Java
Étoiles
12
Forks
6
Merge moyen
10 h
PR mergées (30 j)
76

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de seleniumboot/selenium-boot

Toutes les issues de seleniumboot/selenium-boot

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.