We shouldn't be wiping the global elements durint tests

[john-science]

What is the change?

We have been calling elements.factory() during the unit tests, this PR removes that.

Why is the change being made?

In some of our downstream unit testing, which is significantly more parallel, wiping the global elements object from memory, even temporarily, can cause other unit tests to fail.

This has always been quite hard to track down, since it is an async bug that only happens very rarely. Happily, these calls were not necessary in the unit tests in the first place, so we can just remove them.

---

Checklist

• This PR has only one purpose or idea.
• Tests have been added/updated to verify any new/changed code.

• The code style follows good practices.
• The commit message(s) follow good practices.

• The release notes have been updated if necessary.
• The documentation is still up-to-date in the doc folder.
• The dependencies are still up-to-date in pyproject.toml.

[opotowsky]

Easy!! Nice work!