Depends on D2222
- Group Reviewers
- rDCIDX01f1c385d0a1: tests: update/add copyright/license headers
rDCIDX5150eea140ed: tests: migrate storage tests to pytest
Why not inheritance instead of so many assigments? (Also, non-methods with a "self" argument...)
And I would prefer explicitly skipping/removing tests in each class than whitelisting which ones run
bad name (_w_)
Why an uuid instead of os.random?
because the uuid hypothesis generator is used in gen_content_mimetypes() I guess, so I stuck to it without really thinking about it. And it's a simple way of generating a small random string/bytes. Is there a problem with using uuid there?
not really here (but I could have added them obviously) but I do in test_storage.py for sure
Refactor the tests to prevent these ugly injected functions
doing so, also break the TestIndexerStorageOthers class in pieces
(one for each covered endpoint type).
why so? uuid is a perfectly valid way of generating a small random piece of bytes. Also, it's not clear if this really is a blocker for you or not.