Contributor

Fixes #417

These items are all optional. It's not necessary to update ZKAPAuthorizer for these changes to be valid (they just won't do anything until ZKAPAuthorizer is updated). There is still at least one more related ZKAPAuthorizer change to make before any of this will particularly help GridSync so it's not worth making that update yet.

This makes it easier to identify the ZKAPAuthorizer pieces and easier to test
them directly.

refactor ZKAPAuthorizer optional items handling to reduce duplication

009a5b8

add three new optional config items

9ea7083

black and isort

516d7e0

remove remnants of the old implementation

5086a55

explicit Any is disallowed

8374768

Member

Thanks for this. Looks good to me... My only suggestion here would be to use pytest.mark.parametrize to deduplicate some of those tests (since, in many cases, they structurally equivalent to one another and differ only according to the values being passed/checked) but this is mostly a minor cosmetic preference on my part and I'm happy to merge this as-is.