scripts

mtojek · mtojek · commit e3be04760f61 · 2024-07-03T13:27:34.000+02:00
diff --git a/provisioner/terraform/resources_test.go b/provisioner/terraform/resources_test.go
@@ -331,10 +331,12 @@ func TestConvertResources(t *testing.T) {
 						{
 							DisplayName: "Foobar Script 1",
 							Script:      "echo foobar 1",
+							RunOnStart:  true,
 						},
 						{
 							DisplayName: "Foobar Script 2",
 							Script:      "echo foobar 2",
+							RunOnStart:  true,
 						},
 					},
 					Auth:                     &proto.Agent_Token{},
@@ -352,6 +354,7 @@ func TestConvertResources(t *testing.T) {
 						{
 							DisplayName: "Foobar Script 3",
 							Script:      "echo foobar 3",
+							RunOnStart:  true,
 						},
 					},
 					Auth:                     &proto.Agent_Token{},
@@ -780,12 +783,6 @@ func TestConvertResources(t *testing.T) {
 						if agent.GetInstanceId() != "" {
 							agent.Auth = &proto.Agent_InstanceId{}
 						}
-
-						for _, script := range agent.Scripts {
-							// FIXME `RunOnStart`` is set only in "Provision", so we need to clear it
-							// to simplify table tests. Current tests do not verify `RunOnStart` properties.
-							script.RunOnStart = false
-						}
 					}
 				}
 				// Convert expectedNoMetadata and resources into a

Original file line number	Diff line number	Diff line change
`@@ -331,10 +331,12 @@ func TestConvertResources(t *testing.T) {`
`331`	`331`	`{`
`332`	`332`	`DisplayName: "Foobar Script 1",`
`333`	`333`	`Script: "echo foobar 1",`
	`334`	`+ RunOnStart: true,`
`334`	`335`	`},`
`335`	`336`	`{`
`336`	`337`	`DisplayName: "Foobar Script 2",`
`337`	`338`	`Script: "echo foobar 2",`
	`339`	`+ RunOnStart: true,`
`338`	`340`	`},`
`339`	`341`	`},`
`340`	`342`	`Auth: &proto.Agent_Token{},`
`@@ -352,6 +354,7 @@ func TestConvertResources(t *testing.T) {`
`352`	`354`	`{`
`353`	`355`	`DisplayName: "Foobar Script 3",`
`354`	`356`	`Script: "echo foobar 3",`
	`357`	`+ RunOnStart: true,`
`355`	`358`	`},`
`356`	`359`	`},`
`357`	`360`	`Auth: &proto.Agent_Token{},`
`@@ -780,12 +783,6 @@ func TestConvertResources(t *testing.T) {`
`780`	`783`	`if agent.GetInstanceId() != "" {`
`781`	`784`	`agent.Auth = &proto.Agent_InstanceId{}`
`782`	`785`	`}`
`783`		`-`
`784`		`- for _, script := range agent.Scripts {`
`785`		- // FIXME `RunOnStart`` is set only in "Provision", so we need to clear it
`786`		- // to simplify table tests. Current tests do not verify `RunOnStart` properties.
`787`		`- script.RunOnStart = false`
`788`		`- }`
`789`	`786`	`}`
`790`	`787`	`}`
`791`	`788`	`// Convert expectedNoMetadata and resources into a`