Skip to content

Commit 5c4f0a6

Browse files
peanballbeyhan
authored andcommitted
test: ops generator test with correct input
1 parent 722da8f commit 5c4f0a6

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

cloudconfig/aws/ops_generator.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -103,15 +103,14 @@ func (o OpsGenerator) GenerateVars(state storage.State) (string, error) {
103103
}
104104
dualstackOutput, ok := terraformOutputs.Map["dualstack"]
105105
if !ok {
106-
return "", fmt.Errorf("dualstack output not present")
106+
return "", fmt.Errorf("missing dualstack terraform output")
107107
}
108-
var dualstack bool
109-
if dualstackOutput.(bool) {
108+
dualstack := dualstackOutput.(bool)
109+
if dualstack {
110110
requiredOutputs = append(requiredOutputs,
111111
"internal_cidr_ipv6",
112112
"internal_az_subnet_ipv6_cidr_mapping",
113113
)
114-
dualstack = true
115114
}
116115

117116
switch state.LB.Type {

cloudconfig/aws/ops_generator_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ var _ = Describe("OpsGenerator", func() {
4747
"cf_tcp_lb_internal_security_group": "some-cf-tcp-lb-internal-security-group",
4848
"concourse_lb_target_groups": []string{"some-concourse-lb-target-group", "some-other-concourse-lb-target-group"},
4949
"concourse_lb_internal_security_group": "some-concourse-lb-internal-security-group",
50-
"dualstack": "false",
50+
"dualstack": false,
5151
"internal_az_subnet_id_mapping": map[string]interface{}{
5252
"us-east-1c": "some-internal-subnet-ids-3",
5353
"us-east-1a": "some-internal-subnet-ids-1",
@@ -137,6 +137,7 @@ cf_tcp_lb_internal_security_group: some-cf-tcp-lb-internal-security-group
137137
cf_iso_router_lb_name: some-cf-iso-seg-router-lb-name
138138
concourse_lb_target_groups: [some-concourse-lb-target-group, some-other-concourse-lb-target-group]
139139
concourse_lb_internal_security_group: some-concourse-lb-internal-security-group
140+
dualstack: false
140141
internal_az_subnet_cidr_mapping:
141142
us-east-1a: 10.0.16.0/20
142143
us-east-1b: 10.0.32.0/20

0 commit comments

Comments
 (0)