Run tests if skip condition not met

[amyeroberts]

What does this PR do?

Some tests that had unittest.skipIf decorators weren't calling a test if the condition was false. This mean some test would be showing as passing even though they weren't being run.

Before submitting

• This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
• Did you read the contributor guideline,
  Pull Request section?
• Was this discussed/approved via a Github issue or the forum? Please add a link
  to it if that's the case.
• Did you make sure to update the documentation with your changes? Here are the
  documentation guidelines, and
  here are tips on formatting docstrings.
• Did you write any new necessary tests?

[HuggingFaceDocBuilderDev]

The documentation is not available anymore as the PR was closed or merged.

[amyeroberts]

cc @Rocketknight1

[gante]

👍

If it's not asking too much -- can you also remove the (<=2.8) from the reason? I think the reason is also true for TF 2.9 (and looking at the release notes, for 2.10 as well)

[amyeroberts]

@gante I removed the (<=2.8) from all grouped conv comments in the repo

[ydshieh]

Very nice catch! Thank you for the fix.

[LysandreJik]

LGTM, thank you @amyeroberts!