Closed Bug 1261101 Opened 5 years ago Closed 5 years ago

Add full update snippet contents to the update results dictionary

Categories

(Testing :: Firefox UI Tests, defect)

defect
Not set
normal

Tracking

(firefox49 fixed, firefox50 fixed)

RESOLVED FIXED
mozilla50
Tracking Status
firefox49 --- fixed
firefox50 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

Details

Attachments

(1 file)

In case of failures in update tests related to busted update snippets it would be great to have all the details listed in the log or even better in the update results dictionary as printed at the end of the test.

The URL of the update snippet is getting added here:

https://dxr.mozilla.org/mozilla-central/source/testing/puppeteer/firefox/firefox_puppeteer/api/software_update.py#229

> 'url_aus': self.get_update_url(True),

So we could use urllib or similar to actually retrieve the content of that remote file.
Re-thinking that I feel it would be too much content inside the update results dictionary and lines won't be that readable. I would suggest we output the content via the info logger.
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Ok, I reverted my opinion from above and it should be better to add it to the update result dictionary. With that we can store the whole dict as json file on disk and upload it to taskcluster.
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a4abd7b926c7
Add full update snippet contents to the update results dictionary. r=maja_zf
https://hg.mozilla.org/mozilla-central/rev/a4abd7b926c7
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Would be great to get this test only change onto mozilla-aurora. It will help us a lot with investigation of test failures of update tests.

Please make sure that the patch on bug 1282080 lands first on mozilla-aurora. Thanks.
Whiteboard: [checkin-needed-aurora][needs bug 1282080 first]
https://hg.mozilla.org/releases/mozilla-aurora/rev/e2a02d7d4e71
Whiteboard: [checkin-needed-aurora][needs bug 1282080 first]
You need to log in before you can comment on or make changes to this bug.