Closed Bug 906339 Opened 6 years ago Closed 6 years ago
Remove testing for "Automatic updates" on release builds
https://tbpl.mozilla.org/?tree=Try&rev=708ead144572 is current aurora pushed to try as though it were on beta. Note the rc2 failures, https://tbpl.mozilla.org/php/getParsedLog.php?id=26415724&tree=Try, where testSettingsMenuItems is trying to find the "Automatic updates" section which doesn't exist because on beta and below the custom updater is shut off.
Summary: testSettingsMenuItems will fail when 25 hits mozilla-beta → Remove testing for "Automatic updates" on release builds
Try build (with patch from bug 903535, which needs something similar): https://tbpl.mozilla.org/?tree=Try&rev=b0d95219d095 I just added some build flag checks to the test, not sure if this is the right way to do branch-specific testing.
Philor, what's the case where the Automatic updates setting is hidden? Your try pushes show "late beta" but I'd like to confirm that this is for all beta and release.
It changes when the branding changes (EARLY_BETA_OR_EARLIER was just the tipmost thing in that try push, https://hg.mozilla.org/try/rev/c8d5855a9572 was the part that changed branding), both http://mxr.mozilla.org/mozilla-aurora/source/mobile/android/branding/beta/configure.sh and http://mxr.mozilla.org/mozilla-aurora/source/mobile/android/branding/official/configure.sh unset MOZ_UPDATER.
This patch depends on the patches from bug 900564 for conditional tests.
Green try build when Automatic updates are not built: https://tbpl.mozilla.org/?tree=Try&rev=8e1023fe869f
low risk uplift when ready with risk evaluation and user impact should be good enough.Need not track it.
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox 26
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Unless we want to uplift the test changes  from bug 900564, using #ifdefs seems like the most straightforward way to do conditional testing.  https://hg.mozilla.org/mozilla-central/rev/7b5fe881c650
Attachment #796307 - Flags: review?(gbrown)
Comment on attachment 796307 [details] [diff] [review] Aurora patch: remove testing for auto updates [Approval Request Comment] Bug caused by (feature/regressing bug #): Automatic updates is not built on beta/release User impact if declined: Robocop testSettingsMenuItems will fail when 25 merges to beta Testing completed (on m-c, etc.): locally tested Risk to taking this patch (and alternatives if risky): low - just checks for a single declaration of a build flag String or IDL/UUID changes made by this patch: none
Attachment #796307 - Flags: approval-mozilla-aurora?
Attachment #796307 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Landed on aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/6ec4a999aa87
You need to log in before you can comment on or make changes to this bug.