Changeset 704

Show
Ignore:
Timestamp:
2007-06-03 14:45:25 (1 year ago)
Author:
schwa
Message:
Make sure all names of temporary directories and files have a leading
and a trailing underscore. Thus, it's easier to clean up the ftp test
directory if the tests are interrupted and there are still test
directories or files laying around.
Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/_test_real_ftp.py

    r694 r704  
    128128        host = self.host 
    129129        # no `dir1` yet 
    130         self.failIf('dir1' in host.listdir(host.curdir)) 
     130        self.failIf('_dir1_' in host.listdir(host.curdir)) 
    131131        # vanilla case, all should go well 
    132         host.makedirs('dir1/dir2/dir3/dir4') 
     132        host.makedirs('_dir1_/dir2/dir3/dir4') 
    133133        # check host 
    134         self.failUnless(host.path.isdir('dir1')) 
    135         self.failUnless(host.path.isdir('dir1/dir2')) 
    136         self.failUnless(host.path.isdir('dir1/dir2/dir3')) 
    137         self.failUnless(host.path.isdir('dir1/dir2/dir3/dir4')) 
    138         # clean up 
    139         host.rmdir('dir1/dir2/dir3/dir4') 
    140         host.rmdir('dir1/dir2/dir3') 
    141         host.rmdir('dir1/dir2') 
    142         host.rmdir('dir1') 
     134        self.failUnless(host.path.isdir('_dir1_')) 
     135        self.failUnless(host.path.isdir('_dir1_/dir2')) 
     136        self.failUnless(host.path.isdir('_dir1_/dir2/dir3')) 
     137        self.failUnless(host.path.isdir('_dir1_/dir2/dir3/dir4')) 
     138        # clean up 
     139        host.rmdir('_dir1_/dir2/dir3/dir4') 
     140        host.rmdir('_dir1_/dir2/dir3') 
     141        host.rmdir('_dir1_/dir2') 
     142        host.rmdir('_dir1_') 
    143143 
    144144    def test_makedirs_of_existing_directory(self): 
     
    149149    def test_makedirs_with_file_in_the_way(self): 
    150150        host = self.host 
    151         host.mkdir('dir1') 
    152         self.make_file('dir1/file1') 
     151        host.mkdir('_dir1_') 
     152        self.make_file('_dir1_/file1') 
    153153        # try it 
    154         self.assertRaises(ftp_error.PermanentError, host.makedirs, 'dir1/file1') 
    155154        self.assertRaises(ftp_error.PermanentError, host.makedirs, 
    156                           'dir1/file1/dir2') 
    157         # clean up 
    158         host.unlink('dir1/file1') 
    159         host.rmdir('dir1') 
     155                          '_dir1_/file1') 
     156        self.assertRaises(ftp_error.PermanentError, host.makedirs, 
     157                          '_dir1_/file1/dir2') 
     158        # clean up 
     159        host.unlink('_dir1_/file1') 
     160        host.rmdir('_dir1_') 
    160161 
    161162    def test_makedirs_with_existing_directory(self): 
    162163        host = self.host 
    163         host.mkdir('dir1') 
    164         host.makedirs('dir1/dir2') 
     164        host.mkdir('_dir1_') 
     165        host.makedirs('_dir1_/dir2') 
    165166        # check 
    166         self.failUnless(host.path.isdir('dir1')) 
    167         self.failUnless(host.path.isdir('dir1/dir2')) 
    168         # clean up 
    169         host.rmdir('dir1/dir2') 
    170         host.rmdir('dir1') 
     167        self.failUnless(host.path.isdir('_dir1_')) 
     168        self.failUnless(host.path.isdir('_dir1_/dir2')) 
     169        # clean up 
     170        host.rmdir('_dir1_/dir2') 
     171        host.rmdir('_dir1_') 
    171172 
    172173    def test_makedirs_in_non_writable_directory(self): 
     
    189190        host = self.host 
    190191        # build a tree 
    191         host.makedirs('dir1/dir2') 
    192         self.make_file('dir1/file1') 
    193         self.make_file('dir1/file2') 
    194         self.make_file('dir1/dir2/file3') 
    195         self.make_file('dir1/dir2/file4') 
     192        host.makedirs('_dir1_/dir2') 
     193        self.make_file('_dir1_/file1') 
     194        self.make_file('_dir1_/file2') 
     195        self.make_file('_dir1_/dir2/file3') 
     196        self.make_file('_dir1_/dir2/file4') 
    196197        # try to remove a _file_ with `rmtree` 
    197         self.assertRaises(ftp_error.PermanentError, host.rmtree, 'dir1/file2') 
     198        self.assertRaises(ftp_error.PermanentError, host.rmtree, '_dir1_/file2') 
    198199        # remove dir2 
    199         host.rmtree('dir1/dir2') 
    200         self.failIf(host.path.exists('dir1/dir2')) 
    201         self.failUnless(host.path.exists('dir1/file2')) 
    202         # remake dir2 and remove dir1 
    203         host.mkdir('dir1/dir2') 
    204         self.make_file('dir1/dir2/file3') 
    205         self.make_file('dir1/dir2/file4') 
    206         host.rmtree('dir1') 
    207         self.failIf(host.path.exists('dir1')) 
     200        host.rmtree('_dir1_/dir2') 
     201        self.failIf(host.path.exists('_dir1_/dir2')) 
     202        self.failUnless(host.path.exists('_dir1_/file2')) 
     203        # remake dir2 and remove _dir1_ 
     204        host.mkdir('_dir1_/dir2') 
     205        self.make_file('_dir1_/dir2/file3') 
     206        self.make_file('_dir1_/dir2/file4') 
     207        host.rmtree('_dir1_') 
     208        self.failIf(host.path.exists('_dir1_')) 
    208209 
    209210    def test_rmtree_with_error_handler(self): 
    210211        host = self.host 
    211         host.mkdir('dir1') 
    212         self.make_file('dir1/file1') 
     212        host.mkdir('_dir1_') 
     213        self.make_file('_dir1_/file1') 
    213214        # prepare error "handler" 
    214215        log = [] 
     
    216217            log.append(args) 
    217218        # try to remove a file as root "directory" 
    218         host.rmtree('dir1/file1', ignore_errors=True, onerror=error_handler) 
     219        host.rmtree('_dir1_/file1', ignore_errors=True, onerror=error_handler) 
    219220        self.assertEqual(log, []) 
    220         host.rmtree('dir1/file1', ignore_errors=False, onerror=error_handler) 
     221        host.rmtree('_dir1_/file1', ignore_errors=False, onerror=error_handler) 
    221222        self.assertEqual(log[0][0], host.listdir) 
    222         self.assertEqual(log[0][1], 'dir1/file1') 
     223        self.assertEqual(log[0][1], '_dir1_/file1') 
    223224        self.assertEqual(log[1][0], host.rmdir) 
    224         self.assertEqual(log[1][1], 'dir1/file1') 
    225         host.rmtree('dir1') 
     225        self.assertEqual(log[1][1], '_dir1_/file1') 
     226        host.rmtree('_dir1_') 
    226227        # try to remove a non-existent directory 
    227228        del log[:] 
    228         host.rmtree('dir1', ignore_errors=False, onerror=error_handler) 
     229        host.rmtree('_dir1_', ignore_errors=False, onerror=error_handler) 
    229230        self.assertEqual(log[0][0], host.listdir) 
    230         self.assertEqual(log[0][1], 'dir1') 
     231        self.assertEqual(log[0][1], '_dir1_') 
    231232        self.assertEqual(log[1][0], host.rmdir) 
    232         self.assertEqual(log[1][1], 'dir1') 
     233        self.assertEqual(log[1][1], '_dir1_') 
    233234 
    234235    def test_stat(self): 
     
    355356    def test_rename(self): 
    356357        host = self.host 
    357         self.make_file("testfile1") 
    358         host.rename("testfile1", "testfile2") 
    359         self.failIf(host.path.exists("testfile1")) 
    360         self.failUnless(host.path.exists("testfile2")) 
    361         host.remove("testfile2") 
     358        self.make_file("_testfile1_") 
     359        host.rename("_testfile1_", "_testfile2_") 
     360        self.failIf(host.path.exists("_testfile1_")) 
     361        self.failUnless(host.path.exists("_testfile2_")) 
     362        host.remove("_testfile2_") 
    362363        host.close() 
    363364