Skip to content

Commit

Permalink
minor changes
Browse files Browse the repository at this point in the history
Signed-off-by: sanjain <[email protected]>
  • Loading branch information
sanjain-px committed Sep 10, 2024
1 parent 534111b commit 76e9520
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions cluster/manager/manager_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -143,9 +143,11 @@ func TestRemoveOnlineNode(t *testing.T) {
err = kvdb.SetInstance(kv)
assert.NoError(t, err)

// when force flag is false, node status check should take precedence
err = testManager.Remove([]api.Node{nodeToRemove}, false)
assert.ErrorContains(t, err, fmt.Sprintf(decommissionErrMsg, testNodeID))

// when force flag is true, we shouldn't abort due to node status
mockListener.EXPECT().String().Return(testNodeID)
mockListener.EXPECT().MarkNodeDown(gomock.Any()).Return(mockErr)

Expand Down

0 comments on commit 76e9520

Please sign in to comment.