Fix deliverable names for taxprofiler

[sofstam]

Description

Changed

• This PR removes the _pe suffix from the fixtures in taxprofiler
• The multiqc files (multiqc_general_stats.json, multiqc_fastp.json) are removed because with 1.28 version, they were moved to multiqc_data.json file instead.

Fixed

Testing

How to prepare for test

• ssh to Hasta
• Test your branch with

hermes-test-deploy <branch-name>
hermes-test <command here>

How to test

• login to ...
• do ...

Expected test result

• check that ...
• Take a screenshot and attach or copy/paste the output.

Review

• code approved by
• tests executed by
• "Merge and deploy" approved by
  Thanks for filling in who performed the code review and the test!

This version is a:

• MAJOR - when you make incompatible API changes
• MINOR - when you add functionality in a backwards compatible manner
• PATCH - when you make backwards compatible bug fixes or documentation/instructions

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.78%. Comparing base (30521a8) to head (c876156).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #155   +/-   ##
=======================================
  Coverage   92.78%   92.78%           
=======================================
  Files          28       28           
  Lines        1012     1012           
=======================================
  Hits          939      939           
  Misses         73       73           

Flag	Coverage Δ
unittests	92.78% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sofstam]

[fellen31]

LGTM!

[fellen31]

Just, there's no description in the PR why the MultiQC files are removed.

[sofstam]

Just, there's no description in the PR why the MultiQC files are removed.

Updated the description, hope it is clear now.

[peterpru]

Looks good to me. They also exist in cg in cg/resources/taxprofiler_bundle_filenames.yaml if you wanted to remove them completely.

[sofstam]

Looks good to me. They also exist in cg in cg/resources/taxprofiler_bundle_filenames.yaml if you wanted to remove them completely.

I removed the corresponding multiqc files from cg/resources/taxprofiler_bundle_filenames.yaml. Clinical-Genomics/cg#4839