Addopt --import-mode=importlib for pytest to prevent errors with importlib.metadata

[abravalheri]

Closes #200
Possibly obviates #201

The key aspect for the solution seems to be a pytest update and using importlib mode.

[abravalheri]

Probably having both norecursedirs and testpaths is redundant... (Note however that norecursedirs would give a wider selection of files to test (including for example docs).

docs/conf.py may be dangerous to evaluate as it adds the src folder to sys.path and as established in #200 that causes problems with importlib.metadata.

So we probably just want to remove norecursedirs in a follow-up PR.

[henryiii]

No, I don’t believe they are redundant, because you can manually pass in testpaths. That setting is just a default, while this one makes sure that things that might contain tests (like virtual environments) never get picked up.

[abravalheri]

The macos error is related to a different problem with the recent change in how artifacts are cached in Cirrus CI and should be fixed in a different PR.

[abravalheri]

@mtelka would this work for you?

[abravalheri]

Removing the redundancy.

[mtelka]

I'm happy to report that with git main + this change all tests pass here. Thank you!