Add test for folder skip recognition

Author: furtib

analyzer/tests/functional/skip/test_files/skip_folder/Makefile

@@ -0,0 +1,13 @@
+OBJS = $(SRCS:.cpp=.o)
+
+CXXFLAGS = -Wno-all -Wno-extra -Wno-division-by-zero
+
+SRCS = skipme/skipme.cpp
+
+.cpp.o:
+	$(CXX) $(CXXFLAGS) -c $<  -o $@
+
+all: $(OBJS)
+
+clean:
+	rm -rf *.o

analyzer/tests/functional/skip/test_files/skip_folder/skipfile

@@ -0,0 +1 @@
+-*skipme

analyzer/tests/functional/skip/test_files/skip_folder/skipme/skipme.cpp

@@ -0,0 +1,12 @@
+// -------------------------------------------------------------------------
+//  Part of the CodeChecker project, under the Apache License v2.0 with
+//  LLVM Exceptions. See LICENSE for license information.
+//  SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+// -------------------------------------------------------------------------
+
+// This file is to be included in a skip list file.
+
+void skipped_test(int z) {
+  if (z == 0)
+    int x = 1 / z; // warn
+}

analyzer/tests/functional/skip/test_skip.py

@@ -171,6 +171,62 @@ def test_skip(self):
                             "Reports from headers should be kept"
                             " if the header is not on the skiplist")
 
+    def test_skip_directory(self):
+        """Test wether directories in skipfile entries works correctly"""
+        with tempfile.NamedTemporaryFile(
+            mode='w',
+            suffix="skipfile",
+            encoding='utf-8') as skip_file:
+
+            # Skip the `skipme` folder, the way its in the documentation
+            skip_file.write('\n'.join([
+                '-*skipme/*',
+            ]))
+            skip_file.flush()
+            self.__log_and_analyze("skip_folder", ["--ignore", skip_file.name])
+
+            # Check if the folder is skipped
+            # There shouldn't be any report generated
+            report_dir_files = os.listdir(self.report_dir)
+            for f in report_dir_files:
+                self.assertFalse("skipme.cpp" in f)
+
+        with tempfile.NamedTemporaryFile(
+            mode='w',
+            suffix="skipfile",
+            encoding='utf-8') as skip_file:
+            
+            # Skip the `skipme` folder, leaving out the '*'
+            skip_file.write('\n'.join([
+                '-*skipme/',
+            ]))
+            skip_file.flush()
+            self.__log_and_analyze("skip_folder", ["--ignore", skip_file.name])
+
+            # Check if the folder is skipped
+            # There shouldn't be any report generated
+            report_dir_files = os.listdir(self.report_dir)
+            for f in report_dir_files:
+                self.assertFalse("skipme.cpp" in f)
+
+        with tempfile.NamedTemporaryFile(
+            mode='w',
+            suffix="skipfile",
+            encoding='utf-8') as skip_file:
+            
+            # Skip the `skipme` folder, without any indication that its a folder
+            skip_file.write('\n'.join([
+                '-*skipme',
+            ]))
+            skip_file.flush()
+            self.__log_and_analyze("skip_folder", ["--ignore", skip_file.name])
+
+            # Check if the folder is skipped
+            # There shouldn't be any report generated
+            report_dir_files = os.listdir(self.report_dir)
+            for f in report_dir_files:
+                self.assertFalse("skipme.cpp" in f)
+
     def test_drop_reports(self):
         """Analyze a project with a skip file."""
         # we should not see a report from skip.h