Avoided running internal unit tests at start in release builds. You can now do this with a --run-tests argument if needed

Author: drowaudio

Source/CommandLine.cpp

@@ -356,6 +356,27 @@ static void validate (CommandLineValidator& validator, const ArgumentList& args)
     }
 }
 
+int getNumTestFailures (UnitTestRunner& testRunner)
+{
+    int numFailures = 0;
+    
+    for (int i = 0; i < testRunner.getNumResults(); ++i)
+        if (auto result = testRunner.getResult (i))
+            numFailures += result->failures;
+
+    return numFailures;
+}
+
+void runUnitTests()
+{
+    UnitTestRunner testRunner;
+    testRunner.runTestsInCategory ("pluginval");
+    const int numFailures = getNumTestFailures (testRunner);
+    
+    if (numFailures > 0)
+        ConsoleApplication::fail (String (numFailures) + " tests failed!!!");
+}
+
 //==============================================================================
 void performCommandLine (CommandLineValidator& validator, const ArgumentList& args)
 {
@@ -368,6 +389,10 @@ void performCommandLine (CommandLineValidator& validator, const ArgumentList& ar
                       "--validate [list]",
                       "Validates the files (or IDs for AUs).", String(),
                       [&validator] (const auto& args) { validate (validator, args); }});
+    cli.addCommand ({ "--run-tests",
+                      "--run-tests",
+                      "Runs the internal unit tests.", String(),
+                      [] (const auto&) { runUnitTests(); }});
 
     JUCEApplication::getInstance()->setApplicationReturnValue (cli.findAndRunCommand (args));
 
@@ -380,7 +405,8 @@ bool shouldPerformCommandLine (const ArgumentList& args)
 {
     return args.containsOption ("--help|-h")
         || args.containsOption ("--version")
-        || args.containsOption ("--validate");
+        || args.containsOption ("--validate")
+        || args.containsOption ("--run-tests");
 }
 
 //==============================================================================

Source/Main.cpp

@@ -48,8 +48,10 @@ class PluginValidatorApplication  : public JUCEApplication,
     //==============================================================================
     void initialise (const String& commandLine) override
     {
+       #if JUCE_DEBUG
         UnitTestRunner testRunner;
         testRunner.runTestsInCategory ("pluginval");
+       #endif
 
         if (shouldPerformCommandLine (commandLine))
         {

install/linux_build

@@ -71,6 +71,7 @@ cd "$ROOT/Builds/LinuxMakefile"
 rm -rf $ROOT/Builds/LinuxMakefile/build/
 #make clean
 make CONFIG=Release -j4
+"$APP_FILE" --run-tests || exit 1
 
 
 #============================================================

install/mac_build

@@ -71,6 +71,8 @@ rm -rf $ROOT/Builds/MacOSX/build/$PROJECT_NAME.build
 xcodebuild -configuration Release clean
 xcodebuild -configuration Release GCC_TREAT_WARNINGS_AS_ERRORS=YES
 
+"$APP_FILE/Contents/MacOS/$PROJECT_NAME" --run-tests || exit 1
+
 #============================================================
 #   Sign with hardened runtime
 #============================================================

install/windows_build.bat

@@ -75,7 +75,7 @@ echo "Building products"
 cd "%ROOT%/Builds/VisualStudio2017"
 rd /S /Q "x64/Release"
 "%MSBUILD_EXE%" %PROJECT_NAME%.sln /p:VisualStudioVersion=15.0 /m /t:Build /p:Configuration=Release /p:Platform=x64 /p:PreferredToolArchitecture=x64  /p:TreatWarningsAsErrors=true
-
+call "%APP_FILE%" --run-tests || exit 1
 
 ::============================================================
 ::   Copy to deployment directory