Fix tests

PaloAltoNetworks · Oct 27, 2023 · 0e1994b · 0e1994b
1 parent 4258f9a
commit 0e1994b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/commit/pano.go b/commit/pano.go
@@ -225,8 +225,8 @@ type pcaDgInfoEntry struct {
 }
 
 type pcaTemplate struct {
-	Devices             *util.MemberType `xml:"device"`
 	Admins              *util.MemberType `xml:"admin,omitempty"`
+	Devices             *util.MemberType `xml:"device"`
 	Name                string           `xml:"name"`
 	Description         string           `xml:"description,omitempty"`
 	ForceTemplateValues string           `xml:"force-template-values"`

diff --git a/commit/pano_test.go b/commit/pano_test.go
@@ -119,6 +119,7 @@ func TestPanoCommitAllDeviceGroup(t *testing.T) {
 		"<description>device group</description>",
 		"<include-template>yes</include-template>",
 		"<force-template-values>no</force-template-values>",
+		"<validate-only>no</validate-only>",
 		"</shared-policy>",
 		"</commit-all>",
 	}
@@ -142,10 +143,11 @@ func TestPanoCommitAllTemplate(t *testing.T) {
 	s := []string{
 		"<commit-all>",
 		"<template>",
+		"<device><member>12321</member></device>",
 		"<name>tmpl1</name>",
 		"<description>template</description>",
-		"<device><member>12321</member></device>",
 		"<force-template-values>yes</force-template-values>",
+		"<validate-only>no</validate-only>",
 		"</template>",
 		"</commit-all>",
 	}
@@ -169,10 +171,11 @@ func TestPanoCommitAllTemplateStack(t *testing.T) {
 	s := []string{
 		"<commit-all>",
 		"<template-stack>",
+		"<device><member>12321</member></device>",
 		"<name>ts1</name>",
 		"<description>template stack</description>",
-		"<device><member>12321</member></device>",
 		"<force-template-values>yes</force-template-values>",
+		"<validate-only>no</validate-only>",
 		"</template-stack>",
 		"</commit-all>",
 	}