diff options
| author | jc0b <j@jc0b.computer> | 2020-07-29 21:01:23 +0200 |
|---|---|---|
| committer | Fabian Mastenbroek <mail.fabianm@gmail.com> | 2020-08-24 19:48:47 +0200 |
| commit | 3bdfd4dafbe56872dfa00866e04780fa1d20b033 (patch) | |
| tree | 2379d460e9af7d33f933ed7915a0a280d494ea9d /api | |
| parent | 780e81c5ea7bc5c7988ae2c49bc8e6808337ff25 (diff) | |
Restored missing prefab API endpoints
Diffstat (limited to 'api')
| -rw-r--r-- | api/opendc/api/v2/prefabs/prefabId/endpoint.py | 50 | ||||
| -rw-r--r-- | api/opendc/api/v2/prefabs/prefabId/test_endpoint.py | 140 |
2 files changed, 190 insertions, 0 deletions
diff --git a/api/opendc/api/v2/prefabs/prefabId/endpoint.py b/api/opendc/api/v2/prefabs/prefabId/endpoint.py index e69de29b..7b81f546 100644 --- a/api/opendc/api/v2/prefabs/prefabId/endpoint.py +++ b/api/opendc/api/v2/prefabs/prefabId/endpoint.py @@ -0,0 +1,50 @@ +from datetime import datetime + +from opendc.models.prefab import Prefab +from opendc.util.database import Database +from opendc.util.rest import Response + + +def GET(request): + """Get this Prefab.""" + + request.check_required_parameters(path={'prefabId': 'string'}) + + prefab = Prefab.from_id(request.params_path['prefabId']) + prefab.check_exists() + prefab.check_user_access(request.google_id) + + return Response(200, 'Successfully retrieved prefab', prefab.obj) + + +def PUT(request): + """Update a prefab's name and/or contents.""" + + request.check_required_parameters(body={'prefab': {'name': 'name'}}, path={'prefabId': 'string'}) + + prefab = Prefab.from_id(request.params_path['prefabId']) + + prefab.check_exists() + prefab.check_user_access(request.google_id) + + prefab.set_property('name', request.params_body['prefab']['name']) + prefab.set_property('rack', request.params_body['prefab']['rack']) + prefab.set_property('datetime_last_edited', Database.datetime_to_string(datetime.now())) + prefab.update() + + return Response(200, 'Successfully updated prefab.', prefab.obj) + + +def DELETE(request): + """Delete this Prefab.""" + + request.check_required_parameters(path={'prefabId': 'string'}) + + prefab = Prefab.from_id(request.params_path['prefabId']) + + prefab.check_exists() + prefab.check_user_access(request.google_id) + + old_object = prefab.delete() + + return Response(200, 'Successfully deleted prefab.', old_object) diff --git a/api/opendc/api/v2/prefabs/prefabId/test_endpoint.py b/api/opendc/api/v2/prefabs/prefabId/test_endpoint.py index e69de29b..b25c881d 100644 --- a/api/opendc/api/v2/prefabs/prefabId/test_endpoint.py +++ b/api/opendc/api/v2/prefabs/prefabId/test_endpoint.py @@ -0,0 +1,140 @@ +from opendc.util.database import DB +from unittest.mock import Mock + + +def test_get_prefab_non_existing(client, mocker): + mocker.patch.object(DB, 'fetch_one', return_value=None) + assert '404' in client.get('/api/v2/prefabs/1').status + +def test_get_private_prefab_not_authorized(client, mocker): + DB.fetch_one = Mock() + DB.fetch_one.side_effect = [{ + '_id': '1', + 'name': 'test prefab', + 'authorId': '2', + 'visibility': 'private', + 'rack': {} + }, + { + '_id': '1' + } + ] + res = client.get('/api/v2/prefabs/1') + assert '403' in res.status + + +def test_get_private_prefab(client, mocker): + DB.fetch_one = Mock() + DB.fetch_one.side_effect = [{ + '_id': '1', + 'name': 'test prefab', + 'authorId': '1', + 'visibility': 'private', + 'rack': {} + }, + { + '_id': '1' + } + ] + res = client.get('/api/v2/prefabs/1') + assert '200' in res.status + +def test_get_public_prefab(client, mocker): + DB.fetch_one = Mock() + DB.fetch_one.side_effect = [{ + '_id': '1', + 'name': 'test prefab', + 'authorId': '2', + 'visibility': 'public', + 'rack': {} + }, + { + '_id': '1' + } + ] + res = client.get('/api/v2/prefabs/1') + assert '200' in res.status + + +def test_update_prefab_missing_parameter(client): + assert '400' in client.put('/api/v2/prefabs/1').status + + +def test_update_prefab_non_existing(client, mocker): + mocker.patch.object(DB, 'fetch_one', return_value=None) + assert '404' in client.put('/api/v2/prefabs/1', json={'prefab': {'name': 'S'}}).status + + +def test_update_prefab_not_authorized(client, mocker): + DB.fetch_one = Mock() + DB.fetch_one.side_effect = [{ + '_id': '1', + 'name': 'test prefab', + 'authorId': '2', + 'visibility': 'private', + 'rack': {} + }, + { + '_id': '1' + } + ] + mocker.patch.object(DB, 'update', return_value={}) + assert '403' in client.put('/api/v2/prefabs/1', json={'prefab': {'name': 'test prefab', 'rack' : {}}}).status + + +def test_update_prefab(client, mocker): + DB.fetch_one = Mock() + DB.fetch_one.side_effect = [{ + '_id': '1', + 'name': 'test prefab', + 'authorId': '1', + 'visibility': 'private', + 'rack': {} + }, + { + '_id': '1' + } + ] + mocker.patch.object(DB, 'update', return_value={}) + res = client.put('/api/v2/prefabs/1', json={'prefab': {'name': 'test prefab', 'rack' : {}}}) + assert '200' in res.status + + +def test_delete_prefab_non_existing(client, mocker): + mocker.patch.object(DB, 'fetch_one', return_value=None) + assert '404' in client.delete('/api/v2/prefabs/1').status + + +def test_delete_prefab_different_user(client, mocker): + DB.fetch_one = Mock() + DB.fetch_one.side_effect = [{ + '_id': '1', + 'name': 'test prefab', + 'authorId': '2', + 'visibility': 'private', + 'rack': {} + }, + { + '_id': '1' + } + ] + mocker.patch.object(DB, 'delete_one', return_value=None) + assert '403' in client.delete('/api/v2/prefabs/1').status + + +def test_delete_prefab(client, mocker): + DB.fetch_one = Mock() + DB.fetch_one.side_effect = [{ + '_id': '1', + 'name': 'test prefab', + 'authorId': '1', + 'visibility': 'private', + 'rack': {} + }, + { + '_id': '1' + } + ] + mocker.patch.object(DB, 'delete_one', return_value={'prefab': {'name': 'name'}}) + res = client.delete('/api/v2/prefabs/1') + assert '200' in res.status |
