ExecTestCase should support args

Bug #353510 reported by Robert Collins
2
Affects Status Importance Assigned to Milestone
subunit
Fix Released
High
Robert Collins

Bug Description

parrot use it to run a vm with a script, to test pynie..

> === modified file 'python/subunit/__init__.py'
> --- python/subunit/__init__.py 2009-03-18 09:33:42 +0000
> +++ python/subunit/__init__.py 2009-03-29 23:59:06 +0000
> @@ -429,7 +429,7 @@
>
> def _run(self, result):
> protocol = TestProtocolServer(result)
> - output = subprocess.Popen([self.script],
> + output = subprocess.Popen(self.script,
>
> stdout=subprocess.PIPE).communicate()[0]
> protocol.readFrom(StringIO(output))
>
>
> may let it work for you with arguments [not tested].

Ah-ha! It needed one more fix, which is:

> + output = subprocess.Popen(self.script, shell=True,

Related branches

Changed in subunit:
assignee: nobody → lifeless
importance: Undecided → High
status: New → Confirmed
Changed in subunit:
status: Confirmed → Fix Committed
Changed in subunit:
milestone: none → 0.0.2
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.