Skip to content

Commit fa893ba

Browse files
committed
update mocks
1 parent 984819b commit fa893ba

File tree

1 file changed

+15
-14
lines changed

1 file changed

+15
-14
lines changed

tests/test_version.py

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -19,20 +19,20 @@ def setUp(self):
1919
version_number="4",
2020
)
2121

22-
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4/checkGenerating"
22+
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4"
2323

2424
@responses.activate
2525
def test_download_raises_exception_on_bad_request(self):
2626
responses.add(responses.GET, self.api_url, status=404, json={"error": "Broken"})
27-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
27+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
2828

2929
with self.assertRaises(RuntimeError):
3030
self.version.download("coco")
3131

3232
@responses.activate
3333
def test_download_raises_exception_on_api_failure(self):
3434
responses.add(responses.GET, self.api_url, status=500)
35-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
35+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
3636
with self.assertRaises(requests.exceptions.HTTPError):
3737
self.version.download("coco")
3838

@@ -42,7 +42,7 @@ def test_download_raises_exception_on_api_failure(self):
4242
@patch.object(Version, "_Version__reformat_yaml")
4343
def test_download_returns_dataset(self, *_):
4444
responses.add(responses.GET, self.api_url, json={"export": { "link": None }})
45-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
45+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
4646
dataset = self.version.download("coco", location="/my-spot")
4747
self.assertEqual(dataset.name, self.version.name)
4848
self.assertEqual(dataset.version, self.version.version)
@@ -55,30 +55,31 @@ def setUp(self):
5555
super(TestExport, self).setUp()
5656
self.api_url = "https://api.roboflow.com/test-workspace/test-project/4/test-format"
5757
self.version = get_version(project_name="Test Dataset", id="test-workspace/test-project/2", version_number="4")
58-
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4/checkGenerating"
58+
59+
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4"
5960

6061
@responses.activate
6162
def test_export_returns_true_on_api_success(self):
6263
responses.add(responses.GET, self.api_url, status=200)
63-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
64+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
6465
export = self.version.export("test-format")
6566
request = responses.calls[0].request
6667

6768
self.assertTrue(export)
6869
self.assertEqual(request.method, "GET")
69-
self.assertDictEqual(request.params, { "api_key": "test-api-key" })
70+
self.assertDictEqual(request.params, {'nocache': 'true', "api_key": "test-api-key" })
7071

7172
@responses.activate
7273
def test_export_raises_error_on_bad_request(self):
7374
responses.add(responses.GET, self.api_url, status=400, json={ "error": "BROKEN!!"})
74-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
75+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
7576
with self.assertRaises(RuntimeError):
7677
self.version.export("test-format")
7778

7879
@responses.activate
7980
def test_export_raises_error_on_api_failure(self):
8081
responses.add(responses.GET, self.api_url, status=500)
81-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
82+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
8283
with self.assertRaises(requests.exceptions.HTTPError):
8384
self.version.export("test-format")
8485

@@ -95,17 +96,17 @@ def setUp(self, *_):
9596

9697
# This is a weird python thing to get access to the private function for testing.
9798
self.get_download_location = self.version._Version__get_download_location
98-
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4/checkGenerating"
99+
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4"
99100

100101
@responses.activate
101102
def test_get_download_location_with_env_variable(self, *_):
102-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
103+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
103104
with patch.dict(os.environ, { "DATASET_DIRECTORY": "/my/exports"}, clear=True):
104105
self.assertEqual(self.get_download_location(), "/my/exports/Test-Dataset-3")
105106

106107
@responses.activate
107108
def test_get_download_location_without_env_variable(self, *_):
108-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
109+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
109110
self.assertEqual(self.get_download_location(), "Test-Dataset-3")
110111

111112

@@ -120,11 +121,11 @@ def setUp(self):
120121

121122
# This is a weird python thing to get access to the private function for testing.
122123
self.get_download_url = self.version._Version__get_download_url
123-
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4/checkGenerating"
124+
self.generating_url = "https://api.roboflow.com/Test Workspace Name/Test Dataset/4"
124125

125126
@responses.activate
126127
def test_get_download_url(self):
127-
responses.add(responses.GET, self.generating_url, json={"generating": False, "progress": 1.0})
128+
responses.add(responses.GET, self.generating_url, json={"version": {"generating": False, "progress": 1.0}})
128129
url = self.get_download_url("yolo1337")
129130
self.assertEqual(
130131
url, "https://api.roboflow.com/test-workspace/test-project/3/yolo1337"

0 commit comments

Comments
 (0)