Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 1 | """Test the API for crashreports, devices, heartbeats and logfiles.""" |
Franz-Xaver Geiger | 4d022d5 | 2018-03-20 13:12:49 +0100 | [diff] [blame] | 2 | import os |
| 3 | import tempfile |
Mitja Nikolaus | 9c3b29e | 2018-08-22 11:17:50 +0200 | [diff] [blame^] | 4 | from typing import Optional |
Franz-Xaver Geiger | 4d022d5 | 2018-03-20 13:12:49 +0100 | [diff] [blame] | 5 | |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 6 | from django.contrib.auth.models import User |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 7 | from django.urls import reverse |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 8 | |
Dirk Vogt | 3663569 | 2016-10-17 12:19:10 +0200 | [diff] [blame] | 9 | from rest_framework import status |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 10 | from rest_framework.test import APIClient, APITestCase |
Dirk Vogt | f130c75 | 2016-08-23 14:45:01 +0200 | [diff] [blame] | 11 | |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 12 | from crashreports.models import Crashreport |
| 13 | |
| 14 | |
| 15 | class InvalidCrashTypeError(BaseException): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 16 | """Invalid crash type encountered. |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 17 | |
| 18 | The valid crash type values (strings) are: |
| 19 | - 'crash'; |
| 20 | - 'smpl'; |
| 21 | - 'other'. |
| 22 | |
| 23 | Args: |
| 24 | - crash_type: The invalid crash type. |
| 25 | """ |
| 26 | |
| 27 | def __init__(self, crash_type): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 28 | """Initialise the exception using the crash type to build a message. |
| 29 | |
| 30 | Args: |
| 31 | crash_type: The invalid crash type. |
| 32 | """ |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 33 | super(InvalidCrashTypeError, self).__init__( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 34 | "{} is not a valid crash type".format(crash_type) |
| 35 | ) |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 36 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 37 | |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 38 | class Dummy: |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 39 | """Dummy values for devices, heartbeats and crashreports.""" |
| 40 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 41 | DEFAULT_DUMMY_DEVICE_REGISTER_VALUES = { |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 42 | "board_date": "2015-12-15T01:23:45Z", |
| 43 | "chipset": "Qualcomm MSM8974PRO-AA", |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 44 | } |
| 45 | |
| 46 | DEFAULT_DUMMY_HEARTBEAT_VALUES = { |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 47 | "uuid": None, |
| 48 | "app_version": 10100, |
| 49 | "uptime": ( |
| 50 | "up time: 16 days, 21:49:56, idle time: 5 days, 20:55:04, " |
| 51 | "sleep time: 10 days, 20:46:27" |
| 52 | ), |
| 53 | "build_fingerprint": ( |
| 54 | "Fairphone/FP2/FP2:6.0.1/FP2-gms-18.03.1/FP2-gms-18.03.1:user/" |
| 55 | "release-keys" |
| 56 | ), |
| 57 | "radio_version": "4437.1-FP2-0-08", |
| 58 | "date": "2018-03-19T09:58:30.386Z", |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 59 | } |
| 60 | |
| 61 | DEFAULT_DUMMY_CRASHREPORTS_VALUES = DEFAULT_DUMMY_HEARTBEAT_VALUES.copy() |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 62 | DEFAULT_DUMMY_CRASHREPORTS_VALUES.update( |
| 63 | { |
| 64 | "is_fake_report": 0, |
| 65 | "boot_reason": "why?", |
| 66 | "power_on_reason": "it was powered on", |
| 67 | "power_off_reason": "something happened and it went off", |
| 68 | } |
| 69 | ) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 70 | |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 71 | CRASH_TYPE_TO_BOOT_REASON_MAP = { |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 72 | "crash": Crashreport.BOOT_REASON_KEYBOARD_POWER_ON, |
| 73 | "smpl": Crashreport.BOOT_REASON_RTC_ALARM, |
| 74 | "other": "whatever", |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 75 | } |
| 76 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 77 | @staticmethod |
| 78 | def _update_copy(original, update): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 79 | """Merge fields of update into a copy of original.""" |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 80 | data = original.copy() |
| 81 | data.update(update) |
| 82 | return data |
| 83 | |
| 84 | @staticmethod |
| 85 | def device_register_data(**kwargs): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 86 | """Return the data required to register a device. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 87 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 88 | Use the values passed as keyword arguments or default to the ones |
| 89 | from `Dummy.DEFAULT_DUMMY_DEVICE_REGISTER_VALUES`. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 90 | """ |
| 91 | return Dummy._update_copy( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 92 | Dummy.DEFAULT_DUMMY_DEVICE_REGISTER_VALUES, kwargs |
| 93 | ) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 94 | |
| 95 | @staticmethod |
| 96 | def heartbeat_data(**kwargs): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 97 | """Return the data required to create a heartbeat. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 98 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 99 | Use the values passed as keyword arguments or default to the ones |
| 100 | from `Dummy.DEFAULT_DUMMY_HEARTBEAT_VALUES`. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 101 | """ |
| 102 | return Dummy._update_copy(Dummy.DEFAULT_DUMMY_HEARTBEAT_VALUES, kwargs) |
| 103 | |
| 104 | @staticmethod |
Mitja Nikolaus | 9c3b29e | 2018-08-22 11:17:50 +0200 | [diff] [blame^] | 105 | def crashreport_data(report_type: Optional[str] = None, **kwargs): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 106 | """Return the data required to create a crashreport. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 107 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 108 | Use the values passed as keyword arguments or default to the ones |
| 109 | from `Dummy.DEFAULT_DUMMY_CRASHREPORTS_VALUES`. |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 110 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 111 | Args: |
Mitja Nikolaus | 9c3b29e | 2018-08-22 11:17:50 +0200 | [diff] [blame^] | 112 | report_type: A valid value from |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 113 | `Dummy.CRASH_TYPE_TO_BOOT_REASON_MAP.keys()` that will |
| 114 | define the boot reason if not explicitly defined in the |
| 115 | keyword arguments already. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 116 | """ |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 117 | data = Dummy._update_copy( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 118 | Dummy.DEFAULT_DUMMY_CRASHREPORTS_VALUES, kwargs |
| 119 | ) |
| 120 | if report_type and "boot_reason" not in kwargs: |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 121 | if report_type not in Dummy.CRASH_TYPE_TO_BOOT_REASON_MAP: |
| 122 | raise InvalidCrashTypeError(report_type) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 123 | data["boot_reason"] = Dummy.CRASH_TYPE_TO_BOOT_REASON_MAP.get( |
| 124 | report_type |
| 125 | ) |
Borjan Tchakaloff | fe5f25c | 2018-03-19 18:33:24 +0400 | [diff] [blame] | 126 | return data |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 127 | |
| 128 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 129 | class DeviceRegisterAPITestCase(APITestCase): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 130 | """Base class that offers a device registration method.""" |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 131 | |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 132 | REGISTER_DEVICE_URL = "api_v1_register_device" |
| 133 | |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 134 | def setUp(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 135 | """Create an admin user for accessing the API. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 136 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 137 | The APIClient that can be used to make authenticated requests to the |
| 138 | server is stored in self.admin. |
| 139 | """ |
| 140 | admin_user = User.objects.create_superuser( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 141 | "somebody", "somebody@example.com", "thepassword" |
| 142 | ) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 143 | self.admin = APIClient() |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 144 | self.admin.force_authenticate(admin_user) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 145 | |
| 146 | def _register_device(self, **kwargs): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 147 | """Register a new device. |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 148 | |
| 149 | Arguments: |
| 150 | **kwargs: The data to pass the dummy data creation |
| 151 | method `Dummy.device_register_data`. |
| 152 | Returns: |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 153 | (UUID, APIClient, str): The uuid of the new device as well as an |
| 154 | authentication token and the associated user with credentials. |
| 155 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 156 | """ |
| 157 | data = Dummy.device_register_data(**kwargs) |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 158 | response = self.client.post(reverse(self.REGISTER_DEVICE_URL), data) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 159 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 160 | |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 161 | uuid = response.data["uuid"] |
| 162 | token = response.data["token"] |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 163 | user = APIClient() |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 164 | user.credentials(HTTP_AUTHORIZATION="Token " + token) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 165 | |
| 166 | return uuid, user, token |
| 167 | |
| 168 | |
| 169 | class DeviceTestCase(DeviceRegisterAPITestCase): |
| 170 | """Test cases for registering devices.""" |
| 171 | |
| 172 | def test_register(self): |
| 173 | """Test registration of devices.""" |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 174 | response = self.client.post( |
| 175 | reverse(self.REGISTER_DEVICE_URL), Dummy.device_register_data() |
| 176 | ) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 177 | self.assertTrue("token" in response.data) |
| 178 | self.assertTrue("uuid" in response.data) |
| 179 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
| 180 | |
| 181 | def test_create_missing_fields(self): |
| 182 | """Test registration with missing fields.""" |
| 183 | response = self.client.post(reverse(self.REGISTER_DEVICE_URL)) |
| 184 | self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST) |
| 185 | |
| 186 | def test_create_missing_board_date(self): |
| 187 | """Test registration with missing board date.""" |
| 188 | data = Dummy.device_register_data() |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 189 | data.pop("board_date") |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 190 | response = self.client.post(reverse(self.REGISTER_DEVICE_URL), data) |
| 191 | self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST) |
| 192 | |
| 193 | def test_create_missing_chipset(self): |
| 194 | """Test registration with missing chipset.""" |
| 195 | data = Dummy.device_register_data() |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 196 | data.pop("chipset") |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 197 | response = self.client.post(reverse(self.REGISTER_DEVICE_URL), data) |
| 198 | self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST) |
| 199 | |
| 200 | def test_create_invalid_board_date(self): |
| 201 | """Test registration with invalid board date.""" |
| 202 | data = Dummy.device_register_data() |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 203 | data["board_date"] = "not_a_valid_date" |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 204 | response = self.client.post(reverse(self.REGISTER_DEVICE_URL), data) |
| 205 | self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST) |
| 206 | |
| 207 | def test_create_non_existent_time_board_date(self): |
| 208 | """Test registration with non existing time. |
| 209 | |
| 210 | Test the resolution of a naive date-time in which the |
| 211 | Europe/Amsterdam daylight saving time transition moved the time |
| 212 | "forward". The server should not crash when receiving a naive |
| 213 | date-time which does not exist in the server timezone or locale. |
| 214 | """ |
| 215 | data = Dummy.device_register_data() |
| 216 | # In 2017, the Netherlands changed from CET to CEST on March, |
| 217 | # 26 at 02:00 |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 218 | data["board_date"] = "2017-03-26 02:34:56" |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 219 | response = self.client.post(reverse(self.REGISTER_DEVICE_URL), data) |
| 220 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
| 221 | |
| 222 | def test_create_ambiguous_time_board_date(self): |
| 223 | """Test registration with ambiguous time. |
| 224 | |
| 225 | Test the resolution of a naive date-time in which the |
| 226 | Europe/Amsterdam daylight saving time transition moved the time |
| 227 | "backward". The server should not crash when receiving a naive |
| 228 | date-time that can belong to multiple timezones. |
| 229 | """ |
| 230 | data = Dummy.device_register_data() |
| 231 | # In 2017, the Netherlands changed from CEST to CET on October, |
| 232 | # 29 at 03:00 |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 233 | data["board_date"] = "2017-10-29 02:34:56" |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 234 | response = self.client.post(reverse(self.REGISTER_DEVICE_URL), data) |
| 235 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 236 | |
| 237 | |
| 238 | class ListDevicesTestCase(DeviceRegisterAPITestCase): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 239 | """Test cases for listing and deleting devices.""" |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 240 | |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 241 | LIST_CREATE_URL = "api_v1_list_devices" |
| 242 | RETRIEVE_URL = "api_v1_retrieve_device" |
| 243 | |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 244 | def test_device_list(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 245 | """Test registration of 2 devices.""" |
| 246 | number_of_devices = 2 |
| 247 | uuids = [ |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 248 | str(self._register_device()[0]) for _ in range(number_of_devices) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 249 | ] |
| 250 | |
| 251 | response = self.admin.get(reverse(self.LIST_CREATE_URL), {}) |
| 252 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 253 | self.assertEqual(len(response.data["results"]), number_of_devices) |
| 254 | for result in response.data["results"]: |
| 255 | self.assertIn(result["uuid"], uuids) |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 256 | |
| 257 | def test_device_list_unauth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 258 | """Test listing devices without authentication.""" |
| 259 | response = self.client.get(reverse(self.LIST_CREATE_URL), {}) |
| 260 | self.assertEqual(response.status_code, status.HTTP_401_UNAUTHORIZED) |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 261 | |
| 262 | def test_retrieve_device_auth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 263 | """Test retrieval of devices as admin user.""" |
| 264 | uuid, _, token = self._register_device() |
| 265 | response = self.admin.get(reverse(self.RETRIEVE_URL, args=[uuid]), {}) |
| 266 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 267 | self.assertEqual(response.data["uuid"], str(uuid)) |
| 268 | self.assertEqual(response.data["token"], token) |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 269 | |
| 270 | def test_retrieve_device_unauth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 271 | """Test retrieval of devices without authentication.""" |
| 272 | uuid, _, _ = self._register_device() |
| 273 | response = self.client.get(reverse(self.RETRIEVE_URL, args=[uuid]), {}) |
| 274 | self.assertEqual(response.status_code, status.HTTP_401_UNAUTHORIZED) |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 275 | |
| 276 | def test_delete_device_auth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 277 | """Test deletion of devices as admin user.""" |
| 278 | uuid, _, _ = self._register_device() |
| 279 | url = reverse(self.RETRIEVE_URL, args=[uuid]) |
| 280 | response = self.admin.delete(url, {}) |
| 281 | self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT) |
| 282 | response = self.admin.delete(url, {}) |
| 283 | self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND) |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 284 | |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 285 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 286 | class HeartbeatListTestCase(DeviceRegisterAPITestCase): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 287 | """Test cases for heartbeats.""" |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 288 | |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 289 | LIST_CREATE_URL = "api_v1_heartbeats" |
| 290 | RETRIEVE_URL = "api_v1_heartbeat" |
| 291 | LIST_CREATE_BY_UUID_URL = "api_v1_heartbeats_by_uuid" |
| 292 | RETRIEVE_BY_UUID_URL = "api_v1_heartbeat_by_uuid" |
| 293 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 294 | @staticmethod |
| 295 | def _create_dummy_data(**kwargs): |
| 296 | return Dummy.heartbeat_data(**kwargs) |
| 297 | |
| 298 | def _post_multiple(self, client, data, count): |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 299 | return [ |
| 300 | client.post(reverse(self.LIST_CREATE_URL), data) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 301 | for _ in range(count) |
| 302 | ] |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 303 | |
| 304 | def _retrieve_single(self, user): |
| 305 | count = 5 |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 306 | response = self._post_multiple(self.admin, self.data, count) |
| 307 | self.assertEqual(len(response), count) |
| 308 | self.assertEqual(response[0].status_code, status.HTTP_201_CREATED) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 309 | url = reverse(self.RETRIEVE_URL, args=[response[0].data["id"]]) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 310 | request = user.get(url) |
| 311 | return request.status_code |
| 312 | |
| 313 | def _retrieve_single_by_device(self, user): |
| 314 | count = 5 |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 315 | response = self._post_multiple(self.user, self.data, count) |
| 316 | self.assertEqual(len(response), count) |
| 317 | self.assertEqual(response[0].status_code, status.HTTP_201_CREATED) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 318 | url = reverse( |
| 319 | self.RETRIEVE_BY_UUID_URL, |
| 320 | args=[self.uuid, response[0].data["device_local_id"]], |
| 321 | ) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 322 | request = user.get(url) |
| 323 | return request.status_code |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 324 | |
| 325 | def setUp(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 326 | """Set up a device and some data.""" |
| 327 | super().setUp() |
| 328 | self.uuid, self.user, self.token = self._register_device() |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 329 | self.data = self._create_dummy_data(uuid=self.uuid) |
Dirk Vogt | 67eb148 | 2016-10-13 12:42:56 +0200 | [diff] [blame] | 330 | |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 331 | def test_create_no_auth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 332 | """Test creation without authentication.""" |
| 333 | noauth_client = APIClient() |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 334 | response = noauth_client.post(reverse(self.LIST_CREATE_URL), self.data) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 335 | self.assertEqual(response.status_code, status.HTTP_401_UNAUTHORIZED) |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 336 | |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 337 | def test_create_as_admin(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 338 | """Test creation as admin.""" |
| 339 | response = self.admin.post(reverse(self.LIST_CREATE_URL), self.data) |
| 340 | self.assertEqual(response.status_code, status.HTTP_201_CREATED) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 341 | self.assertTrue(response.data["id"] > 0) |
Dirk Vogt | f2a3342 | 2016-10-11 17:17:26 +0200 | [diff] [blame] | 342 | |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 343 | def test_create_as_admin_not_existing_device(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 344 | """Test creation of heartbeat on non-existing device.""" |
| 345 | response = self.admin.post( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 346 | reverse(self.LIST_CREATE_URL), self._create_dummy_data() |
| 347 | ) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 348 | self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND) |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 349 | |
| 350 | def test_create_as_uuid_owner(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 351 | """Test creation as owner.""" |
| 352 | response = self.user.post( |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 353 | reverse(self.LIST_CREATE_URL), |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 354 | self._create_dummy_data(uuid=self.uuid), |
| 355 | ) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 356 | self.assertEqual(response.status_code, status.HTTP_201_CREATED) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 357 | self.assertEqual(response.data["id"], -1) |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 358 | |
| 359 | def test_create_as_uuid_not_owner(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 360 | """Test creation as non-owner.""" |
| 361 | uuid, _, _ = self._register_device() |
| 362 | response = self.user.post( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 363 | reverse(self.LIST_CREATE_URL), self._create_dummy_data(uuid=uuid) |
| 364 | ) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 365 | self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 366 | |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 367 | def test_list(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 368 | """Test listing of heartbeats.""" |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 369 | count = 5 |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 370 | self._post_multiple(self.user, self.data, count) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 371 | response = self.admin.get(reverse(self.LIST_CREATE_URL)) |
| 372 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 373 | self.assertEqual(len(response.data["results"]), count) |
Franz-Xaver Geiger | d612fd2 | 2018-02-28 10:33:08 +0100 | [diff] [blame] | 374 | |
| 375 | def test_retrieve_single_admin(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 376 | """Test retrieval as admin.""" |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 377 | self.assertEqual(self._retrieve_single(self.admin), status.HTTP_200_OK) |
Dirk Vogt | e178488 | 2016-10-13 16:09:38 +0200 | [diff] [blame] | 378 | |
| 379 | def test_retrieve_single_device_owner(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 380 | """Test retrieval as device owner.""" |
Franz-Xaver Geiger | d612fd2 | 2018-02-28 10:33:08 +0100 | [diff] [blame] | 381 | self.assertEqual( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 382 | self._retrieve_single(self.user), status.HTTP_403_FORBIDDEN |
| 383 | ) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 384 | |
| 385 | def test_retrieve_single_noauth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 386 | """Test retrieval without authentication.""" |
| 387 | noauth_client = APIClient() |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 388 | self.assertEqual( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 389 | self._retrieve_single(noauth_client), status.HTTP_401_UNAUTHORIZED |
| 390 | ) |
Dirk Vogt | e178488 | 2016-10-13 16:09:38 +0200 | [diff] [blame] | 391 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 392 | def test_retrieve_single_by_device_admin(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 393 | """Test retrieval by device as admin.""" |
Franz-Xaver Geiger | d612fd2 | 2018-02-28 10:33:08 +0100 | [diff] [blame] | 394 | self.assertEqual( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 395 | self._retrieve_single_by_device(self.admin), status.HTTP_200_OK |
| 396 | ) |
Dirk Vogt | 0d9d5d2 | 2016-10-13 16:17:57 +0200 | [diff] [blame] | 397 | |
| 398 | def test_retrieve_single_by_device_device_owner(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 399 | """Test retrieval by device as owner.""" |
Franz-Xaver Geiger | d612fd2 | 2018-02-28 10:33:08 +0100 | [diff] [blame] | 400 | self.assertEqual( |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 401 | self._retrieve_single_by_device(self.user), |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 402 | status.HTTP_403_FORBIDDEN, |
| 403 | ) |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 404 | |
| 405 | def test_retrieve_single_by_device_noauth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 406 | """Test retrieval by device without authentication.""" |
| 407 | noauth_client = APIClient() |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 408 | self.assertEqual( |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 409 | self._retrieve_single_by_device(noauth_client), |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 410 | status.HTTP_401_UNAUTHORIZED, |
| 411 | ) |
Dirk Vogt | 0d9d5d2 | 2016-10-13 16:17:57 +0200 | [diff] [blame] | 412 | |
Dirk Vogt | e178488 | 2016-10-13 16:09:38 +0200 | [diff] [blame] | 413 | def test_list_by_uuid(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 414 | """Test listing of devices by UUID.""" |
Dirk Vogt | e178488 | 2016-10-13 16:09:38 +0200 | [diff] [blame] | 415 | count = 5 |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 416 | uuid, _, _ = self._register_device() |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 417 | self._post_multiple(self.user, self.data, count) |
| 418 | self._post_multiple( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 419 | self.admin, self._create_dummy_data(uuid=uuid), count |
| 420 | ) |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 421 | url = reverse(self.LIST_CREATE_BY_UUID_URL, args=[self.uuid]) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 422 | response = self.admin.get(url) |
| 423 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 424 | self.assertEqual(len(response.data["results"]), count) |
Dirk Vogt | e178488 | 2016-10-13 16:09:38 +0200 | [diff] [blame] | 425 | |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 426 | def test_list_noauth(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 427 | """Test listing of devices without authentication.""" |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 428 | count = 5 |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 429 | noauth_client = APIClient() |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 430 | self._post_multiple(self.user, self.data, count) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 431 | response = noauth_client.get(reverse(self.LIST_CREATE_URL)) |
| 432 | self.assertEqual(response.status_code, status.HTTP_401_UNAUTHORIZED) |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 433 | |
| 434 | def test_list_device_owner(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 435 | """Test listing as device owner.""" |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 436 | count = 5 |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 437 | self._post_multiple(self.user, self.data, count) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 438 | response = self.user.get(reverse(self.LIST_CREATE_URL)) |
| 439 | self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 440 | |
Borjan Tchakaloff | 869cf92 | 2018-02-19 11:01:16 +0100 | [diff] [blame] | 441 | def test_no_radio_version(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 442 | """Test creation and retrieval without radio version.""" |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 443 | data = self._create_dummy_data(uuid=self.uuid) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 444 | data.pop("radio_version") |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 445 | response = self.user.post(reverse(self.LIST_CREATE_URL), data) |
| 446 | self.assertEqual(response.status_code, status.HTTP_201_CREATED) |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 447 | url = reverse(self.LIST_CREATE_BY_UUID_URL, args=[self.uuid]) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 448 | response = self.admin.get(url) |
| 449 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 450 | self.assertEqual(len(response.data["results"]), 1) |
| 451 | self.assertIsNone(response.data["results"][0]["radio_version"]) |
Borjan Tchakaloff | 869cf92 | 2018-02-19 11:01:16 +0100 | [diff] [blame] | 452 | |
| 453 | def test_radio_version_field(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 454 | """Test retrieval of radio version field.""" |
| 455 | response = self.user.post(reverse(self.LIST_CREATE_URL), self.data) |
| 456 | self.assertEqual(response.status_code, status.HTTP_201_CREATED) |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 457 | url = reverse(self.LIST_CREATE_BY_UUID_URL, args=[self.uuid]) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 458 | response = self.admin.get(url) |
| 459 | self.assertEqual(response.status_code, status.HTTP_200_OK) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 460 | self.assertEqual(len(response.data["results"]), 1) |
| 461 | self.assertEqual( |
| 462 | response.data["results"][0]["radio_version"], |
| 463 | self.data["radio_version"], |
| 464 | ) |
Borjan Tchakaloff | 869cf92 | 2018-02-19 11:01:16 +0100 | [diff] [blame] | 465 | |
Borjan Tchakaloff | 866f75e | 2018-03-01 12:04:00 +0400 | [diff] [blame] | 466 | def test_send_non_existent_time(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 467 | """Test sending of heartbeat with non existent time. |
| 468 | |
| 469 | Test the resolution of a naive date-time in which the |
| 470 | Europe/Amsterdam daylight saving time transition moved the time |
| 471 | "forward". |
Borjan Tchakaloff | 866f75e | 2018-03-01 12:04:00 +0400 | [diff] [blame] | 472 | """ |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 473 | data = self._create_dummy_data(uuid=self.uuid) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 474 | # In 2017, the Netherlands changed from CET to CEST on March, |
| 475 | # 26 at 02:00 |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 476 | data["date"] = "2017-03-26 02:34:56" |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 477 | response = self.user.post(reverse(self.LIST_CREATE_URL), data) |
| 478 | self.assertEqual(response.status_code, status.HTTP_201_CREATED) |
Borjan Tchakaloff | 866f75e | 2018-03-01 12:04:00 +0400 | [diff] [blame] | 479 | |
| 480 | def test_send_ambiguous_time(self): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 481 | """Test sending of heartbeat with ambiguous time. |
| 482 | |
| 483 | Test the resolution of a naive date-time in which the |
| 484 | Europe/Amsterdam daylight saving time transition moved the time |
| 485 | "backward". |
Borjan Tchakaloff | 866f75e | 2018-03-01 12:04:00 +0400 | [diff] [blame] | 486 | """ |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 487 | data = self._create_dummy_data(uuid=self.uuid) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 488 | # In 2017, the Netherlands changed from CEST to CET on October, |
| 489 | # 29 at 03:00 |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 490 | data["date"] = "2017-10-29 02:34:56" |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 491 | response = self.user.post(reverse(self.LIST_CREATE_URL), data) |
| 492 | self.assertEqual(response.status_code, status.HTTP_201_CREATED) |
Borjan Tchakaloff | 866f75e | 2018-03-01 12:04:00 +0400 | [diff] [blame] | 493 | |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 494 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 495 | # pylint: disable=too-many-ancestors |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 496 | class CrashreportListTestCase(HeartbeatListTestCase): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 497 | """Test cases for crash reports.""" |
Dirk Vogt | c9e10ab | 2016-10-12 13:58:15 +0200 | [diff] [blame] | 498 | |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 499 | LIST_CREATE_URL = "api_v1_crashreports" |
| 500 | RETRIEVE_URL = "api_v1_crashreport" |
| 501 | LIST_CREATE_BY_UUID_URL = "api_v1_crashreports_by_uuid" |
| 502 | RETRIEVE_BY_UUID_URL = "api_v1_crashreport_by_uuid" |
Dirk Vogt | 67eb148 | 2016-10-13 12:42:56 +0200 | [diff] [blame] | 503 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 504 | @staticmethod |
| 505 | def _create_dummy_data(**kwargs): |
| 506 | return Dummy.crashreport_data(**kwargs) |
Dirk Vogt | 67eb148 | 2016-10-13 12:42:56 +0200 | [diff] [blame] | 507 | |
| 508 | |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 509 | class LogfileUploadTest(DeviceRegisterAPITestCase): |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 510 | """Test cases for upload of log files.""" |
Borjan Tchakaloff | ec814a7 | 2018-02-28 11:16:53 +0400 | [diff] [blame] | 511 | |
Franz-Xaver Geiger | 67504d0 | 2018-03-19 15:04:48 +0100 | [diff] [blame] | 512 | LIST_CREATE_URL = "api_v1_crashreports" |
| 513 | PUT_LOGFILE_URL = "api_v1_putlogfile_for_device_id" |
| 514 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 515 | def _upload_crashreport(self, user, uuid): |
| 516 | """ |
| 517 | Upload dummy crashreport data. |
Dirk Vogt | 67eb148 | 2016-10-13 12:42:56 +0200 | [diff] [blame] | 518 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 519 | Args: |
| 520 | user: The user which should be used for uploading the report |
| 521 | uuid: The uuid of the device to which the report should be uploaded |
Dirk Vogt | 3663569 | 2016-10-17 12:19:10 +0200 | [diff] [blame] | 522 | |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 523 | Returns: The local id of the device for which the report was uploaded. |
| 524 | |
| 525 | """ |
| 526 | data = Dummy.crashreport_data(uuid=uuid) |
| 527 | response = user.post(reverse(self.LIST_CREATE_URL), data) |
| 528 | self.assertEqual(status.HTTP_201_CREATED, response.status_code) |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 529 | self.assertTrue("device_local_id" in response.data) |
| 530 | device_local_id = response.data["device_local_id"] |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 531 | |
| 532 | return device_local_id |
| 533 | |
| 534 | def _test_logfile_upload(self, user, uuid): |
| 535 | # Upload crashreport |
| 536 | device_local_id = self._upload_crashreport(user, uuid) |
| 537 | |
| 538 | # Upload a logfile for the crashreport |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 539 | logfile = tempfile.NamedTemporaryFile("w+", suffix=".log", delete=True) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 540 | logfile.write(u"blihblahblub") |
| 541 | response = user.post( |
Mitja Nikolaus | cb50f2c | 2018-08-24 13:54:48 +0200 | [diff] [blame] | 542 | reverse( |
| 543 | self.PUT_LOGFILE_URL, |
| 544 | args=[uuid, device_local_id, os.path.basename(logfile.name)], |
| 545 | ), |
| 546 | {"file": logfile}, |
| 547 | format="multipart", |
| 548 | ) |
Mitja Nikolaus | 3518160 | 2018-08-03 13:55:30 +0200 | [diff] [blame] | 549 | self.assertEqual(status.HTTP_201_CREATED, response.status_code) |
| 550 | |
| 551 | def test_logfile_upload_as_user(self): |
| 552 | """Test upload of logfiles as device owner.""" |
| 553 | uuid, user, _ = self._register_device() |
| 554 | self._test_logfile_upload(user, uuid) |
| 555 | |
| 556 | def test_logfile_upload_as_admin(self): |
| 557 | """Test upload of logfiles as admin user.""" |
| 558 | uuid, _, _ = self._register_device() |
| 559 | self._test_logfile_upload(self.admin, uuid) |