victorb/autochecker

Proper cleanup

Open

#4 opened on Apr 13, 2016

View on GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (601 stars) (22 forks)batch import
enhancementhelp wanted

Description

Right now, autochecker doesn't clean up after itself when it finishes running all the tests.

  • os.tmpdir() contains one copy per nodejs version and commit version of the project.
  • Docker containers are also not cleaned up.

Contributor guide

Tech stack
javascriptdocker
Domain
testing
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Node.jsDocker
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
85
Research direction
Identify the main test runner module (likely a .js file) where temporary files are created under os.tmpdir() and Docker containers are launched. Add cleanup routines to remove these artifacts after all tests complete. The issue lists two specific actions: delete the per version directories from os.tmpdir() and stop/remove any leftover Docker containers.