Open Bug 1196106 Opened 5 years ago Updated 4 years ago

Replace most xpcshell unit tests in ImageLib with gtests

Categories

(Core :: ImageLib, defect)

defect
Not set
normal

Tracking

()

People

(Reporter: seth, Assigned: seth)

Details

(Keywords: feature)

Debugging xpcshell tests is much more painful than debugging gtests, and the test harness is much less pleasant to use. Let's convert as many ImageLib xpcshell tests as possible to gtests.

I do see some value in keeping a minimal number of xpcshell tests around just to verify that we don't break APIs that addons are using. I don't see any useful role beyond that, though. Any unit tests that verify deeper aspects of behavior would definitely be better implemented as gtests.
Keywords: feature
You need to log in before you can comment on or make changes to this bug.