test: add get_args tests

[athith-g]

Adds tests for get_args().

Summary by Sourcery

This pull request introduces unit tests for the get_args function, covering scenarios such as correct argument parsing, default output directory, invalid arguments, and missing file paths.

• Tests:
  • Added unit tests for the get_args function to ensure correct argument parsing, default values, and error handling.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request adds unit tests for the get_args function in the crypt4gh_middleware/decrypt.py file. The tests cover various scenarios including correct argument parsing, default values, and handling of invalid arguments.

File-Level Changes

Files	Changes
tests/decryption/test_decrypt.py	Added unit tests for the get_args function, covering argument parsing, default values, and error handling.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @athith-g - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider adding a test case for multiple file paths to ensure that scenario is handled correctly.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟡 Testing: 2 issues found
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[jvkersch]

LGTM. Maybe one small thing that I would do is to create a helper function to patch sys.argv:

@contextlib.contextmanager
def patch_cli(args):
    with mock.patch("sys.argv", args):
        yield

This would make it a bit easier to track what is being patched IMO, but it is a very minor comment.

[uniqueg]

LGTM :)