Applied Ruff formatting and added kwarg to SimpleTestResourceWrapper …

[riebecj]

…to disable debug capture

• Ran Ruff format on source code to increase readability and maintainability
• Added capture_output boolean kwarg to SimpleTestResourceWrapper to disable output capturing for use in test debugging. Defaults to True, so it must be disabled manually. Allow for disabling output capture in SimpleTestResourceWrapper to allow using pdb in tests. #21

[CLAassistant]

All committers have signed the CLA.

[gchqdev227]

Hi @riebecj,

Thank you for opening the PR. It's a bit of a pain, but please could I ask that you undo the ruff formatting you've applied, as it's not a style we're applying across the codebase. I'm aware this isn't mentioned in the contribution guide, so I'll make a note to update that ASAP.

Once that's done I'll take a look at the implementation and run the tests.

[riebecj]

Hi @riebecj,

Thank you for opening the PR. It's a bit of a pain, but please could I ask that you undo the ruff formatting you've applied, as it's not a style we're applying across the codebase. I'm aware this isn't mentioned in the contribution guide, so I'll make a note to update that ASAP.

Once that's done I'll take a look at the implementation and run the tests.

What format do you use within your code? There doesn't seem to be any apparent code formatting.

[gchqdev227]

@riebecj the pre-commit checks will run flake8 to enforce PEP-8, but other than isort for the imports, it mostly comes down to what is the most readable.

That, and double-quote strings!

[sonarqubecloud]

Quality Gate failed

Failed conditions
6.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud