From 5721a324c1989e42448f62b65ca48c0c5f75260b Mon Sep 17 00:00:00 2001
From: Molkobain <lajarige.guillaume@free.fr>
Date: Tue, 18 Oct 2022 16:52:01 +0200
Subject: [PATCH] Tests: Always display test status for better feedback

---
 tests/manual-visual-tests/sanitize_test.php | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/tests/manual-visual-tests/sanitize_test.php b/tests/manual-visual-tests/sanitize_test.php
index 411e863492..cfb7b561fb 100644
--- a/tests/manual-visual-tests/sanitize_test.php
+++ b/tests/manual-visual-tests/sanitize_test.php
@@ -18,7 +18,7 @@ function testSanitize ($sValue, $sType, &$index ){
 	<td>{$sValue}</td>
 	<td class="sanitized_php">{$sSanitizedValue}</td>
 	<td class="sanitized_js"></td>
-	<td class="hasDiff"></td>
+	<td class="status"></td>
 </tr>
 <script>
 var parentTr = $("tr#test{$index}"),
@@ -27,9 +27,15 @@ var parentTr = $("tr#test{$index}"),
 
 parentTr.find("td.sanitized_js").text(sanitizedJs);
 
-if (sanitizedJs !== sanitizedPhp) {
+if (sanitizedJs === sanitizedPhp) {
+	parentTr.find("td.status")
+		.addClass("status-success")
+		.text("OK");
+} else {
 	console.error("difference detected !", "{$sValueEscapedJs}", '{$sType}', sanitizedPhp, sanitizedJs);
-	parentTr.find("td.hasDiff").text("KO");
+	parentTr.find("td.status")
+		.addClass("status-error")
+		.text("KO");
 }
 </script>
 HTML;
@@ -73,7 +79,10 @@ table, tr, td {
 	border-collapse: collapse;
 }
 
-td.hasDiff {
+.status-success {
+	color: green;
+}
+.status-error {
 	color: red;
 }