[Testbed] Fix crash when Testbed is launched with wrong module name.

Author: LukasBanana

tests/Testbed/TestbedContext.cpp

@@ -142,7 +142,9 @@ TestbedContext::TestbedContext(const char* moduleName, int version, int argc, ch
             rendererDesc.rendererConfigSize = sizeof(cfgGL);
         }
     }
-    if ((renderer = RenderSystem::Load(rendererDesc)) != nullptr)
+
+    Report report;
+    if ((renderer = RenderSystem::Load(rendererDesc, &report)) != nullptr)
     {
         // Create swap chain
         SwapChainDescriptor swapChainDesc;
@@ -181,6 +183,11 @@ TestbedContext::TestbedContext(const char* moduleName, int version, int argc, ch
         CreateSamplerStates();
         LoadDefaultProjectionMatrix();
     }
+    else
+    {
+        // Log error report
+        Log::Errorf(Log::ColorFlags::StdError, "%s", report.GetText());
+    }
 }
 
 TestbedContext::~TestbedContext()

tests/Testbed/TestbedContext.h

@@ -45,6 +45,12 @@ class TestbedContext
         // Runs all tests and returns the number of failed ones. If all succeeded, the return value is 0.
         unsigned RunAllTests();
 
+        // Returns true if this context has a valid renderer.
+        inline bool IsValid() const
+        {
+            return (renderer.get() != nullptr);
+        }
+
     public:
 
         static unsigned RunRendererIndependentTests(int argc, char* argv[]);

tests/Testbed/TestbedMain.cpp

@@ -35,8 +35,12 @@ static unsigned RunTestbedForRenderer(const char* moduleName, int version, int a
         Log::Printf("Run Testbed: %s (%d)\n", moduleName, version);
     else
         Log::Printf("Run Testbed: %s\n", moduleName);
+
     TestbedContext::PrintSeparator();
     TestbedContext context{ moduleName, version, argc, argv };
+    if (!context.IsValid())
+        return 1;
+
     unsigned failures = context.RunAllTests();
     TestbedContext::PrintSeparator();
     Log::Printf("\n");