Skip to content

Commit

Permalink
Fix: Repair "uninstall_all"
Browse files Browse the repository at this point in the history
Refs: #2209
- The "uninstall_all" was broken during development of #2120,
to fix the issue a failure when there is no CNF to uninstall
has been relegated to a warning and exit 1 was removed.
- Two spec tests have been added to avoid this scenario
in the future.
- Additionally some renaming was left-over from #2184 that
was resolved.

Signed-off-by: svteb <[email protected]>
  • Loading branch information
svteb committed Jan 17, 2025
1 parent 0140acb commit 0c2e2e6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion spec/setup_spec.cr
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ describe "Installation" do
(/A CNF is already set up. Setting up multiple CNFs is not allowed./ =~ result[:output]).should_not be_nil
ensure
result = ShellCmd.cnf_uninstall()
(/All CNF deployments were uninstalled/ =~ result[:output]).should_not be_nil
(/CNF uninstallation skipped/ =~ result[:output]).should_not be_nil
end
end

Expand Down

0 comments on commit 0c2e2e6

Please sign in to comment.