RSeQC: Handle empty files nicely.

Closes #1604
MultiQC · Jan 27, 2022 · bae13ef · bae13ef
1 parent a1b4cdf
commit bae13ef
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -59,6 +59,7 @@
 - **RSeQC**
   - Fixed minor bug in new TIN parsing where the sample name was not being correctly cleaned ([#1484](https://github.com/ewels/MultiQC/issues/1484))
   - Fixed bug in the `junction_saturation` submodule ([#1582](https://github.com/ewels/MultiQC/issues/1582))
+  - Fixed bug where empty files caused `tin` submodule to crash ([#1604](https://github.com/ewels/MultiQC/issues/1604))
 - **Mosdepth**
   - Added mean coverage, as requested by [#1566](https://github.com/ewels/MultiQC/issues/1566)
   - Fixed issue [#1568](https://github.com/ewels/MultiQC/issues/1568)

diff --git a/multiqc/modules/rseqc/tin.py b/multiqc/modules/rseqc/tin.py
@@ -23,8 +23,8 @@ def parse_reports(self):
         try:
             reader = csv.DictReader(f["f"], delimiter="\t")
             contents = next(reader)
-        except csv.Error:
-            log.error(f"Could not parse file '{f['fn']}'")
+        except (csv.Error, StopIteration) as e:
+            log.error(f"Could not parse file '{f['fn']}': {e}")
             continue
 
         s_name = self.clean_s_name(contents["Bam_file"], f)