Open Bug 1744574 Opened 3 years ago Updated 18 days ago

Update IOUtils tests to use decorators for setup and cleanup, a la Normandy tests

Categories

(Core :: XPCOM, task, P4)

Product:
Core
Component:
XPCOM
Type:
task

Tracking

()

People

(Reporter: beth, Unassigned)

Details

IOUtils tests call cleanup() at the end of their tests, but if that test uses Assert and that assert fails, the test will exit before running cleanup, leaving temp files on disk.

TIL registerCleanupFunction only runs at the end of all tasks in a test, so we don't want to do this. Instead we want to wrap tests in try { ... } finally {}

Component: OS.File → IOUtils and PathUtils
Summary: Update IOUtils tests to use registerCleanupFunction → Update IOUtils tests to use decorators for setup and cleanup, a la Normandy tests
Assignee: brennie → nobody
Priority: P3 → P4

Mass move of open Toolkit :: IOUtils and PathUtils bugs to Core :: XPCOM

Component: IOUtils and PathUtils → XPCOM
Product: Toolkit → Core
You need to log in before you can comment on or make changes to this bug.