refactor(settings): Update URL route name for AppSettings model
ref: #833 #834
This commit is contained in:
@ -17,7 +17,7 @@ class AppSettingsBaseSerializer(serializers.ModelSerializer):
|
||||
return str( item )
|
||||
|
||||
url = serializers.HyperlinkedIdentityField(
|
||||
view_name="v2:_api_v2_app_settings-detail", format="html"
|
||||
view_name="v2:_api_appsettings-detail", format="html"
|
||||
)
|
||||
|
||||
class Meta:
|
||||
@ -47,7 +47,7 @@ class AppSettingsModelSerializer(AppSettingsBaseSerializer):
|
||||
def get_url(self, item) -> dict:
|
||||
|
||||
return {
|
||||
'_self': reverse("v2:_api_v2_app_settings-detail", request=self._context['view'].request, kwargs={'pk': item.pk}),
|
||||
'_self': reverse("v2:_api_appsettings-detail", request=self._context['view'].request, kwargs={'pk': item.pk}),
|
||||
}
|
||||
|
||||
|
||||
|
@ -36,7 +36,7 @@ class ViewSetBase:
|
||||
|
||||
app_namespace = 'v2'
|
||||
|
||||
url_name = '_api_v2_app_settings'
|
||||
url_name = '_api_appsettings'
|
||||
|
||||
change_data = {'device_model_is_global': True}
|
||||
|
||||
|
19
app/settings/tests/unit/app_settings/conftest.py
Normal file
19
app/settings/tests/unit/app_settings/conftest.py
Normal file
@ -0,0 +1,19 @@
|
||||
import pytest
|
||||
|
||||
|
||||
|
||||
@pytest.fixture( scope = 'class')
|
||||
def model(model_appsettings):
|
||||
|
||||
yield model_appsettings
|
||||
|
||||
|
||||
@pytest.fixture( scope = 'class', autouse = True)
|
||||
def model_kwargs(request, kwargs_appsettings):
|
||||
|
||||
request.cls.kwargs_create_item = kwargs_appsettings.copy()
|
||||
|
||||
yield kwargs_appsettings.copy()
|
||||
|
||||
if hasattr(request.cls, 'kwargs_create_item'):
|
||||
del request.cls.kwargs_create_item
|
@ -81,7 +81,7 @@ class AppSettingsAPI(
|
||||
self.url_view_kwargs = {'pk': self.item.id}
|
||||
|
||||
client = Client()
|
||||
url = reverse('v2:_api_v2_app_settings-detail', kwargs=self.url_view_kwargs)
|
||||
url = reverse('v2:_api_appsettings-detail', kwargs=self.url_view_kwargs)
|
||||
|
||||
|
||||
client.force_login(self.view_user)
|
||||
|
@ -18,7 +18,7 @@ class AppSettingsViewsetList(
|
||||
|
||||
viewset = ViewSet
|
||||
|
||||
route_name = 'v2:_api_v2_app_settings'
|
||||
route_name = 'v2:_api_appsettings'
|
||||
|
||||
|
||||
@classmethod
|
||||
|
@ -33,7 +33,7 @@ from project_management.viewsets import (
|
||||
)
|
||||
|
||||
from settings.viewsets import (
|
||||
app_settings as app_settings_v2,
|
||||
app_settings,
|
||||
external_link as external_link_v2,
|
||||
index as settings_index_v2,
|
||||
user_settings as user_settings_v2
|
||||
@ -52,8 +52,8 @@ router.register(
|
||||
basename = '_api_v2_settings_home'
|
||||
)
|
||||
router.register(
|
||||
prefix = 'app_settings', viewset = app_settings_v2.ViewSet,
|
||||
basename = '_api_v2_app_settings'
|
||||
prefix = 'app_settings', viewset = app_settings.ViewSet,
|
||||
basename = '_api_appsettings'
|
||||
)
|
||||
router.register(
|
||||
prefix = 'celery_log', viewset = celery_log_v2.ViewSet,
|
||||
|
@ -107,7 +107,7 @@ class Index(IndexViewset):
|
||||
|
||||
return Response(
|
||||
{
|
||||
"app_settings": reverse('v2:_api_v2_app_settings-detail', request=request, kwargs={'pk': 1}),
|
||||
"app_settings": reverse('v2:_api_appsettings-detail', request=request, kwargs={'pk': 1}),
|
||||
"celery_log": reverse('v2:_api_v2_celery_log-list', request=request),
|
||||
"cluster_type": reverse('v2:_api_clustertype-list', request=request),
|
||||
"device_model": reverse('v2:_api_devicemodel-list', request=request),
|
||||
|
Reference in New Issue
Block a user