onivim/oni

Automation - CiTests: Make debugging easier

Open

#1 438 ouverte le 3 févr. 2018

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)TypeScript (11 389 stars) (336 forks)batch import
developmenthelp wantedinsider

Description

Debugging the CiTests isn't very straightforward today. Once #1436 and #1437 are addressed, and it's easier to run/iterate on them, this will definitely be the next pain point.

A few possibilities we could have to make this easier:

  • Add a npm run debug:test:integration command, that opens a debug window and stops at a debugger prior to executing the test.
  • All the npm run debug:test:integration to take some sort of filter that is scoped to a particular test (so we don't have to run them all).

Guide contributeur

Stack technique
typescriptnodejs
Domaine
testingdeveloper experience
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Resolve issues #1436 and #1437Understand integration test setup
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45
Direction de recherche
Review the current npm scripts in 'package.json' to understand existing test commands. Examine the test configuration and framework (likely Jest or Mocha) to implement a debug command that uses node's inspect flag. Also look at the discussion in #1436 and #1437 for context on test improvements.