388 - Adding coverage to pytest-pixl

[mxochicale]

Description

Fixes #388: Adding coverage to pytest-pixl, including the following tests to cover _create_default_json and FtpHostAddress:

• test_create_default_json_file
• test_ftp_host_address

See report in codecov

Type of change

Please delete options accordingly to the description.

• [ ] Bug fix (non-breaking change which fixes an issue)
• [x] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] This change requires a documentation update

Suggested Checklist

• [x] I have performed a self-review of my own code.
• [x] I have made corresponding changes to the documentation.
• [x] My changes generate no new warnings.
• [x] I have commented my code, particularly in hard-to-understand areas.
• [x] I have passed on my local host device. (see further details at the CONTRIBUTING document)
• [x] Make sure your branch is up-to-date with main branch. See CONTRIBUTING for a general example to syncronise your branch with the main branch.
• [x] I have requested PR review to UCLH-Foundtry/arc-dev
• [x] I have adddressed and marked as resolved all the review comments in my PR.
• [x] Finally, I have selected squash and merge

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.37%. Comparing base (7ae5a68) to head (f573520). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #501 +/- ## ========================================== + Coverage 86.08% 87.37% +1.28% ========================================== Files 86 76 -10 Lines 4083 3367 -716 ========================================== - Hits 3515 2942 -573 + Misses 568 425 -143 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[mxochicale]

Hi, this is a good start for covering pytest-pixl. Just requesting your review. Thanks, Miguel.

[mxochicale]

Thanks Stef, let me know your thoughts! Coverage report for dicom.py, ftpserver.py, and plugin.py seem to be >90%. I am not sure if we would like dive in to address the uncovered parts of helpers.