improve test cases for python config tests

Signed-off-by: annie <[email protected]>

Author: annieln

tests/python/ConfigTest.py

@@ -839,14 +839,27 @@ def test_virtual_display(self):
         self.assertEqual(1, len(cfg.getViews(OCIO.VIEW_SHARED, "sRGB")))
 
         self.assertTrue(cfg.displayHasView("sRGB", "sview1"))
+        self.assertFalse(cfg.displayHasView("sRGB", "sview2"))
+        self.assertFalse(cfg.displayHasView("sRGB", " "))
+
         self.assertTrue(cfg.viewIsShared("sRGB", "sview1"))
+        self.assertFalse(cfg.viewIsShared("sRGB", "Raw"))
+        self.assertFalse(cfg.viewIsShared("sRGB", " "))
 
         other = copy.deepcopy(cfg)
 
         self.assertTrue(OCIO.Config.ViewsAreEqual(cfg, other, "sRGB", "sview1"))
         self.assertTrue(OCIO.Config.ViewsAreEqual(cfg, other, "sRGB", "Raw"))
         self.assertTrue(OCIO.Config.ViewsAreEqual(cfg, other, "sRGB", "view"))
 
+        cfg.removeDisplayView("sRGB", "sview1")
+        cfg.removeDisplayView("sRGB", "Raw")
+        cfg.removeDisplayView("sRGB", "view")
+
+        self.assertFalse(OCIO.Config.ViewsAreEqual(cfg, other, "sRGB", "sview1"))
+        self.assertFalse(OCIO.Config.ViewsAreEqual(cfg, other, "sRGB", "Raw"))
+        self.assertFalse(OCIO.Config.ViewsAreEqual(cfg, other, "sRGB", "view"))
+
         # Validate the virtual display information
         self.assertEqual(
             2,
@@ -885,11 +898,28 @@ def test_virtual_display(self):
         self.assertTrue(cfg.virtualViewIsShared("sview2"))
         self.assertTrue(OCIO.Config.VirtualViewsAreEqual(cfg, other, "sview2"))
 
+        other.addVirtualDisplayView("sview3", "", "raw")
+
+        self.assertFalse(cfg.hasVirtualView("sview3"))
+        self.assertTrue(other.hasVirtualView("sview3"))
+        self.assertFalse(OCIO.Config.VirtualViewsAreEqual(cfg, other, "sview3"))
+
+        # Test a virtual view that doesn't exist in either
+        self.assertFalse(cfg.hasVirtualView(" "))
+        self.assertFalse(other.hasVirtualView(" "))
+        self.assertFalse(OCIO.Config.VirtualViewsAreEqual(cfg, other, " "))
+
+        self.assertFalse(cfg.hasVirtualView("nonexistent"))
+        self.assertFalse(other.hasVirtualView("nonexistent"))
+        self.assertFalse(OCIO.Config.VirtualViewsAreEqual(cfg, other, "nonexistent"))
+
         # Remove a view from the virtual display
         cfg.removeVirtualDisplayView("Raw")
 
-        self.assertFalse(OCIO.Config.VirtualViewsAreEqual(cfg, other, "Raw"))
         self.assertFalse(cfg.hasVirtualView("Raw"))
+        self.assertTrue(other.hasVirtualView("Raw"))
+
+        self.assertFalse(OCIO.Config.VirtualViewsAreEqual(cfg, other, "Raw"))
 
         self.assertEqual(
             1,
@@ -911,6 +941,10 @@ def test_virtual_display(self):
 
         self.assertFalse(cfg.hasVirtualView("sview2"))
         self.assertFalse(cfg.virtualViewIsShared("sview2"))
+
+        self.assertTrue(other.hasVirtualView("sview2"))
+        self.assertTrue(other.virtualViewIsShared("sview2"))
+
         self.assertFalse(OCIO.Config.VirtualViewsAreEqual(cfg, other, "sview2"))
 
         cfg.addVirtualDisplaySharedView("sview2")