Skip to content

Commit b0fdf8a

Browse files
author
Brett Mack
committed
Fixed failing test
1 parent 29dfc43 commit b0fdf8a

2 files changed

Lines changed: 2 additions & 9 deletions

File tree

builtin/providers/vcd/resource_vcd_firewall_rules.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -184,13 +184,6 @@ func deleteFirewallRules(d *schema.ResourceData, gateway *types.EdgeGateway) []*
184184
func matchFirewallRule(d *schema.ResourceData, prefix string, rules []*types.FirewallRule) (string, error) {
185185

186186
for _, m := range rules {
187-
log.Printf("[INFO] %s - %s", d.Get(prefix+".description").(string), m.Description)
188-
log.Printf("[INFO] %s - %s", d.Get(prefix+".policy").(string), m.Policy)
189-
log.Printf("[INFO] %s - %s", d.Get(prefix+".protocol").(string), getProtocol(*m.Protocols))
190-
log.Printf("[INFO] %s - %s", d.Get(prefix+".destination_port").(string), getPortString(m.Port))
191-
log.Printf("[INFO] %s - %s", strings.ToLower(d.Get(prefix+".destination_ip").(string)), strings.ToLower(m.DestinationIP))
192-
log.Printf("[INFO] %s - %s", d.Get(prefix+".source_port").(string), getPortString(m.SourcePort))
193-
log.Printf("[INFO] %s - %s", strings.ToLower(d.Get(prefix+".source_ip").(string)), strings.ToLower(m.SourceIP))
194187
if d.Get(prefix+".description").(string) == m.Description &&
195188
d.Get(prefix+".policy").(string) == m.Policy &&
196189
strings.ToLower(d.Get(prefix+".protocol").(string)) == getProtocol(*m.Protocols) &&

builtin/providers/vcd/resource_vcd_vapp_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ func TestAccVcdVApp_PowerOff(t *testing.T) {
3232
),
3333
},
3434
resource.TestStep{
35-
Config: testAccCheckVcdVApp_powerOff,
35+
Config: fmt.Sprintf(testAccCheckVcdVApp_powerOff, os.Getenv("VCD_EDGE_GATWEWAY")),
3636
Check: resource.ComposeTestCheckFunc(
3737
testAccCheckVcdVAppExists("vcd_vapp.foobar", &vapp),
3838
testAccCheckVcdVAppAttributes_off(&vapp),
@@ -83,7 +83,7 @@ func testAccCheckVcdVAppDestroy(s *terraform.State) error {
8383
_, err := conn.OrgVdc.FindVAppByName(rs.Primary.ID)
8484

8585
if err == nil {
86-
return fmt.Errorf("VPCs still exist.")
86+
return fmt.Errorf("VPCs still exist")
8787
}
8888

8989
return nil

0 commit comments

Comments
 (0)