fix(cli): env ls should work if not in copilot dir (#1021)

<!-- Provide summary of changes -->
Fixes #1004.
<!-- Issue number, if available. E.g. "Fixes #31", "Addresses #42, 77" -->

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

internal/pkg/cli/env_list.go

@@ -48,14 +48,6 @@ func newListEnvOpts(vars listEnvVars) (*listEnvOpts, error) {
 	}, nil
 }
 
-// Validate returns an error if the values passed by flags are invalid.
-func (o *listEnvOpts) Validate() error {
-	if o.AppName() == "" {
-		return fmt.Errorf("no application found: run %s or %s into your workspace please", color.HighlightCode("app init"), color.HighlightCode("cd"))
-	}
-	return nil
-}
-
 // Ask asks for fields that are required but not passed in.
 func (o *listEnvOpts) Ask() error {
 	if o.AppName() != "" {
@@ -135,9 +127,6 @@ func BuildEnvListCmd() *cobra.Command {
 			if err != nil {
 				return err
 			}
-			if err := opts.Validate(); err != nil {
-				return err
-			}
 			if err := opts.Ask(); err != nil {
 				return err
 			}

internal/pkg/cli/env_list_test.go

@@ -15,44 +15,6 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestListEnvOpts_Validate(t *testing.T) {
-	testCases := map[string]struct {
-		inAppName string
-
-		wantedErr string
-	}{
-		"valid app": {
-			inAppName: "phonetool",
-		},
-		"invalid app": {
-			inAppName: "",
-
-			wantedErr: fmt.Sprint("no application found: run `app init` or `cd` into your workspace please"),
-		},
-	}
-
-	for name, tc := range testCases {
-		t.Run(name, func(t *testing.T) {
-			// GIVEN
-			opts := &initEnvOpts{
-				initEnvVars: initEnvVars{
-					GlobalOpts: &GlobalOpts{appName: tc.inAppName},
-				},
-			}
-
-			// WHEN
-			err := opts.Validate()
-
-			// THEN
-			if tc.wantedErr != "" {
-				require.EqualError(t, err, tc.wantedErr)
-			} else {
-				require.Nil(t, err)
-			}
-		})
-	}
-}
-
 func TestEnvList_Ask(t *testing.T) {
 	testCases := map[string]struct {
 		inputApp string