Skip to content

Commit

Permalink
fix: fixed failing unit tests because of new behavior
Browse files Browse the repository at this point in the history
Signed-off-by: Armin Schlegel <[email protected]>
  • Loading branch information
siredmar committed Mar 1, 2023
1 parent a8cfd64 commit 5c46c3a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions paths_issue_account_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -361,7 +361,7 @@ func TestAccountBeforeOperatorCreatingSysAccount(t *testing.T) {
Data: map[string]interface{}{},
})
assert.NoError(t, err)
assert.False(t, resp.IsError())
assert.True(t, resp.IsError())
// 1.1b create the account
resp, err = b.HandleRequest(context.Background(), &logical.Request{
Operation: logical.CreateOperation,
Expand All @@ -370,7 +370,7 @@ func TestAccountBeforeOperatorCreatingSysAccount(t *testing.T) {
Data: map[string]interface{}{},
})
assert.NoError(t, err)
assert.False(t, resp.IsError())
assert.True(t, resp.IsError())

// 1.2 list the accounts - ac1 should be present
resp, err = b.HandleRequest(context.Background(), &logical.Request{
Expand Down Expand Up @@ -555,7 +555,7 @@ func TestAccountBeforeOperatorAndSysAccount(t *testing.T) {
Data: map[string]interface{}{},
})
assert.NoError(t, err)
assert.False(t, resp.IsError())
assert.True(t, resp.IsError())

// 1.2 list the accounts - ac1 should be present
resp, err = b.HandleRequest(context.Background(), &logical.Request{
Expand Down
2 changes: 1 addition & 1 deletion paths_issue_operator_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -558,6 +558,6 @@ func TestCRUDOperatorIssue(t *testing.T) {
Storage: reqStorage,
})
assert.NoError(t, err)
assert.False(t, resp.IsError())
assert.True(t, resp.IsError())
})
}

0 comments on commit 5c46c3a

Please sign in to comment.