heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make sure workbench won't kill directories out of scope.

Browse Source
This commit is contained in:
Christopher Allan Webber 2011-06-11 18:52:48 -05:00
parent 2616d70903
commit 2ecee34f08
1 changed files with 16 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
mediagoblin/tests/test_workbench.py
View File

@ -17,6 +17,8 @@
import os import os
import tempfile import tempfile
from nose.tools import assert_raises
from mediagoblin.process_media import workbench from mediagoblin.process_media import workbench
@ -32,13 +34,21 @@ class TestWorkbench(object):
def test_destroy_workbench(self): def test_destroy_workbench(self):
# kill a workbench # kill a workbench
workbench = self.workbench_manager.create_workbench() this_workbench = self.workbench_manager.create_workbench()
tmpfile = file(os.path.join(workbench, 'temp.txt'), 'w') tmpfile = file(os.path.join(this_workbench, 'temp.txt'), 'w')
with tmpfile: with tmpfile:
tmpfile.write('lollerskates') tmpfile.write('lollerskates')
assert os.path.exists(os.path.join(workbench, 'temp.txt')) assert os.path.exists(os.path.join(this_workbench, 'temp.txt'))
self.workbench_manager.destroy_workbench(workbench) self.workbench_manager.destroy_workbench(this_workbench)
assert not os.path.exists(os.path.join(workbench, 'temp.txt')) assert not os.path.exists(os.path.join(this_workbench, 'temp.txt'))
assert not os.path.exists(workbench) assert not os.path.exists(this_workbench)
# make sure we can't kill other stuff though
dont_kill_this = tempfile.mkdtemp()
assert_raises(
workbench.WorkbenchOutsideScope,
self.workbench_manager.destroy_workbench,
dont_kill_this)