After implementing proper tests get a working implementation of runfolder name from...

[htsworkflow.git] / htsworkflow / pipelines / ipar.py

diff --git a/htsworkflow/pipelines/ipar.py b/htsworkflow/pipelines/ipar.py
index 8203f5e0f29fda173728e3f504522f6cd579520f..6c3acbe19b50c8959b5deb011e36eb9cf7396212 100644 (file)
--- a/htsworkflow/pipelines/ipar.py
+++ b/htsworkflow/pipelines/ipar.py
@@ -79,9 +79,14 @@ class IPAR(object):
         """Return runfolder name"""
         if self.tree is None:
             raise ValueError("Can't query an empty run")
-        runfolder = self.tree.xpath('RunParameters/Runfolder')
-        return runfolder
-    runfolder_name = property(_get_runfolder)
+        runfolder = self.tree.xpath('RunParameters/RunFolder')
+        if len(runfolder) == 0:
+            return None
+        elif len(runfolder) > 1:
+            raise RuntimeError("RunXml parse error looking for RunFolder")
+        else:
+            return runfolder[0].text
+    runfolder_name = property(_get_runfolder_name)
     
     def _get_software(self):
         """Return software name"""