Add SerializeOpVec output prefix check.

meimchu · meimchu · commit facfee2870bc · 2026-06-05T22:55:35.000-07:00
Signed-off-by: Mei Chu &lt;meimchu@gmail.com&gt;
diff --git a/tests/cpu/Op_tests.cpp b/tests/cpu/Op_tests.cpp
@@ -514,4 +514,7 @@ OCIO_ADD_TEST(OpRcPtrVec, serialize)
 
     // Serialize not optimized OpVec i.e. contains some NoOps.
     OCIO_CHECK_NO_THROW(OCIO::SerializeOpVec(ops));
+
+    // Check Serialize output's prefix for indentation.
+    OCIO_CHECK_EQUAL(std::string_view(SerializeOpVec(ops, 4), 7), "    Op ");
 }
diff --git a/tests/cpu/ops/noop/NoOps_tests.cpp b/tests/cpu/ops/noop/NoOps_tests.cpp
@@ -242,8 +242,6 @@ OCIO_ADD_TEST(NoOps, partition_gpu_ops)
     OCIO_CHECK_NO_THROW( AssertPartitionIntegrity(gpuPreOps,
                                                   gpuLatticeOps,
                                                   gpuPostOps) );
-    std::cerr << "gpuPreOps" << std::endl;
-    std::cerr << SerializeOpVec(gpuPreOps, 4) << std::endl;
     /*
     std::cerr << "gpuPreOps" << std::endl;
     std::cerr << SerializeOpVec(gpuPreOps, 4) << std::endl;

Original file line number	Diff line number	Diff line change
`@@ -514,4 +514,7 @@ OCIO_ADD_TEST(OpRcPtrVec, serialize)`
`514`	`514`
`515`	`515`	`// Serialize not optimized OpVec i.e. contains some NoOps.`
`516`	`516`	`OCIO_CHECK_NO_THROW(OCIO::SerializeOpVec(ops));`
	`517`	`+`
	`518`	`+ // Check Serialize output's prefix for indentation.`
	`519`	`+ OCIO_CHECK_EQUAL(std::string_view(SerializeOpVec(ops, 4), 7), " Op ");`
`517`	`520`	`}`