diff options
author | BonfaceKilz | 2021-02-16 22:17:41 +0300 |
---|---|---|
committer | BonfaceKilz | 2021-02-16 22:19:31 +0300 |
commit | 0a70905e5b16e372bea9fd3e13a16fbd9152a416 (patch) | |
tree | a13894ad650e64c7a86a26c7d5ae1a198565c0aa /tests/unit | |
parent | 53d81e0d9f66f09b63e5b36bd33c79e8bc930215 (diff) | |
download | genenetwork3-0a70905e5b16e372bea9fd3e13a16fbd9152a416.tar.gz |
Add procedure for uploading gzipped file
* gn3/file_utils.py (extract_uploaded_file): New procedure.
* tests/unit/test_file_utils.py: Test cases for ^^.
* tests/unit/upload-data.tar.gz: New test data.
Diffstat (limited to 'tests/unit')
-rw-r--r-- | tests/unit/test_file_utils.py | 47 | ||||
-rw-r--r-- | tests/unit/upload-data.tar.gz | bin | 0 -> 445 bytes |
2 files changed, 46 insertions, 1 deletions
diff --git a/tests/unit/test_file_utils.py b/tests/unit/test_file_utils.py index f5efa02..166e576 100644 --- a/tests/unit/test_file_utils.py +++ b/tests/unit/test_file_utils.py @@ -1,11 +1,22 @@ """Test cases for procedures defined in file_utils.py""" import os +import shutil import unittest +from dataclasses import dataclass +from typing import Callable from unittest import mock +from gn3.file_utils import extract_uploaded_file +from gn3.file_utils import get_dir_hash from gn3.file_utils import jsonfile_to_dict from gn3.file_utils import lookup_file -from gn3.file_utils import get_dir_hash + + +@dataclass +class MockFile: + """Mock File object returned by request.files""" + filename: str + save: Callable class TestFileUtils(unittest.TestCase): @@ -63,3 +74,37 @@ non-existent""" self.assertRaises(FileNotFoundError, jsonfile_to_dict, "/non-existent-dir") + + @mock.patch("gn3.file_utils.secure_filename") + def test_extract_uploaded_file(self, mock_file): + """Test that the gzip file is extracted to the right location""" + file_loc = os.path.join( + os.path.dirname(__file__), + "upload-data.tar.gz") + mock_file.return_value = file_loc + mock_fileobj = MockFile(save=mock.MagicMock(), + filename="upload-data.tar.gz") + result = extract_uploaded_file(mock_fileobj, "/tmp") + mock_fileobj.save.assert_called_once_with(file_loc) + mock_file.assert_called_once_with("upload-data.tar.gz") + # Clean up! + shutil.rmtree(os.path.join("/tmp", + "d41d8cd98f00b204e9800998ecf8427e")) + self.assertEqual(result, + {"status": 0, + "token": "d41d8cd98f00b204e9800998ecf8427e"}) + + @mock.patch("gn3.file_utils.secure_filename") + def test_extract_uploaded_file_non_existent_gzip(self, mock_file): + """Test that the right error message is returned when there is a problem +extracting the file""" + file_loc = os.path.join( + os.path.dirname(__file__), + "CTtyodSTh5") # Does not exist! + mock_file.return_value = file_loc + mock_fileobj = MockFile(save=mock.MagicMock(), + filename="") + result = extract_uploaded_file(mock_fileobj, "/tmp") + self.assertEqual(result, + {"status": 128, + "error": "gzip failed to unpack file"}) diff --git a/tests/unit/upload-data.tar.gz b/tests/unit/upload-data.tar.gz Binary files differnew file mode 100644 index 0000000..86077d9 --- /dev/null +++ b/tests/unit/upload-data.tar.gz |