Skip to content

Commit

Permalink
mark failed tests that are not mandatory more clearly
Browse files Browse the repository at this point in the history
  • Loading branch information
mitxp committed Dec 9, 2014
1 parent 3fce477 commit 4b1f03c
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 4 deletions.
3 changes: 3 additions & 0 deletions base.css
Expand Up @@ -55,4 +55,7 @@ div.statok {
div.statfail {
color: #ff5030;
}
div.statwarn {
color: #ffe550;
}

2 changes: 1 addition & 1 deletion base.js
Expand Up @@ -78,7 +78,7 @@ function menuSelect(i) {

function showStatus(succss, txt) {
var elem = document.getElementById('status');
elem.className = succss ? 'statok' : 'statfail';
elem.className = succss===2 ? 'statwarn' : (succss ? 'statok' : 'statfail');
if (!txt) {
elem.innerHTML = '';
return;
Expand Down
2 changes: 1 addition & 1 deletion clientssl/index.php
Expand Up @@ -62,7 +62,7 @@ function performauth() {
if (authok) {
document.location.href = 'finalcheck.php';
} else {
showStatus(false, 'Client SSL certificate verification failed.');
showStatus(2, 'Client SSL certificate verification failed. As this certificate is not mandatory, this might be expected.');
setInstr(logtxt);
}
};
Expand Down
2 changes: 1 addition & 1 deletion configuration/index.php
Expand Up @@ -40,7 +40,7 @@ function runStep(name) {
return;
}
if (name.length>8 && name.substring(0, 8)==='localSys' && !oipfcfg.localSystem) {
showStatus(false, 'Cannot find .localSystem in application/oipfConfiguration');
showStatus(2, 'Cannot find .localSystem in application/oipfConfiguration. As this attribute is not mandatory in HbbTV, this might be expected.');
return;
}
var config = oipfcfg.configuration;
Expand Down
2 changes: 1 addition & 1 deletion videocontrol/index.php
Expand Up @@ -98,7 +98,7 @@ function checkPlaySpeed(fact) {
showStatus(true, 'Video playback speed should now be '+fact);
} else {
var addmsg = (fact==0||fact==1) ? '' : '. Note: test is OK even though this test failed, as feature is not mandatory.';
showStatus(addmsg, 'Setting speed succeeded, but reported speed is '+vid.speed+addmsg);
showStatus(addmsg?2:false, 'Setting speed succeeded, but reported speed is '+vid.speed+addmsg);
}
}
function gotoPos(scnds) {
Expand Down

0 comments on commit 4b1f03c

Please sign in to comment.