Build: De-hardcode coverage regex exclusion

Author: Pagghiu

Tools/SC-build.cpp

@@ -99,6 +99,21 @@ Result buildTestProject(const Parameters& parameters, Project& project)
     project.addPresetConfiguration(Configuration::Preset::Debug, parameters);
     project.addPresetConfiguration(Configuration::Preset::Release, parameters);
     project.addPresetConfiguration(Configuration::Preset::DebugCoverage, parameters);
+    project.configurations.back().coverage.excludeRegex =
+        ".*\\/Tools.*|"
+        ".*\\Test.(cpp|h|c)|"
+        ".*\\test.(c|h)|"
+        ".*\\/Tests/.*\\.*|"
+        ".*\\/LibC\\+\\+.inl|"              // new / delete overloads
+        ".*\\/Assert.h|"                    // Can't test Assert::unreachable
+        ".*\\/PluginMacros.h|"              // macros for client plugins
+        ".*\\/ProcessPosixFork.inl|"        // Can't compute coverage for fork
+        ".*\\/EnvironmentTable.h|"          // Can't compute coverage for fork
+        ".*\\/InitializerList.h|"           // C++ Language Support
+        ".*\\/Reflection/.*\\.*|"           // constexpr and templates
+        ".*\\/ContainersReflection/.*\\.*|" // constexpr and templates
+        ".*\\/SerializationBinary/.*\\.*|"  // constexpr and templates
+        ".*\\/LibrariesExtra/.*\\.*";
     if (parameters.platform == Platform::Linux)
     {
         project.addPresetConfiguration(Configuration::Preset::Debug, parameters, "DebugValgrind");

Tools/SC-build.h

@@ -62,11 +62,11 @@ constexpr StringView INTERMEDIATES_SUBDIR = "_Intermediates";
         StringView afterSplit;
         if (arguments.arguments[0].splitBefore(SC_NATIVE_STR(":"), action.workspaceName))
         {
-            SC_TRUST_RESULT(arguments.arguments[0].splitAfter(SC_NATIVE_STR(":"), action.target));
+            SC_TRUST_RESULT(arguments.arguments[0].splitAfter(SC_NATIVE_STR(":"), action.projectName));
         }
         else
         {
-            action.target = arguments.arguments[0];
+            action.projectName = arguments.arguments[0];
         }
     }
     // TODO: We should run a matrix of all generators / platforms / architectures
@@ -124,11 +124,11 @@ constexpr StringView INTERMEDIATES_SUBDIR = "_Intermediates";
         StringView afterSplit;
         if (arguments.arguments[0].splitBefore(SC_NATIVE_STR(":"), action.workspaceName))
         {
-            SC_TRUST_RESULT(arguments.arguments[0].splitAfter(SC_NATIVE_STR(":"), action.target));
+            SC_TRUST_RESULT(arguments.arguments[0].splitAfter(SC_NATIVE_STR(":"), action.projectName));
         }
         else
         {
-            action.target = arguments.arguments[0];
+            action.projectName = arguments.arguments[0];
         }
     }
     StringSpan args[3];
@@ -146,7 +146,7 @@ constexpr StringView INTERMEDIATES_SUBDIR = "_Intermediates";
 
     if (not args[0].isEmpty())
     {
-        action.configuration = args[0];
+        action.configurationName = args[0];
     }
 
     if (args[1] == "xcode")

Tools/SC-build/Build.h

@@ -304,6 +304,12 @@ struct SourceFiles
     [[nodiscard]] bool removeSelection(StringView directory, StringView filter);
 };
 
+/// @brief Coverage specific flags
+struct CoverageFlags
+{
+    String excludeRegex; ///< Regex of files to exclude from coverage
+};
+
 /// @brief Groups SC::Build::CompileFlags and SC::Build::LinkFlags for a given SC::Build::Architecture
 struct Configuration
 {
@@ -350,6 +356,8 @@ struct Configuration
     CompileFlags compile; ///< Configuration compile flags
     LinkFlags    link;    ///< Configuration link flags
 
+    CoverageFlags coverage; ///< Configuration coverage flags
+
     Architecture::Type architecture = Architecture::Any; ///< Restrict this configuration to a specific architecture
 
     Configuration();
@@ -491,6 +499,10 @@ struct Definition
     /// @param workspaceName Name of the workspace to generate
     /// @param parameters Set of parameters with the wanted platforms, architectures and generators to generate
     Result configure(StringView workspaceName, const Parameters& parameters) const;
+
+    /// @brief Finds the configuration with given name in given workspace and project
+    [[nodiscard]] bool findConfiguration(StringView workspaceName, StringView projectName, StringView configurationName,
+                                         Workspace*& workspace, Project*& project, Configuration*& configuration);
 };
 
 //! @}
@@ -516,10 +528,12 @@ struct Action
     Type action = Configure;
 
     Parameters parameters;
-    StringView configuration;
-    StringView target;
+    StringView configurationName;
+    StringView projectName;
     StringView workspaceName;
 
+    bool allTargets = false;
+
     Span<const StringView> additionalArguments;
 
   private: