From ddbd77cca1a024544c593e8a6da035625546f9a8 Mon Sep 17 00:00:00 2001
From: Shannon Booth <shannon@serenityos.org>
Date: Sun, 30 Jul 2023 12:14:00 +1200
Subject: [PATCH] run-tests: Correct logic for calculating number of
 skipped/xfail tests

Skipped tests were mistakenly being considered as expected failures.
---
 Userland/Utilities/run-tests.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Userland/Utilities/run-tests.cpp b/Userland/Utilities/run-tests.cpp
index f208ed846b..06e5a6cea0 100644
--- a/Userland/Utilities/run-tests.cpp
+++ b/Userland/Utilities/run-tests.cpp
@@ -121,10 +121,10 @@ void TestRunner::do_run_single_test(DeprecatedString const& test_path, size_t cu
         ++m_counts.tests_passed;
         break;
     case Test::Result::ExpectedFail:
-        ++m_counts.tests_passed;
+        ++m_counts.tests_expected_failed;
         break;
     case Test::Result::Skip:
-        ++m_counts.tests_expected_failed;
+        ++m_counts.tests_skipped;
         break;
     case Test::Result::Fail:
         ++m_counts.tests_failed;