Disable OptiX tests by default. (#1331)

When running `slang-test`, the OptiX tests will be skipped by default for now, and must be explicitly enabled by adding `-category optix` on the command line.

I will need to add a better discovery mechanism down the line, closer to how support for different graphics APIs is being tested, but for now this should be enough to unblock our CI builds.

2 files changed, 8 insertions, 1 deletions

diff --git a/tests/pipeline/ray-tracing/raygen.slang b/tests/pipeline/ray-tracing/raygen.slang
index 1b13a63c2..3edde6c1a 100644
--- a/tests/pipeline/ray-tracing/raygen.slang
+++ b/tests/pipeline/ray-tracing/raygen.slang
@@ -1,6 +1,6 @@
 // raygen.slang
 
-//TEST:COMPARE_COMPUTE_EX:-cuda -rt -output-using-type -compute-dispatch 4,1,1
+//TEST(optix):COMPARE_COMPUTE_EX:-cuda -rt -output-using-type -compute-dispatch 4,1,1
 
 //TEST_INPUT:ubuffer(data=[0 0 0 0],stride=4):name gOutput,out
 
diff --git a/tools/slang-test/slang-test-main.cpp b/tools/slang-test/slang-test-main.cpp
index da03d67fe..8b3fda9dd 100644
--- a/tools/slang-test/slang-test-main.cpp
+++ b/tools/slang-test/slang-test-main.cpp
@@ -3105,6 +3105,7 @@ SlangResult innerMain(int argc, char** argv)
     auto vulkanTestCategory = categorySet.add("vulkan", fullTestCategory);
     auto unitTestCatagory = categorySet.add("unit-test", fullTestCategory);
     auto cudaTestCategory = categorySet.add("cuda", fullTestCategory);
+    auto optixTestCategory = categorySet.add("optix", cudaTestCategory);
 
     auto compatibilityIssueCategory = categorySet.add("compatibility-issue", fullTestCategory);
         
@@ -3206,6 +3207,12 @@ SlangResult innerMain(int argc, char** argv)
         options.includeCategories.Add(fullTestCategory, fullTestCategory);
     }
 
+    // Don't include OptiX tests unless the client has explicit opted into them.
+    if( !options.includeCategories.ContainsKey(optixTestCategory) )
+    {
+        options.excludeCategories.Add(optixTestCategory, optixTestCategory);
+    }
+
     // Exclude rendering tests when building under AppVeyor.
     //
     // TODO: this is very ad hoc, and we should do something cleaner.