[Test] Add tests that no imports have been added that are not a hard dependency

[patrickvonplaten]

As seen in this PR: #2193 our tests can all pass even if dependencies are added to PRs that are no hard dependencies. The reason for this is that we use torchvision for example testing and thus have it in our docker container, but it's not a hard dependency and therefore at least one test should fail.

=> We should add a new test that verifies this.

Just putting up this issue as a reminder cc @williamberman @patil-suraj @pcuenca

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

[patrickvonplaten]

In case someone has time to pick it up cc @yiyixuxu @sayakpaul @williamberman

[sayakpaul]

@patrickvonplaten, to what extent do we want this to be tested?

For example, one test suite could be added to https://github.com/huggingface/diffusers/tree/main/tests (something like test_dependencies.py) that would check this. I think that should suffice. WDYT?

[patrickvonplaten]

Yes good idea @sayakpaul - agree that this should suffice

[sayakpaul]

@patrickvonplaten bumping this late, sorry.

AFAIK, our Docker container has (which you also mentioned) the extra deps:

diffusers/setup.py

Line 189 in 73b125d

extras["test"] = deps_list(

If we were to write a test for this where would that be executed in the first place?

[patrickvonplaten]

The test would run on the CI at every pull request so we would create a GitHub action here