Do not overload docker_client fixtures

[s-t-e-v-e-n-k]

pytest fixtures are now executed in the order they are found, which means a redeclared fixture will take precedence. This means that an attempt to spin up the salt_factories fixture in the container integration tests now has a dependency loop. Remove the overloaded fixture, and perform the check further up the fixture chain.

[dwoz]

@s-t-e-v-e-n-k pre-commit is failing.

[s-t-e-v-e-n-k]

Run bandit against Salt..................................................Failed
- hook id: bandit
- exit code: 1

Traceback (most recent call last):
  File "/home/runner/.cache/pre-commit/repor8zcz89w/py_env-python3/bin/bandit", line 3, in <module>
    from bandit.cli.main import main
  File "/home/runner/.cache/pre-commit/repor8zcz89w/py_env-python3/lib/python3.11/site-packages/bandit/__init__.py", line 5, in <module>
    import pbr.version
ModuleNotFoundError: No module named 'pbr'

@dwoz That seems like a more fundamental CI issue to me