Cutlery/paperless-ngx
1
0
Fork 0
mirror of https://github.com/paperless-ngx/paperless-ngx.git synced 2025-11-04 03:27:12 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1025 from paperless-ngx/fix-pytest-api-auth

Browse Source 
Fix: use `force_authentication` instead of `force_login` for python api tests
This commit is contained in:
shamoon 2022-05-24 09:59:14 -07:00 committed by GitHub
commit 6b9c07b809
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/documents/tests/test_api.py
View File

@ -41,7 +41,7 @@ class TestDocumentApi(DirectoriesMixin, APITestCase):
super().setUp() super().setUp()
self.user = User.objects.create_superuser(username="temp_admin") self.user = User.objects.create_superuser(username="temp_admin")
self.client.force_login(user=self.user) self.client.force_authenticate(user=self.user)
def testDocuments(self): def testDocuments(self):
@ -1176,7 +1176,7 @@ class TestDocumentApi(DirectoriesMixin, APITestCase):
self.assertEqual(self.client.get(f"/api/saved_views/{v1.id}/").status_code, 404) self.assertEqual(self.client.get(f"/api/saved_views/{v1.id}/").status_code, 404)
self.client.force_login(user=u1) self.client.force_authenticate(user=u1)
response = self.client.get("/api/saved_views/") response = self.client.get("/api/saved_views/")
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
@ -1184,7 +1184,7 @@ class TestDocumentApi(DirectoriesMixin, APITestCase):
self.assertEqual(self.client.get(f"/api/saved_views/{v1.id}/").status_code, 200) self.assertEqual(self.client.get(f"/api/saved_views/{v1.id}/").status_code, 200)
self.client.force_login(user=u2) self.client.force_authenticate(user=u2)
response = self.client.get("/api/saved_views/") response = self.client.get("/api/saved_views/")
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
@ -1358,7 +1358,7 @@ class TestDocumentApiV2(DirectoriesMixin, APITestCase):
self.user = User.objects.create_superuser(username="temp_admin") self.user = User.objects.create_superuser(username="temp_admin")
self.client.force_login(user=self.user) self.client.force_authenticate(user=self.user)
self.client.defaults["HTTP_ACCEPT"] = "application/json; version=2" self.client.defaults["HTTP_ACCEPT"] = "application/json; version=2"
def test_tag_validate_color(self): def test_tag_validate_color(self):
@ -1434,7 +1434,7 @@ class TestDocumentApiV2(DirectoriesMixin, APITestCase):
def test_ui_settings(self): def test_ui_settings(self):
test_user = User.objects.create_superuser(username="test") test_user = User.objects.create_superuser(username="test")
self.client.force_login(user=test_user) self.client.force_authenticate(user=test_user)
response = self.client.get("/api/ui_settings/", format="json") response = self.client.get("/api/ui_settings/", format="json")
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
@ -1473,7 +1473,7 @@ class TestBulkEdit(DirectoriesMixin, APITestCase):
super().setUp() super().setUp()
user = User.objects.create_superuser(username="temp_admin") user = User.objects.create_superuser(username="temp_admin")
self.client.force_login(user=user) self.client.force_authenticate(user=user)
patcher = mock.patch("documents.bulk_edit.async_task") patcher = mock.patch("documents.bulk_edit.async_task")
self.async_task = patcher.start() self.async_task = patcher.start()
@ -2049,7 +2049,7 @@ class TestBulkDownload(DirectoriesMixin, APITestCase):
super().setUp() super().setUp()
user = User.objects.create_superuser(username="temp_admin") user = User.objects.create_superuser(username="temp_admin")
self.client.force_login(user=user) self.client.force_authenticate(user=user)
self.doc1 = Document.objects.create(title="unrelated", checksum="A") self.doc1 = Document.objects.create(title="unrelated", checksum="A")
self.doc2 = Document.objects.create( self.doc2 = Document.objects.create(
@ -2250,7 +2250,7 @@ class TestApiAuth(APITestCase):
def test_api_version_with_auth(self): def test_api_version_with_auth(self):
user = User.objects.create_superuser(username="test") user = User.objects.create_superuser(username="test")
self.client.force_login(user) self.client.force_authenticate(user)
response = self.client.get("/api/") response = self.client.get("/api/")
self.assertIn("X-Api-Version", response) self.assertIn("X-Api-Version", response)
self.assertIn("X-Version", response) self.assertIn("X-Version", response)