From c7f5a9acd37ab034d0bfd127aaf8e37cd60dd715 Mon Sep 17 00:00:00 2001 From: Emmanuel Lepage Vallee Date: Fri, 23 Sep 2016 23:50:17 -0400 Subject: [PATCH] tests: Fix awful.screen tests To ensure that some features such as SNID rules work, we need to ensure that the screen isn't set by other code paths. Only a single algorithm can be executed for the screen. As soon as many algorithms are executed on events such as "manage", it will most likely regress again. This commit make sure of that by disabling the default normal source of c.screen. After that, any other c.screen changes can be considered bugs. --- tests/test-awful-client.lua | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/test-awful-client.lua b/tests/test-awful-client.lua index 93155bcc..25d9bfa2 100644 --- a/tests/test-awful-client.lua +++ b/tests/test-awful-client.lua @@ -79,6 +79,9 @@ end local has_error +-- Disable awful.screen.preferred(c) +awful.rules.rules[1].properties.screen = nil + table.insert(steps, function() -- Make sure there is no extra callbacks that causes double screen changes -- (regress #1028 #1052)