infrastructure/extensions-website
4
8
Fork 10
Code Issues 20 Pull Requests 2 Projects Releases Wiki Activity

API: Upload new version of an extension #138

Merged
Dalai Felinto merged 12 commits from tokens-version-api into main 2024-05-27 16:18:05 +02:00
Conversation 8 Commits 12 Files Changed 7 +10 -6
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 81d4bfd3b9 - Show all commits

16
extensions/tests/test_api.py
View File

@ -8,7 +8,7 @@ from common.tests.factories.users import UserFactory
from common.tests.factories.extensions import create_approved_version from common.tests.factories.extensions import create_approved_version
from extensions.models import Version from extensions.models import Version
from tokens.models import UserToken from apitokens.models import UserToken
TEST_FILES_DIR = Path(__file__).resolve().parent / 'files' TEST_FILES_DIR = Path(__file__).resolve().parent / 'files'
@ -17,7 +17,12 @@ TEST_FILES_DIR = Path(__file__).resolve().parent / 'files'
class VersionUploadAPITest(APITestCase): class VersionUploadAPITest(APITestCase):
dfelinto marked this conversation as resolved Outdated
Oleg-Komarov commented 2024-05-23 13:17:53 +02:00
Outdated
Review
Copy Link

should include a test for a non-authenticated request

should include a test for a non-authenticated request
Oleg-Komarov commented 2024-05-27 09:46:12 +02:00
Outdated
Review
Copy Link

returning 403 is ok: maybe not 100% correct from a purist perspective, but it serves the purpose

returning 403 is ok: maybe not 100% correct from a purist perspective, but it serves the purpose
def setUp(self): def setUp(self):
self.user = UserFactory() self.user = UserFactory()
self.token = UserToken.objects.create(user=self.user) self.token_key = UserToken.generate_token_key()
self.token = UserToken.objects.create(
user=self.user,
token_prefix=UserToken.generate_token_prefix(self.token_key),
token_hash=UserToken.generate_hash(self.token_key),
)
self.client = APIClient() self.client = APIClient()
self.version = create_approved_version( self.version = create_approved_version(
@ -44,7 +49,7 @@ class VersionUploadAPITest(APITestCase):
'release_notes': 'These are the release notes', 'release_notes': 'These are the release notes',
}, },
format='multipart', format='multipart',
HTTP_AUTHORIZATION=f'Bearer {self.token.token}', HTTP_AUTHORIZATION=f'Bearer {self.token_key}',
) )
self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
@ -64,7 +69,7 @@ class VersionUploadAPITest(APITestCase):
'release_notes': 'These are the release notes', 'release_notes': 'These are the release notes',
}, },
format='multipart', format='multipart',
HTTP_AUTHORIZATION=f'Bearer {self.token.token}', HTTP_AUTHORIZATION=f'Bearer {self.token_key}',
) )
self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND) self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
@ -81,9 +86,8 @@ class VersionUploadAPITest(APITestCase):
'release_notes': 'These are the release notes', 'release_notes': 'These are the release notes',
}, },
format='multipart', format='multipart',
dfelinto marked this conversation as resolved Outdated
Oleg-Komarov commented 2024-05-23 13:19:29 +02:00
Outdated
Review
Copy Link

this assert must be failing now after the changes to the response

this assert must be failing now after the changes to the response
HTTP_AUTHORIZATION=f'Bearer {self.token.token}', HTTP_AUTHORIZATION=f'Bearer {self.token_key}',
) )
self.assertEqual(response.status_code, status.HTTP_201_CREATED) self.assertEqual(response.status_code, status.HTTP_201_CREATED)
self.assertIn('version_id', response.data)
self.assertEqual(Version.objects.filter(extension=self.extension).count(), 2) self.assertEqual(Version.objects.filter(extension=self.extension).count(), 2)