Closed
Description
Follow-up to #1858
We need to add a few more test cases and DRY up the mock preauthorizer. Per Joe's comment on 1858:
- "Consolidate noOpPreAuthorizer and errorPreAuthorizer into a single struct (maybe mockPreAuthorizer?) that has fields for missingRules and err. Then populate them according to the desired test scenario
- We should have tests at this level for:
- missingRules: nil, err: non-nil -> error result includes err (we have this now)
- missingRules: non-nil, err: nil -> error result includes missing rules
- missingRules: non-nil, err: non-nil -> error result includes missing rules and err
- missingRules: nil, err: nil -> no error (we have this now)"
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
No status