prefer NoError/Error over Nil/NotNil
Kubernetes-commit: 203679cc6105ea490e75af1efa83497b771d7d36
This commit is contained in:
parent
db2da9544d
commit
5f7ddf0f68
|
|
@ -82,7 +82,7 @@ func TestEnforcePolicy(t *testing.T) {
|
||||||
objectFuzzer.Fuzz(e)
|
objectFuzzer.Fuzz(e)
|
||||||
ev, err := EnforcePolicy(e, tc.level, tc.omitStages)
|
ev, err := EnforcePolicy(e, tc.level, tc.omitStages)
|
||||||
if omitSet.Has(string(e.Stage)) {
|
if omitSet.Has(string(e.Stage)) {
|
||||||
require.Nil(t, err)
|
require.NoError(t, err)
|
||||||
require.Nil(t, ev)
|
require.Nil(t, ev)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
@ -136,10 +136,10 @@ func TestEnforcePolicy(t *testing.T) {
|
||||||
expected.Level = tc.level
|
expected.Level = tc.level
|
||||||
require.Equal(t, expected, ev)
|
require.Equal(t, expected, ev)
|
||||||
default:
|
default:
|
||||||
require.NotNil(t, err)
|
require.Error(t, err)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
require.Nil(t, err)
|
require.NoError(t, err)
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue