Merge pull request #2110 from ActiveState/mitchell/dx-1237

`state use show` shows a better error message if the default project has been deleted

Author: mitchell-as

internal/runners/use/show.go

@@ -3,9 +3,11 @@ package use
 import (
 	"github.com/ActiveState/cli/internal/config"
 	"github.com/ActiveState/cli/internal/constants"
+	"github.com/ActiveState/cli/internal/errs"
 	"github.com/ActiveState/cli/internal/locale"
 	"github.com/ActiveState/cli/internal/output"
 	"github.com/ActiveState/cli/pkg/project"
+	"github.com/ActiveState/cli/pkg/projectfile"
 )
 
 type Show struct {
@@ -41,6 +43,11 @@ func (s *Show) Run() error {
 
 	proj, err := project.FromPath(projectDir)
 	if err != nil {
+		if errs.Matches(err, &projectfile.ErrorNoProject{}) {
+			return locale.WrapError(err,
+				"err_use_show_default_project_does_not_exist",
+				"The default project no longer exists. Please either check it out again with [ACTIONABLE]state checkout[/RESET] or run [ACTIONABLE]state use reset[/RESET] to unset your default project.")
+		}
 		return locale.WrapError(err, "err_use_show_get_project", "Could not get default project.")
 	}
 

test/integration/use_int_test.go

@@ -1,6 +1,7 @@
 package integration
 
 import (
+	"os"
 	"path/filepath"
 	"runtime"
 	"testing"
@@ -197,6 +198,13 @@ func (suite *UseIntegrationTestSuite) TestShow() {
 	}
 	cp.ExpectExitCode(0)
 
+	err := os.RemoveAll(projectDir)
+	suite.Require().NoError(err)
+
+	cp = ts.SpawnWithOpts(e2e.WithArgs("use", "show"))
+	cp.ExpectLongString("The default project no longer exists")
+	cp.ExpectExitCode(1)
+
 	cp = ts.SpawnWithOpts(e2e.WithArgs("use", "reset", "--non-interactive"))
 	cp.Expect("Reset")
 	cp.ExpectExitCode(0)