refactor(core): ViewSet Test Suite re-written to pytest for model Manufacturer

ref: #908 #911 #914
This commit is contained in:
2025-07-30 10:00:46 +09:30
parent 9e6322b5f1
commit ee5055950d
2 changed files with 84 additions and 42 deletions

View File

@ -0,0 +1,84 @@
import pytest
from api.tests.unit.test_unit_common_viewset import ModelViewSetInheritedCases
from core.viewsets.manufacturer import (
Manufacturer,
ViewSet,
)
@pytest.mark.model_manufacturer
class ViewsetTestCases(
ModelViewSetInheritedCases,
):
@pytest.fixture( scope = 'function' )
def viewset(self):
return ViewSet
@property
def parameterized_class_attributes(self):
return {
'_model_documentation': {
'type': type(None),
},
'back_url': {
'type': type(None),
},
'documentation': {
'type': type(None),
'value': None
},
'filterset_fields': {
'value': [
'organization'
]
},
'model': {
'value': Manufacturer
},
'model_documentation': {
'type': type(None),
},
'queryset': {
'type': type(None),
},
'serializer_class': {
'type': type(None),
},
'search_fields': {
'value': [
'name'
]
},
'view_description': {
'value': 'Manufacturer(s) / Publishers'
},
'view_name': {
'type': type(None),
},
'view_serializer_name': {
'type': type(None),
}
}
class KnowledgeBaseViewsetInheritedCases(
ViewsetTestCases,
):
pass
@pytest.mark.module_core
class KnowledgeBaseViewsetPyTest(
ViewsetTestCases,
):
pass

View File

@ -1,42 +0,0 @@
from django.test import Client, TestCase
from rest_framework.reverse import reverse
from api.tests.unit.test_unit_common_viewset import ModelViewSetInheritedCases
from core.viewsets.manufacturer import ViewSet
@pytest.mark.skip(reason = 'see #895, tests being refactored')
class ManufacturerViewsetList(
ModelViewSetInheritedCases,
TestCase,
):
viewset = ViewSet
route_name = 'v2:_api_manufacturer'
@classmethod
def setUpTestData(self):
"""Setup Test
1. make list request
"""
super().setUpTestData()
client = Client()
url = reverse(
self.route_name + '-list',
kwargs = self.kwargs
)
client.force_login(self.view_user)
self.http_options_response_list = client.options(url)