made validator fastq extentsion checks consistent with manifest.py

cancerit · Nov 26, 2018 · 80b94b9 · 80b94b9
1 parent ba1d59d
commit 80b94b9
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,5 +1,9 @@
 # CHANGES
 
+## 1.4.1
+
+* `validator` now allows upper case in FastQ file name extensions
+
 ## 1.4.0
 
 * Correct base config for file types accepted.  File_2 should only accept `f(?:ast)q.gz` (upper/lower)

diff --git a/cgp_seq_input_val/seq_validator.py b/cgp_seq_input_val/seq_validator.py
@@ -94,7 +94,8 @@ def _prep(self):
             full_ext = ext
             (base, ext) = os.path.splitext(base)
 
-        if ext not in ('.fastq', '.fq'):
+        if ext.lower() not in ('.fastq', '.fq'):
+            # This check should be consistent with FastQ file extension restrictions in manifest.py
             raise SeqValidationError("Input files must be fastq|fq[.gz]")
 
         full_ext = ext + full_ext

diff --git a/setup.py b/setup.py
@@ -9,7 +9,7 @@
     'url': 'https://github.com/cancerit/cgp_seq_input_val',
     'download_url': '',
     'author_email': '[email protected]',
-    'version': '1.4.0',
+    'version': '1.4.1',
     'python_requires': '>= 3.6',
     'setup_requires': ['pytest'],
     'install_requires': ['progressbar2', 'xlrd', 'xopen'],