Fixed duplicate entries

Author: Priyanka-Patil14

aci-preupgrade-validation-script.py

@@ -6441,6 +6441,7 @@ def wred_affected_model_check(tversion, fabric_nodes, **kwargs):
     else:
         return Result(result=PASS, msg="WRED not enabled.")
 
+    unique_list = {}
     for obj in icurl("class", "eqptFC.json"):
         attr = obj["eqptFC"]["attributes"]
         model = attr.get("model", "")
@@ -6450,7 +6451,8 @@ def wred_affected_model_check(tversion, fabric_nodes, **kwargs):
         if not dn_match:
             continue
         node_id = dn_match.group("node")
-        data.append([node_id, node_name_map.get(node_id, ""), "FM", model])
+        unique_list[(node_id, model)] = [node_id, node_name_map.get(node_id, ""), "FM", model]
+    data = list(unique_list.values())
 
     if not data:
         return Result(result=NA, msg="No affected Fabric module found.")

tests/checks/wred_affected_model_check/test_wred_affected_model_check.py

@@ -97,6 +97,19 @@
             script.NA,
             [],
         ),
+        # Case 7: Same node has two FM slots with the same affected model (duplicate eqptFC objects).
+        # Deduplication by (node_id, model) must result in only one row.
+        # Expected: FAIL_O with a single row for node 1001.
+        (
+            "6.2(1g)",
+            read_data(dir, "fabricNode_spine.json"),
+            {
+                qosCong_api: read_data(dir, "qosCong_wred.json"),
+                eqptFC_api: read_data(dir, "eqptFC_duplicate.json"),
+            },
+            script.FAIL_O,
+            [["1001", "spine1001", "FM", "N9K-C9508-FM-E"]],
+        ),
     ],
 )
 def test_logic(run_check, mock_icurl, tversion, fabric_nodes, expected_result, expected_data):