Closed Bug 1120736 Opened 11 years ago Closed 5 years ago

Gecko profiler needs some tests

Categories

(Core :: Gecko Profiler, defect)

Product:
Core
Component:
Gecko Profiler
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: n.nethercote, Unassigned)

Details

The Gecko profiler appears to have no tests. At least, I landed a change that caused it to reliably crash at start-up on Win64 and it took aklotz to notice it (see bug 1120126). Some kind of basic smoke test to detect major bustage like this would be nice.
We have a verity of tests for the backend. We have GTest unit test (part of the motivation for me introducing it), mochitest and xpcshell which provide integrations test. Unless you want to make this more specific there's nothing useful to be done here. But right now Win64 is not supported by the profiler because of a lack of resources.

Lots of tests have been written since 2015, so I'm marking this generic "needs some tests" bug as works-for-me.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INACTIVE
Resolution: INACTIVE → WORKSFORME
You need to log in before you can comment on or make changes to this bug.