Merge pull request #539 from WildernessLabs/runtime_validation

Better messaging if runtime enable/disable fails
WildernessLabs · Mar 19, 2024 · a0555f5 · a0555f5
2 parents 96f17fa + 717097a
commit a0555f5
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/Source/v2/Meadow.Cli/Commands/Current/Runtime/RuntimeDisableCommand.cs b/Source/v2/Meadow.Cli/Commands/Current/Runtime/RuntimeDisableCommand.cs
@@ -28,6 +28,11 @@ protected override async ValueTask ExecuteCommand()
 
             state = await device.IsRuntimeEnabled(CancellationToken);
 
+            if (state == true)
+            {
+                throw new CommandException("Failed to disable runtime");
+            }
+
             Logger?.LogInformation($"Runtime is {(state ? "ENABLED" : "DISABLED")}");
         }
     }

diff --git a/Source/v2/Meadow.Cli/Commands/Current/Runtime/RuntimeEnableCommand.cs b/Source/v2/Meadow.Cli/Commands/Current/Runtime/RuntimeEnableCommand.cs
@@ -28,6 +28,11 @@ protected override async ValueTask ExecuteCommand()
 
             state = await device.IsRuntimeEnabled(CancellationToken);
 
+            if (state == false)
+            {
+                throw new CommandException("Failed to enable runtime");
+            }
+
             Logger?.LogInformation($"Runtime is {(state ? "ENABLED" : "DISABLED")}");
         }
     }