Closed Bug 1522728 Opened 6 years ago Closed 1 year ago

Build the IPDL C++ unit tests by default on at least some build types

Categories

(Core :: IPC, enhancement, P2)

Product:
Core
Component:
IPC
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: jld, Assigned: jld)

References

Details

Less ambitious than running the IPDL unit tests and having them pass, at a minimum we should at least build them as part of a normal set of builds, to avoid unnoticed breakage like in bug 1521003. In bug 1236635 comment #3, billm suggested doing this for debug builds (where the size increase of libxul won't matter), which is one possibility.

Assignee: nobody → jld
Priority: -- → P2
Summary: Build the IPDL unit tests by default on at least some build types → Build the IPDL C++ unit tests by default on at least some build types
Severity: normal → S3

Since bug 1751995 the unit tests have been ported to gtest, and are built on all tier 1 (and most tier 2?) platforms. Should we resolve this as FIXED?

Edit: not sure why the status changed to unconfirmed, maybe I fat-fingered it?

Status: NEW → UNCONFIRMED
Ever confirmed: false
Flags: needinfo?(jld)
Status: UNCONFIRMED → NEW
Ever confirmed: true

It's been a while since I opened this bug, but yes, I think it counts as fixed.

Status: NEW → RESOLVED
Closed: 1 year ago
Flags: needinfo?(jld)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.