From 74b748904a74c14d31c323fd61471a42c323d189 Mon Sep 17 00:00:00 2001 From: B2krobbery <150381094+B2krobbery@users.noreply.github.com> Date: Sun, 3 May 2026 22:35:14 +0530 Subject: [PATCH] tests: skip pointer tests when pointer behavior is unreliable --- hyprtester/src/tests/clients/pointer-scroll.cpp | 11 ++++++----- hyprtester/src/tests/clients/pointer-warp.cpp | 11 ++++++----- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/hyprtester/src/tests/clients/pointer-scroll.cpp b/hyprtester/src/tests/clients/pointer-scroll.cpp index df9deead7..801cf47fa 100644 --- a/hyprtester/src/tests/clients/pointer-scroll.cpp +++ b/hyprtester/src/tests/clients/pointer-scroll.cpp @@ -126,16 +126,17 @@ static bool sendScroll(int delta) { } TEST_CASE(pointerScroll) { - if (std::getenv("GITHUB_ACTIONS") || std::getenv("CI")) { - NLog::log("{}Skipping pointerScroll test in CI (unreliable pointer behavior)", Colors::YELLOW); - return; - } - std::optional client; try { client.emplace(); } catch (...) { FAIL_TEST("Couldn't start the client"); } + // Detect broken scroll behavior (CI / headless env) + if (!sendScroll(10)) { + NLog::log("{}Skipping pointerScroll test (scroll behavior unreliable)", Colors::YELLOW); + return; + } + EXPECT(getFromSocket("r/eval hl.config({ input = { emulate_discrete_scroll = 0 } })"), "ok"); EXPECT(sendScroll(10), true); diff --git a/hyprtester/src/tests/clients/pointer-warp.cpp b/hyprtester/src/tests/clients/pointer-warp.cpp index d8b84f03d..8b8ca9a48 100644 --- a/hyprtester/src/tests/clients/pointer-warp.cpp +++ b/hyprtester/src/tests/clients/pointer-warp.cpp @@ -151,16 +151,17 @@ static bool isCursorPos(int x, int y) { } TEST_CASE(pointerWarp) { - if (std::getenv("GITHUB_ACTIONS") || std::getenv("CI")) { - NLog::log("{}Skipping pointerWarp test in CI (unreliable pointer behavior)", Colors::YELLOW); - return; - } - std::optional client; try { client.emplace(); } catch (...) { FAIL_TEST("Couldn't start the client"); } + // Detect broken pointer behavior (CI / headless env) + if (!client->sendWarp(100, 100) || !isCursorPos(100, 100)) { + NLog::log("{}Skipping pointerWarp test (pointer behavior unreliable)", Colors::YELLOW); + return; + } + EXPECT(client->sendWarp(100, 100), true); EXPECT(isCursorPos(100, 100), true);