Ensure Photon-only tests are enabled past Nightly before uplift

RESOLVED FIXED in Firefox 57

Status

()

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mconley, Assigned: RyanVM)

Tracking

(Blocks 1 bug)

unspecified
Firefox 57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox55 disabled, firefox56 disabled, firefox57+ fixed)

Details

Attachments

(1 attachment)

I've started to spot tests that are Photon-specific. Here's one:

https://hg.mozilla.org/mozilla-central/file/15688bfe56c9/browser/base/content/test/urlbar/browser_page_action_menu.js

I only spotted this because I've started to do periodic merges from central to cedar with Photon disabled to ensure we've not accidentally broken things in the disabled state.

I suspect we'll start to see more and more of these things as Photon ramps up.

This is the sort of thing that can burn us in a silly way if we're not careful - we need to ensure that these tests get enabled _past_ Nightly once we're sure Photon is ready to be uplifted and enabled by default on Beta.

Perhaps that means disabling tests with a photon_build thing instead of nightly_build.
[Tracking Requested - why for this release]:

We need to ensure that this happens before the 57 uplift to beta, or else we're going to risk missing out on some test coverage here.
https://dxr.mozilla.org/mozilla-central/search?q=path:browser/base/content/test%20nightly_build

Looks like it's just those two tests? I can run them in my next uplift simulation and see what happens.
Flags: needinfo?(ryanvm)
Looks good on the uplift simulations. Did some manifest tidying too since I'm OCD like that.
Assignee: nobody → ryanvm
Status: NEW → ASSIGNED
Flags: needinfo?(ryanvm)
Attachment #8906870 - Flags: review?(mconley)
Comment on attachment 8906870 [details] [diff] [review]
remove some obsolete annotations

Review of attachment 8906870 [details] [diff] [review]:
-----------------------------------------------------------------

If try is happy, then so am I! Thanks, RyanVM!
Attachment #8906870 - Flags: review?(mconley) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/247dc838b340
Remove some obsolete nightly_only & release_or_beta test manifest annotations. r=mconley
https://hg.mozilla.org/mozilla-central/rev/247dc838b340
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
You need to log in before you can comment on or make changes to this bug.