Fix parsing of some yaml tests on master (#5315)

* Skip cat template YAML test (#5313)

Co-authored-by: Martijn Laarman <Mpdreamz@gmail.com>
(cherry picked from commit 9251068)

* No longer fail to parse when allowed_warnings is last in the do operation definition

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>
(cherry picked from commit 78a0933)

Author: Mpdreamz

tests/Tests.YamlRunner/Models.fs

@@ -121,6 +121,7 @@ type Do = {
     ApiCall: string * YamlMap
     Catch:DoCatch option
     Warnings:option<string list>
+    AllowedWarnings:option<string list>
     NodeSelector:NodeSelector option
     Headers: Headers option
     AutoFail: bool 

tests/Tests.YamlRunner/SkipList.fs

@@ -183,6 +183,9 @@ let SkipList = dict<SkipFile,SkipSection> [
     // TODO investigate post 7.11.0
     SkipFile "nodes.info/10_basic.yml", Section "node_info role test"
 
+    // TODO investigate post 7.11.0
+    SkipFile "nodes.info/10_basic.yml", Section "node_info role test"
+
 ]
 
 

tests/Tests.YamlRunner/TestsReader.fs

@@ -150,6 +150,8 @@ let private mapDo section (operation:YamlMap) =
         | None -> None
 
     let warnings = tryPickList<string, string> operation "warnings" id
+    // TODO: we don't support this feature yet but do parse it out so we don't crash our yaml parser
+    let allowedWarnings = tryPickList<string, string> operation "allowed_warnings" id
     let nodeSelector = mapNodeSelector operation
 
     let last = operation.Last()
@@ -161,6 +163,7 @@ let private mapDo section (operation:YamlMap) =
         ApiCall = (lastKey, lastValue)
         Catch = catch
         Warnings = warnings
+        AllowedWarnings = allowedWarnings
         NodeSelector = nodeSelector
         Headers = headers
         AutoFail = match section with | "setup" | "teardown" -> false | _ -> false