diff options
| author | Tim Graham <timograham@gmail.com> | 2018-10-31 19:25:53 -0400 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2018-10-31 20:24:16 -0400 |
| commit | cd7d6c8af7e9ecfd033c584ba50c782d6b82817c (patch) | |
| tree | 9ce72bddf998f187d50b24ee37f61d8ed023c7ac /tests | |
| parent | 0f02d71995530f29a2d3246083854fe19697c3a5 (diff) | |
[2.1.x] Fixed #29890 -- Fixed FileSystemStorage crash if concurrent saves try to create the same directory.
Regression in 632c4ffd9cb1da273303bcd8005fff216506c795.
Backport of 98ef3829e96ebc73d4d446f92465e671ff520d2b from master.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/file_storage/tests.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/file_storage/tests.py b/tests/file_storage/tests.py index 16548b86cb..47d7a20891 100644 --- a/tests/file_storage/tests.py +++ b/tests/file_storage/tests.py @@ -415,9 +415,9 @@ class FileStorageTests(SimpleTestCase): real_makedirs(path) elif path == os.path.join(self.temp_dir, 'raced'): real_makedirs(path) - raise FileNotFoundError() - elif path == os.path.join(self.temp_dir, 'error'): raise FileExistsError() + elif path == os.path.join(self.temp_dir, 'error'): + raise PermissionError() else: self.fail('unexpected argument %r' % path) @@ -432,8 +432,8 @@ class FileStorageTests(SimpleTestCase): with self.storage.open('raced/test.file') as f: self.assertEqual(f.read(), b'saved with race') - # Exceptions aside from FileNotFoundError are raised. - with self.assertRaises(FileExistsError): + # Exceptions aside from FileExistsError are raised. + with self.assertRaises(PermissionError): self.storage.save('error/test.file', ContentFile('not saved')) finally: os.makedirs = real_makedirs |
