Fix looper args bug. Fix #41

pepkit · Feb 16, 2017 · 9e7db15 · 9e7db15
1 parent a2bbe11
commit 9e7db15
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/looper/models.py b/looper/models.py
@@ -1245,7 +1245,7 @@ def select_pipeline(self, pipeline_name):
 	def uses_looper_args(self, pipeline_name):
 		config = self.select_pipeline(pipeline_name)
 
-		if hasattr(config, "looper_args") and config.looper_args:
+		if config.has_key("looper_args") and config["looper_args"]:
 			return True
 		else:
 			return False

diff --git a/test_looper.py b/test_looper.py
@@ -84,9 +84,9 @@ def test1(self):
 
 		self.assertTrue(s3.confirm_required_inputs())
 
-
-
-
+		# test uses_looper_args
+		self.assertTrue(pi.uses_looper_args("testngs.sh"))
+		self.assertFalse(pi.uses_looper_args("testpipeline.sh"))
 
 
 

diff --git a/tests/pipelines/config/pipeline_interface.yaml b/tests/pipelines/config/pipeline_interface.yaml
@@ -1,6 +1,6 @@
 testpipeline.sh:
   name: test_pipeline  # Name used by pypiper so looper can find the logs
-  looper_args: True
+  looper_args: False
   arguments:
     "--input": file
   optional_arguments: