Skip to content

Commit 9cdb70a

Browse files
edmundmillerclaude
andcommitted
fix: Improve samtools flagstat stub to generate realistic output
- Replace empty touch command with proper flagstat format in stub - Include realistic read counts and mapping statistics - Ensure stub output matches expected flagstat format for parsing - Fixes stub workflow tests that depend on flagstat content 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 7b18579 commit 9cdb70a

File tree

4 files changed

+49
-4
lines changed

4 files changed

+49
-4
lines changed

modules.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,8 @@
129129
"samtools/flagstat": {
130130
"branch": "master",
131131
"git_sha": "46eca555142d6e597729fcb682adcc791796f514",
132-
"installed_by": ["bam_stats_samtools", "modules"]
132+
"installed_by": ["bam_stats_samtools", "modules"],
133+
"patch": "modules/nf-core/samtools/flagstat/samtools-flagstat.diff"
133134
},
134135
"samtools/idxstats": {
135136
"branch": "master",

modules/nf-core/samtools/flagstat/main.nf

Lines changed: 13 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

modules/nf-core/samtools/flagstat/samtools-flagstat.diff

Lines changed: 32 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/skip_trimming.nf.test.snap

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -858,12 +858,12 @@
858858
},
859859
"skip_trimming with stub": {
860860
"content": [
861-
137
861+
190
862862
],
863863
"meta": {
864864
"nf-test": "0.9.2",
865865
"nextflow": "25.04.6"
866866
},
867-
"timestamp": "2025-09-05T10:08:54.671755"
867+
"timestamp": "2025-09-05T10:38:29.093856"
868868
}
869869
}

0 commit comments

Comments
 (0)