feedback

Author: drewmalin

v1/providers/aws/scripts/network_test.go

@@ -40,13 +40,6 @@ func TestCreateVPC(t *testing.T) {
 	if err != nil {
 		t.Fatalf("failed to get VPC: %v", err)
 	}
-
-	// err = awsClient.DeleteVPC(context.Background(), v1.DeleteVPCArgs{
-	// 	ID: vpc.ID,
-	// })
-	// if err != nil {
-	// 	t.Fatalf("failed to delete VPC: %v", err)
-	// }
 }
 
 func TestDeleteVPC(t *testing.T) {

v1/vpc.go

@@ -9,6 +9,9 @@ import (
 
 // VPC represents the complete specification of a Brev VPC.
 type VPC struct {
+	// The ID assigned by the cloud provider to the VPC.
+	id CloudProviderResourceID
+
 	// The name of the VPC, displayed on clients.
 	name string
 
@@ -22,9 +25,6 @@ type VPC struct {
 	// The cloud that hosts the VPC. For example, "aws".
 	cloud string
 
-	// The ID assigned by the cloud provider to the VPC.
-	id CloudProviderResourceID
-
 	// The location of the VPC. For example, "us-east-1".
 	location string