From ffe64727fe3e360cf018180485954b02c5814d7a Mon Sep 17 00:00:00 2001
From: Uli Schlachter <psychon@users.noreply.github.com>
Date: Tue, 7 Jun 2016 22:57:01 +0200
Subject: [PATCH] Silence warnings in examples (#944)

There was a problem that the examples were considered to have failed as soon as
they produced any kind of output, but there were legitimate cases of warnings
being printed that triggered these checks. Commit 4819be4f4f3037ef3f used a
regular expression to detect and ignore this warnings.

This commit reverts the above commit and instead silences the warnings by
monkey-patching the function that prints the warnings into a no-op.

Signed-off-by: Uli Schlachter <psychon@znc.in>
---
 tests/examples/CMakeLists.txt                 | 29 +------------------
 tests/examples/awful/template.lua             |  3 ++
 tests/examples/gears/shape/template.lua       |  3 ++
 tests/examples/text/template.lua              |  3 ++
 .../wibox/container/defaults/template.lua     |  3 ++
 tests/examples/wibox/layout/template.lua      |  3 ++
 tests/examples/wibox/template.lua             |  3 ++
 7 files changed, 19 insertions(+), 28 deletions(-)

diff --git a/tests/examples/CMakeLists.txt b/tests/examples/CMakeLists.txt
index ab9511fa..253a5924 100644
--- a/tests/examples/CMakeLists.txt
+++ b/tests/examples/CMakeLists.txt
@@ -106,32 +106,6 @@ function(escape_code path escaped_content pre_header post_header)
     set(${post_header} ${example_post_header} PARENT_SCOPE)
 endfunction()
 
-# Only gears.debug.print_warning are allowed on stderr, everything else will
-# trigger a build failure to catch regressions and problem early.
-function(check_for_problems stderr result)
-
-    # If there is nothing to check, return
-    if (TEST_ERROR STREQUAL "")
-        set(${result} 0 PARENT_SCOPE)
-        return()
-    endif()
-
-    string(REGEX REPLACE "\n" ";" error_lines "${stderr}")
-
-    foreach (LINE ${error_lines})
-
-        # gears.debug.print_warning lines look like:
-        # yyyy-mm-dd hh:mm:ss W: message content
-        if (NOT ${LINE} MATCHES "^[0-9 :-]+ W:")
-            set(${result} 1 PARENT_SCOPE)
-            return()
-        endif()
-
-    endforeach()
-
-    set(${result} 0 PARENT_SCOPE)
-endfunction()
-
 # Execute a lua file.
 function(run_test test_path namespace template escaped_content)
 
@@ -159,8 +133,7 @@ function(run_test test_path namespace template escaped_content)
     )
 
     # If there is something on stderr, exit
-    check_for_problems("${TEST_ERROR}" problems_found)
-    if (${problems_found})
+    if (NOT TEST_ERROR STREQUAL "")
         message("${TEST_OUTPUT}")
         message("${TEST_ERROR}")
         message(FATAL_ERROR ${test_path} " A test failed, bye")
diff --git a/tests/examples/awful/template.lua b/tests/examples/awful/template.lua
index 21dfdedc..b31088f3 100644
--- a/tests/examples/awful/template.lua
+++ b/tests/examples/awful/template.lua
@@ -22,6 +22,9 @@ pcall(function()
     require("luacov.runner")(luacovpath)
 end)
 
+-- Silence debug warnings
+require("gears.debug").print_warning = function() end
+
 local color     = require( "gears.color" )
 local shape     = require( "gears.shape" )
 local beautiful = require( "beautiful"   )
diff --git a/tests/examples/gears/shape/template.lua b/tests/examples/gears/shape/template.lua
index 86fcc467..e3b0ca25 100644
--- a/tests/examples/gears/shape/template.lua
+++ b/tests/examples/gears/shape/template.lua
@@ -10,6 +10,9 @@ pcall(function()
     require("luacov.runner")(luacovpath)
 end)
 
+-- Silence debug warnings
+require("gears.debug").print_warning = function() end
+
 local function get_surface(p)
     local img = cairo.SvgSurface.create(p, 288, 76)
     return cairo.Context(img)
diff --git a/tests/examples/text/template.lua b/tests/examples/text/template.lua
index fa7d093c..a185adf3 100644
--- a/tests/examples/text/template.lua
+++ b/tests/examples/text/template.lua
@@ -15,5 +15,8 @@ pcall(function()
     require("luacov.runner")(luacovpath)
 end)
 
+-- Silence debug warnings
+require("gears.debug").print_warning = function() end
+
 -- Execute the test
 loadfile(file_path)()
diff --git a/tests/examples/wibox/container/defaults/template.lua b/tests/examples/wibox/container/defaults/template.lua
index 71a2e876..772389ce 100644
--- a/tests/examples/wibox/container/defaults/template.lua
+++ b/tests/examples/wibox/container/defaults/template.lua
@@ -21,6 +21,9 @@ pcall(function()
     require("luacov.runner")(luacovpath)
 end)
 
+-- Silence debug warnings
+require("gears.debug").print_warning = function() end
+
 -- Let the test request a size and file format
 local before, after = loadfile(file_path)()
 
diff --git a/tests/examples/wibox/layout/template.lua b/tests/examples/wibox/layout/template.lua
index 567cdfb9..50f9bf64 100644
--- a/tests/examples/wibox/layout/template.lua
+++ b/tests/examples/wibox/layout/template.lua
@@ -21,6 +21,9 @@ pcall(function()
     require("luacov.runner")(luacovpath)
 end)
 
+-- Silence debug warnings
+require("gears.debug").print_warning = function() end
+
 -- Create a generic rectangle widget to show layout disposition
 local function generic_widget(text)
     return {
diff --git a/tests/examples/wibox/template.lua b/tests/examples/wibox/template.lua
index 570c0dee..b765e44c 100644
--- a/tests/examples/wibox/template.lua
+++ b/tests/examples/wibox/template.lua
@@ -19,6 +19,9 @@ pcall(function()
     require("luacov.runner")(luacovpath)
 end)
 
+-- Silence debug warnings
+require("gears.debug").print_warning = function() end
+
 -- This is the main widget the tests will use as top level
 local container = wibox.layout.fixed.vertical()