tweaks

meroxa · Feb 8, 2022 · b2d535f · b2d535f
1 parent ef6a1fe
commit b2d535f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 24 deletions.
diff --git a/cmd/meroxa/root/environments/repair.go b/cmd/meroxa/root/environments/repair.go
@@ -18,10 +18,9 @@ import (
 	"errors"
 	"fmt"
 
-	"github.com/meroxa/cli/utils"
-
 	"github.com/meroxa/cli/cmd/meroxa/builder"
 	"github.com/meroxa/cli/log"
+	"github.com/meroxa/cli/utils"
 	"github.com/meroxa/meroxa-go/pkg/meroxa"
 )
 

diff --git a/utils/display.go b/utils/display.go
@@ -684,7 +684,7 @@ func EnvironmentTable(environment *meroxa.Environment) string {
 				{Text: ""},
 			},
 			{
-				{Align: simpletable.AlignRight, Text: "				AWS EC2 Permissions Status:"},
+				{Align: simpletable.AlignRight, Text: "AWS EC2 Permissions Status:"},
 				{Text: strings.Join(environment.Status.PreflightDetails.PreflightPermissions.EC2, " ; ")},
 			},
 			{

diff --git a/utils/display_test.go b/utils/display_test.go
@@ -599,29 +599,10 @@ func TestEnvironmentsTable(t *testing.T) {
 }
 
 func TestEnvironmentsTablePreflightFailed(t *testing.T) {
-	e := &meroxa.Environment{
-		Type:     meroxa.EnvironmentTypePrivate,
-		Name:     "environment-preflight-failed",
-		Provider: meroxa.EnvironmentProviderAws,
-		Region:   meroxa.EnvironmentRegionUsEast1,
-		Status: meroxa.EnvironmentViewStatus{
-			State:   meroxa.EnvironmentStatePreflightError,
-			Details: "",
-			PreflightDetails: &meroxa.PreflightDetails{
-				PreflightPermissions: &meroxa.PreflightPermissions{
-					S3:  []string{"missing read permission for S3", "missing write permissions for S3"},
-					EC2: []string{"missing read permission for S3", "missing write permissions for S3"},
-				},
-				PreflightLimits: &meroxa.PreflightLimits{
-					EIP: "",
-				},
-			},
-		},
-		UUID: "531428f7-4e86-4094-8514-d397d49026f7",
-	}
+	e := GenerateEnvironmentFailed("environment-preflight-failed")
 
 	tests := map[string][]*meroxa.Environment{
-		"Base": {e},
+		"Base": {&e},
 	}
 
 	tableHeaders := []string{"ID", "NAME", "TYPE", "PROVIDER", "REGION", "STATE"}