Skip to content

Commit

Permalink
Remove redundant case in outcome spec
Browse files Browse the repository at this point in the history
  • Loading branch information
yfyf committed Nov 28, 2024
1 parent 395b806 commit 1fc88d5
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions controller/tests/server/update/outcome.ml
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ let state_matches_expected_outcome state outcome =
| (DoNothingOrProduceWarning, OutOfDateVersionSelected, Sleeping _) -> true
| (DoNothingOrProduceWarning, RebootRequired, Sleeping _) -> true
| (DoNothingOrProduceWarning, ReinstallRequired, Sleeping _) -> true
(* should not _directly_ return to GettingVersionInfo state *) (*TODO: redundant now *)
| (DoNothingOrProduceWarning, _, Update.GettingVersionInfo) -> false
(* all the other state combos are treated as errors *)
| (DoNothingOrProduceWarning, _, _) -> false

Expand Down

0 comments on commit 1fc88d5

Please sign in to comment.