Closed Bug 1316687 Opened 3 years ago Closed 3 years ago

Make use of the default value for "app.update.url" because "app.update.url.override" was removed

Categories

(Testing :: Firefox UI Tests, defect)

defect
Not set

Tracking

(firefox53 fixed)

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

Attachments

(1 file)

Bug 1315506 will get rid of this preference, so we have to get rid of this option in our firefox-ui update harness. Instead we have to set the default value of "app.update.url".

I will work on this once bug 1315506 has been fixed.
Depends on: 1317270
Summary: Remove option for "app.update.url.override" preference from firefox-ui update tests → Make use of the default value for "app.update.url" because "app.update.url.override" was removed
Comment on attachment 8811044 [details]
Bug 1316687 - Make use of the default value for "app.update.url" because "app.update.url.override" was removed.

https://reviewboard.mozilla.org/r/93282/#review93538

One question. Everything else look OK.

::: testing/firefox-ui/harness/firefox_ui_harness/runners/update.py
(Diff revision 1)
>  
>          self.original_bin = self.bin
>  
>          self.prefs.update(DEFAULT_PREFS)
>  
> -        # In case of overriding the update URL, set the appropriate preference

Don't you want to set app.update.url here, too? (Right now it's set at restart. Is that enough? Does a restart always happen before the tests start?)
Attachment #8811044 - Flags: review?(mjzffr) → review-
Comment on attachment 8811044 [details]
Bug 1316687 - Make use of the default value for "app.update.url" because "app.update.url.override" was removed.

https://reviewboard.mozilla.org/r/93282/#review93538

> Don't you want to set app.update.url here, too? (Right now it's set at restart. Is that enough? Does a restart always happen before the tests start?)

Good question Maja! So we cannot set the url inside the runner, because the first start of Firefox would reset the value. If we want to set the default at the beginning of the test, it would have to be done in setUp() for the update testcase class.

Right now we always restart Firefox before checking for updates. So the above change would not be necessary. But I would like to get rid of this extra restart soon to get the app.update.log working on Windows. As such it will be good to get this code also added in setUp.
Comment on attachment 8811044 [details]
Bug 1316687 - Make use of the default value for "app.update.url" because "app.update.url.override" was removed.

https://reviewboard.mozilla.org/r/93282/#review93978
Attachment #8811044 - Flags: review?(mjzffr) → review+
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/852fa3bf50d7
Make use of the default value for "app.update.url" because "app.update.url.override" was removed. r=maja_zf
https://hg.mozilla.org/mozilla-central/rev/852fa3bf50d7
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Assignee: nobody → hskupin
You need to log in before you can comment on or make changes to this bug.