Closed Bug 1417657 Opened 2 years ago Closed 2 years ago

Add basic test for prio() to the IPDL unit tests

Categories

(Core :: IPC, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

Details

Attachments

(1 file)

No description provided.
Can you provide a bit more context for this?  Is it an existing feature that we just don't use in the system anywhere?
(In reply to Ben Kelly [:bkelly] from comment #2)
> Can you provide a bit more context for this?  Is it an existing feature that
> we just don't use in the system anywhere?

We use this in real Firefox IPDL files. I just wanted a little test for it in the IPDL unit tests. We don't run those tests automatically, but ideally at some point we would.
Comment on attachment 8930608 [details]
Bug 1417657 - Add basic test for prio().

https://reviewboard.mozilla.org/r/201732/#review207010
Attachment #8930608 - Flags: review?(bkelly) → review+
Another part of the context here is that I am also maintaining an IPDL parser written in Rust ( https://github.com/amccreight/ipdl_parser ), which is used for Searchfox, and it is convenient to have the same set of unit tests for both parsers.
https://hg.mozilla.org/mozilla-central/rev/f515ace4d2da
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.