saveourtool/save-cloud

Improve testing of background tasks

Open

#240 ouverte le 27 août 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (1 assigné)Kotlin (4 forks)github user discovery
enhancementgood first issue

Métriques du dépôt

Stars
 (38 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

We have a lot of cases, when request handler returns and triggers a background job, i.e. subscribes to a Publisher inside it's doOnSuccess. In tests to verify that backgound jobs are working correctly we usually call Thread.sleep. This is sub-optimal, because jobs take indefinite time and blocking increases duration of tests.

Guide contributeur

Direction de recherche
Remplacez Thread.sleep par un mécanisme plus robuste, comme l'utilisation du support asynchrone des frameworks de test ou des coroutines pour attendre la fin des tâches en arrière plan. Envisagez d'utiliser `kotlinx.coroutines.test` ou des outils similaires.
Stack technique
kotlin
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Kotlintesting experience
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60