From 0bf889fa070179957a78b4b4bda79cd525b4a9e5 Mon Sep 17 00:00:00 2001
From: Bo Peng <ben.bog@gmail.com>
Date: Fri, 25 Jan 2019 15:14:49 -0600
Subject: [PATCH] Change related tests #1186

---
 test/test_parser.py | 49 +++++++++++++++++++++++++--------------------
 1 file changed, 27 insertions(+), 22 deletions(-)

diff --git a/test/test_parser.py b/test/test_parser.py
index f9c2ca577..9f6be9337 100644
--- a/test/test_parser.py
+++ b/test/test_parser.py
@@ -524,20 +524,25 @@ def testParameters(self):
         wf = script.workflow()
         self.assertEqual(sorted(list(wf.parameters().keys())), ['a_b', 'c_b'])
 
-#    this test is no longer valid because sos has stopped parsing assignments
-#
-#    def testSectionVariables(self):
-#        '''Test variables in sections'''
-#        # directive name cannot be used as variable
-#        self.assertRaises(ParsingError, SoS_Script,
-#            '''[0]
-# input='a.txt' ''')
-#        self.assertRaises(ParsingError, SoS_Script,
-#            '''[0]
-# output='a.txt' ''')
-#        self.assertRaises(ParsingError, SoS_Script,
-#            '''[0]
-# depends='a.txt' ''')
+    def testParamInTask(self):
+        '''Test specification of parameters in tasks'''
+        self.assertRaises(Exception, SoS_Script, '''\
+[1]
+output: 'a.txt'
+task:
+parameter: value = 'a'
+sh: expand=True
+echo {value} >  a.txt
+''')
+        # multiple parameters
+        script = SoS_Script('''
+parameter: a_b = int
+[0]
+parameter: c_b = list
+''')
+        wf = script.workflow()
+        self.assertEqual(sorted(list(wf.parameters().keys())), ['a_b', 'c_b'])
+
 
     def testTypeTraitParameter(self):
         # type trait
@@ -1659,10 +1664,10 @@ def testNamedOutputInDepends(self):
 _output.touch()
 
 [default]
-dpends: named_output('A')
+depends: named_output('A')
 ''')
         wf = script.workflow()
-        self.assertRaises(Exception, Base_Executor(wf).run)
+        Base_Executor(wf).run()
 
     def testOutputFromInDepends(self):
         '''Test output_from in depends statement'''
@@ -1672,20 +1677,20 @@ def testOutputFromInDepends(self):
 _output.touch()
 
 [default]
-dpends: output_from('A')
+depends: output_from('A')
 ''')
         wf = script.workflow()
-        self.assertRaises(Exception, Base_Executor(wf).run)
+        Base_Executor(wf).run()
 
     def testNamedOutputInOutput(self):
         '''Test named_output in output statement'''
         script = SoS_Script('''
 [A]
-output: A='a.txt'
+output: Aa='a.txt'
 _output.touch()
 
 [default]
-output: named_output('A')
+output: named_output('Aa')
 ''')
         wf = script.workflow()
         self.assertRaises(Exception, Base_Executor(wf).run)
@@ -1693,12 +1698,12 @@ def testNamedOutputInOutput(self):
     def testOutputFromInOutput(self):
         '''Test output_from in output statement'''
         script = SoS_Script('''
-[A]
+[Aa]
 output: A='a.txt'
 _output.touch()
 
 [default]
-output: output_from('A')
+output: output_from('Aa')
 ''')
         wf = script.workflow()
         self.assertRaises(Exception, Base_Executor(wf).run)