Updated the pytest files

Author: dhaselva

docs/docs/validations.md

@@ -2760,6 +2760,7 @@ To avoid this issue, change the target version to another version. Or verify tha
 
 This bug [CSCwt78235][67] validates `F0467` faults where `changeSet` contains 'bgpProt-policy-already-existing'. The fault indicates conflicting BGP protocol timer policy under an L3Outs deployed in same vrf under same node. If this fault is not resolved, l3out will not be programmed properly in the leaf after the upgrade.
 
+
 [0]: https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script
 [1]: https://www.cisco.com/c/dam/en/us/td/docs/Website/datacenter/apicmatrix/index.html
 [2]: https://www.cisco.com/c/en/us/support/switches/nexus-9000-series-switches/products-release-notes-list.html

tests/checks/bgp_timer_policy_already_existing_check/faultDelegate_UNFORMATTED.json

@@ -12,23 +12,21 @@
 faultDelegates = 'faultDelegate.json?query-target-filter=and(eq(faultDelegate.code,"F0467"),wcard(faultDelegate.changeSet,"bgpProt-policy-already-existing"))'
 
 @pytest.mark.parametrize(
-    "icurl_outputs, tversion, expected_result, expected_data, expected_unformatted_data",
+    "icurl_outputs, tversion, expected_result, expected_data",
     [
         # target release not affected (> 6.2(1g))
         (
             {faultDelegates: read_data(dir, "faultDelegate_POS.json")},
             "6.2(2a)",
             script.NA,
             [],
-            [],
         ),
         # target release not affected on 6.1 train (> 6.1(5e))
         (
             {faultDelegates: read_data(dir, "faultDelegate_POS.json")},
             "6.1(5f)",
             script.NA,
             [],
-            [],
         ),
         # boundary version is still affected for strict newer_than check
         (
@@ -49,7 +47,6 @@
                     "configQual:bgpProt-policy-already-existing, configSt:failed-to-apply, temporaryError:no",
                 ],
             ],
-            [],
         ),
         # 6.1 boundary version is still affected for strict newer_than check
         (
@@ -70,7 +67,6 @@
                     "configQual:bgpProt-policy-already-existing, configSt:failed-to-apply, temporaryError:no",
                 ],
             ],
-            [],
         ),
         # target release affected on 6.1 train (< 6.1(5e))
         (
@@ -91,32 +87,16 @@
                     "configQual:bgpProt-policy-already-existing, configSt:failed-to-apply, temporaryError:no",
                 ],
             ],
-            [],
-        ),
-        (
-            {faultDelegates: read_data(dir, "faultDelegate_UNFORMATTED.json")},
-            "6.1(5a)",
-            script.FAIL_O,
-            [],
-            [
-                [
-                    "F0467",
-                    "resPolCont/rtdOutCont/rtdOutDef-[uni/invalid]/nwissues",
-                    "configQual:bgpProt-policy-already-existing, configSt:failed-to-apply, temporaryError:no",
-                ],
-            ],
         ),
         (
             {faultDelegates: read_data(dir, "faultDelegate_NEG.json")},
             "6.1(5a)",
             script.PASS,
             [],
-            [],
         ),
     ],
 )
-def test_logic(run_check, mock_icurl, tversion, expected_result, expected_data, expected_unformatted_data):
+def test_logic(run_check, mock_icurl, tversion, expected_result, expected_data):
     result = run_check(tversion=script.AciVersion(tversion))
     assert result.result == expected_result
-    assert result.data == expected_data
-    assert result.unformatted_data == expected_unformatted_data
+    assert result.data == expected_data