Closed Bug 1631441 Opened 4 years ago Closed 2 years ago

Remove XBL tests

Categories

(Core :: XBL, task)

task

Tracking

()

RESOLVED FIXED
109 Branch
Tracking Status
firefox109 --- fixed

People

(Reporter: ntim, Assigned: marco)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Is there any reason for keeping these tests? Does removing them speed up any aspect of the program? How much of a patch would it be to remove?

Brian, can we remove these tests? It would unlock finding more uncovered files that we could get rid of.

Flags: needinfo?(bgrinstead)

(In reply to Marco Castelluccio [:marco] from comment #2)

Brian, can we remove these tests? It would unlock finding more uncovered files that we could get rid of.

I don't see any reason not to remove them, so long as they are/were only covering XBL functionality and not just accessing moz-bindings as part of a broader test. Though I'd be surprised if these are uniquely providing coverage for other files.

Flags: needinfo?(bgrinstead)
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Pushed by mcastelluccio@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dab79b4f8a22
Remove no longer relevant tests with XBL bindings. r=emilio DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: