Skip to content

Commit cb16b10

Browse files
georges-armTamarChristinaArm
authored andcommitted
Fix highlighting in async daily report.
Deltas in the number of tests run should be highlighted. The initial async-report implementation forgot to do that, so this commit adds it! Change-Id: I64c3a6958f33b31d242180249316bc67da39e60b
1 parent 26e9323 commit cb16b10

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

lnt/server/ui/templates/reporting/daily_report_async.html

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -371,11 +371,14 @@ <h3>Result Table ({{ field.display_name }})</h3>
371371
rmo_elem.innerHTML = '<a href="{{ ts_url }}/' + key_run.id + '">' + key_run.llvm_project_revision + '</a>'
372372
}
373373
}
374+
const nr_tests_for_machine = res.data.nr_tests_table['' + machine.id];
374375
for (var i=0; i<{{ report.num_prior_days_to_include }}; i++) {
375376
const nots_td_id = "nots-" + machine.id + "-" + i;
376-
const nots_val = res.data.nr_tests_table['' + machine.id][i];
377+
const nots_val = nr_tests_for_machine[i];
377378
const nots_elem = document.getElementById(nots_td_id);
378-
nots_elem.style.backgroundColor = "";
379+
if (nr_tests_for_machine[i + 1] === undefined || nr_tests_for_machine[i + 1] == nots_val) {
380+
nots_elem.style.backgroundColor = "";
381+
}
379382
nots_elem.innerHTML = '' + nots_val;
380383
}
381384
var today_key_run = res.data.machine_runs['' + machine.id]["0"];

0 commit comments

Comments
 (0)