Bug 972324 : Test for changes of the revision value in payloads , r?chutten
MozReview-Commit-ID: HaANh3cySNK
--- a/toolkit/components/telemetry/tests/unit/test_TelemetrySession.js
+++ b/toolkit/components/telemetry/tests/unit/test_TelemetrySession.js
@@ -198,16 +198,22 @@ function checkPayloadInfo(data) {
for (let f in EXPECTED_INFO_FIELDS_TYPES) {
Assert.ok(f in data, f + " must be available.");
let checkFunc = EXPECTED_INFO_FIELDS_TYPES[f];
Assert.ok(checkFunc(data[f]),
f + " must have the correct type and valid data " + data[f]);
}
+ // Check for a valid revision.
+ if (data.revision != "") {
+ const revisionUrlRegEx = /^http[s]?:\/\/hg.mozilla.org(\/[a-z\S]+)+(\/rev\/[0-9a-z]+)$/g;
+ Assert.ok(revisionUrlRegEx.test(data.revision));
+ }
+
// Previous buildId is not mandatory.
if (data.previousBuildId) {
Assert.ok(stringCheck(data.previousBuildId));
}
Assert.ok(ALLOWED_REASONS.find(r => r == data.reason),
"Payload must contain an allowed reason.");