dequelabs/axe-core

axe-core timeout with mock timer in JS unit tests

Ouverte

#3 055 ouverte le 30 juin 2021

Voir sur GitHubGuide contributeur
 (9 commentaires) (6 réactions) (0 personne assignée)JavaScript (783 forks)batch import
help wanted

Métriques du dépôt

Stars
 (5 457 étoiles)
Métriques de merge PR
 (Merge moyen 6j) (21 PRs mergées en 30 j)

Description

Product: axe-core

Expectation: axe-core does not timeout when timer is mocked in JS unit tests

Actual: When mock timers are used with axe (e.g. jest.useFakeTimers()), tests fail with error "Timeout - Async callback was not invoked within the 5000ms timeout specified by jest.setTimeout.Timeout"

Motivation: Mocking timers are common in JS unit tests. Getting axe-core to work as expected even when mock timers are being used would be great. The timeout issue occurs even with other third-party mock timer libs such as @sinonjs/fake-timers - so it is not a problem just with Jest.

Something about mocking timers results in axe timeout - not sure why ?

Guide contributeur

Direction de recherche
Enquêtez sur la façon dont axe core utilise les minuteurs en interne. Suivez les opérations asynchrones qui reposent sur des minuteurs réels (par exemple, setTimeout, setInterval) et identifiez où les minuteurs simulés les bloquent. Explorez s’il existe un moyen de détecter les faux minuteurs et de les contourner, ou proposez d’utiliser process.nextTick ou des Promesses à la place des minuteurs pour les délais d’attente internes.
Stack technique
javascriptjest
Domaine
testing
Type d'issue
Bug
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
JavaScriptJest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65