Add test cases

Author: yutongzhang-microsoft

tests/common/plugins/conditional_mark/unit_test/tests_conditions.yaml

@@ -23,3 +23,36 @@ test_conditional_mark.py::test_mark_1:
     reason: "Skip test_conditional_mark.py::test_mark_1"
     conditions:
       - "asic_type in ['mellanox']"
+
+test_conditional_mark.py::test_mark_2:
+  skip:
+    reason: "Skip test_conditional_mark.py::test_mark_2"
+
+test_conditional_mark.py::test_mark_3:
+  skip:
+    reason: "Skip test_conditional_mark.py::test_mark_3"
+    conditions_logical_operator: or
+    conditions:
+      - "topo_type in ['t0']"
+      - "asic_type in ['mellanox']"
+
+test_conditional_mark.py::test_mark_4:
+  skip:
+    reason: "Skip test_conditional_mark.py::test_mark_4"
+    conditions:
+      - "topo_type in ['t0']"
+      - "asic_type in ['mellanox']"
+
+test_conditional_mark.py::test_mark_5:
+  skip:
+    reason: "Skip test_conditional_mark.py::test_mark_5"
+    conditions:
+      - "topo_type in ['t0']"
+      - "topo_type in ['t0']"
+
+test_conditional_mark.py::test_mark_6:
+  skip:
+    reason: "Skip test_conditional_mark.py::test_mark_6"
+    conditions:
+      - "topo_type in ['t0']"
+      - "topo_type not in ['t0']"

tests/common/plugins/conditional_mark/unit_test/unittest_find_all_matches.py

@@ -72,6 +72,101 @@ def test_all_false_conditions_in_matching_path(self):
 
         self.assertFalse(matches)
 
+    # Test case 4: The condition in the longest matching entry is empty
+    def test_empty_conditions(self):
+        conditions, session_mock = load_test_conditions()
+
+        nodeid = "test_conditional_mark.py::test_mark_2"
+
+        marks_found = []
+        matches = find_all_matches(nodeid, conditions, session_mock, DYNAMIC_UPDATE_SKIP_REASON, CUSTOM_BASIC_FACTS)
+
+        for match in matches:
+            for mark_name, mark_details in list(list(match.values())[0].items()):
+                marks_found.append(mark_name)
+
+                if mark_name == "skip":
+                    self.assertEqual(mark_details.get("reason"), "Skip test_conditional_mark.py::test_mark_2")
+
+        self.assertEqual(len(marks_found), 1)
+        self.assertIn('skip', marks_found)
+
+    # Test case 5: Test logic operation `or`
+    def test_logic_operation_or(self):
+        conditions, session_mock = load_test_conditions()
+
+        nodeid = "test_conditional_mark.py::test_mark_3"
+
+        marks_found = []
+        matches = find_all_matches(nodeid, conditions, session_mock, DYNAMIC_UPDATE_SKIP_REASON, CUSTOM_BASIC_FACTS)
+
+        for match in matches:
+            for mark_name, mark_details in list(list(match.values())[0].items()):
+                marks_found.append(mark_name)
+
+                if mark_name == "skip":
+                    self.assertEqual(mark_details.get("reason"), "Skip test_conditional_mark.py::test_mark_3")
+
+        self.assertEqual(len(marks_found), 1)
+        self.assertIn('skip', marks_found)
+
+    # Test case 6: Test logic operation `and`
+    def test_logic_operation_and(self):
+        conditions, session_mock = load_test_conditions()
+
+        nodeid = "test_conditional_mark.py::test_mark_4"
+
+        marks_found = []
+        matches = find_all_matches(nodeid, conditions, session_mock, DYNAMIC_UPDATE_SKIP_REASON, CUSTOM_BASIC_FACTS)
+
+        for match in matches:
+            for mark_name, mark_details in list(list(match.values())[0].items()):
+                marks_found.append(mark_name)
+
+                if mark_name == "skip":
+                    self.assertEqual(mark_details.get("reason"), "Skip test_conditional_mark.py::test_mark")
+
+        self.assertEqual(len(marks_found), 1)
+        self.assertIn('skip', marks_found)
+
+    # Test case 7: Test duplicated conditions
+    def test_duplicated_conditions(self):
+        conditions, session_mock = load_test_conditions()
+
+        nodeid = "test_conditional_mark.py::test_mark_5"
+
+        marks_found = []
+        matches = find_all_matches(nodeid, conditions, session_mock, DYNAMIC_UPDATE_SKIP_REASON, CUSTOM_BASIC_FACTS)
+
+        for match in matches:
+            for mark_name, mark_details in list(list(match.values())[0].items()):
+                marks_found.append(mark_name)
+
+                if mark_name == "skip":
+                    self.assertEqual(mark_details.get("reason"), "Skip test_conditional_mark.py::test_mark_5")
+
+        self.assertEqual(len(marks_found), 1)
+        self.assertIn('skip', marks_found)
+
+    # Test case 8: Test contradicting conditions
+    def test_contradicting_conditions(self):
+        conditions, session_mock = load_test_conditions()
+
+        nodeid = "test_conditional_mark.py::test_mark_6"
+
+        marks_found = []
+        matches = find_all_matches(nodeid, conditions, session_mock, DYNAMIC_UPDATE_SKIP_REASON, CUSTOM_BASIC_FACTS)
+
+        for match in matches:
+            for mark_name, mark_details in list(list(match.values())[0].items()):
+                marks_found.append(mark_name)
+
+                if mark_name == "skip":
+                    self.assertEqual(mark_details.get("reason"), "Skip test_conditional_mark.py::test_mark")
+
+        self.assertEqual(len(marks_found), 1)
+        self.assertIn('skip', marks_found)
+
 
 if __name__ == "__main__":
     unittest.main()