Closed Bug 1316886 Opened 7 years ago Closed 7 years ago

Port sdp_file_parser unit test to standalone binary


(Core :: WebRTC: Signaling, defect, P2)




Tracking Status
firefox53 --- fixed


(Reporter: dminor, Assigned: dminor)




(2 files)

As it stands, we don't run this test in automation anyway.

It appears the intention is to be able to run it as a standalone executable, point it to a binary file containing sdp and manually verify the output.

If we want to run it in automation, we should come up with a standard input and an expected output. Otherwise, we should see if we can build it as a standalone executable, or just remove it.
Byron, what do you think we should do with this one? Thanks!
Flags: needinfo?(docfaraday)
I _think_ this doesn't use any XPCOM in any meaningful way, so we should be able to build it standalone. We could also build it as a gecko gtest but make its only test-case disabled.
Flags: needinfo?(docfaraday)
Dan I created this binary for the purpose of fuzz testing our SDP parser (which is at least 10 times faster via this binary then going all the way through Firefox). And I would like to keep it around for that purpose (assuming the effort to keep it around is not too high).

And yes Byron is right this should be converted into a standalone binary. I don't think it makes sense to keep this around if get converted into a gecko gtest.
Assignee: nobody → dminor
Summary: Port sdp_file_parser unit test to xul gtest or elsewhere → Port sdp_file_parser unit test to standalone binary
There's a prototype of xul-gtestifying this in bug 1257713. It proposes adding an env var, SDP_FILE_PARSER_FILE, rather than passing the file in via command-line.

Building standalone is probably not an option. I'll attach the colossal linking failure.
Comment on attachment 8810003 [details]
Bug 1316886 - Port sdp_file_parser unit test to standalone binary;

LGTM. Thanks for solving it properly!
Attachment #8810003 - Flags: review?(drno) → review+
Pushed by
Port sdp_file_parser unit test to standalone binary; r=drno
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Blocks: 1317764
No longer blocks: 1317764
Depends on: 1317764
You need to log in before you can comment on or make changes to this bug.