Improve the error message when running nested inside a non-Toolbx container

Fixes issue #1183

Signed-off-by: Nieves Montero <[email protected]>

Author: nievesmontero

src/cmd/root.go

@@ -142,6 +142,10 @@ func preRun(cmd *cobra.Command, args []string) error {
 		if _, err := validateSubIDRanges(cmd, args, currentUser); err != nil {
 			return err
 		}
+	} else {
+		if !utils.IsInsideToolboxContainer() {
+			return createErrorNonToolboxContainer()
+		}
 	}
 
 	toolboxPath := os.Getenv("TOOLBOX_PATH")

src/cmd/utils.go

@@ -138,6 +138,16 @@ func createErrorInvalidRelease(hint string) error {
 	return errors.New(errMsg)
 }
 
+func createErrorNonToolboxContainer() error {
+	var builder strings.Builder
+	fmt.Fprintf(&builder, "this is not a Toolbox container.\n")
+	fmt.Fprintf(&builder, "Toolbox is only meant to work with Toolbox containers.\n")
+	fmt.Fprintf(&builder, "Run '%s --help' for usage.", executableBase)
+
+	errMsg := builder.String()
+	return errors.New(errMsg)
+}
+
 func getUsageForCommonCommands() string {
 	var builder strings.Builder
 	fmt.Fprintf(&builder, "create    Create a new toolbox container\n")

test/system/001-version.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers.bash'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
 }
 

test/system/002-help.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers.bash'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
 }
 

test/system/101-create.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
   cleanup_containers
 }

test/system/103-container.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
   cleanup_containers
 }

test/system/104-run.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
   cleanup_containers
 }

test/system/105-enter.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
   cleanup_containers
 }

test/system/106-rm.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
   cleanup_containers
 }

test/system/108-completion.bats

@@ -20,6 +20,7 @@ load 'libs/bats-assert/load'
 load 'libs/helpers'
 
 setup() {
+  bats_require_minimum_version 1.5.0
   _setup_environment
 }