From 2bdb12ed7a694d505996cd7829cbedb7757e208d Mon Sep 17 00:00:00 2001 From: Jon Date: Mon, 4 Aug 2025 14:21:04 +0930 Subject: [PATCH] refactor(project_management): Remove old test suites no longer required model Project ref: #938 closes #934 --- .../tests/unit/project/test_project_api_v2.py | 767 ------------------ .../unit/project/test_unit_project_model.py | 2 +- 2 files changed, 1 insertion(+), 768 deletions(-) delete mode 100644 app/project_management/tests/unit/project/test_project_api_v2.py diff --git a/app/project_management/tests/unit/project/test_project_api_v2.py b/app/project_management/tests/unit/project/test_project_api_v2.py deleted file mode 100644 index e0bb9105..00000000 --- a/app/project_management/tests/unit/project/test_project_api_v2.py +++ /dev/null @@ -1,767 +0,0 @@ -import django -import pytest - -from django.contrib.auth.models import Permission -from django.contrib.contenttypes.models import ContentType -from django.shortcuts import reverse -from django.test import Client, TestCase - -from rest_framework.relations import Hyperlink - -from access.models.tenant import Tenant as Organization -from access.models.team import Team -from access.models.team_user import TeamUsers - -from api.tests.abstract.api_fields import APITenancyObject - -from project_management.models.projects import Project, ProjectState, ProjectType - -from settings.models.user_settings import UserSettings - -User = django.contrib.auth.get_user_model() - - - -@pytest.mark.model_project -@pytest.mark.module_project_management -class ProjectAPI( - TestCase, - APITenancyObject -): - - model = Project - - @classmethod - def setUpTestData(self): - """Setup Test - - 1. Create an organization for user and item - 2. Create an item - - """ - - self.organization = Organization.objects.create(name='test_org') - - - project_type = ProjectType.objects.create( - organization = self.organization, - name = 'proj type' - ) - - project_state = ProjectState.objects.create( - organization = self.organization, - name = 'a state' - ) - - - view_permissions = Permission.objects.get( - codename = 'view_' + self.model._meta.model_name, - content_type = ContentType.objects.get( - app_label = self.model._meta.app_label, - model = self.model._meta.model_name, - ) - ) - - view_team = Team.objects.create( - team_name = 'view_team', - organization = self.organization, - ) - - view_team.permissions.set([view_permissions]) - - self.view_user = User.objects.create_user(username="test_user_view", password="password") - - user_settings = UserSettings.objects.get(user = self.view_user) - - user_settings.default_organization = self.organization - - user_settings.save() - - - teamuser = TeamUsers.objects.create( - team = view_team, - user = self.view_user - ) - - - self.item = self.model.objects.create( - organization = self.organization, - name = 'one', - state = project_state, - project_type = project_type, - description = 'a note', - manager_user = self.view_user, - manager_team = view_team, - planned_start_date = '2024-01-01 00:01:00', - planned_finish_date = '2024-01-01 00:01:01', - real_start_date = '2024-01-02 00:01:00', - real_finish_date = '2024-01-02 00:01:01', - code = 'acode', - external_ref = 1, - external_system = Project.Ticket_ExternalSystem.CUSTOM_1 - ) - - - self.item.team_members.set([ self.view_user ]) - - - self.url_view_kwargs = {'pk': self.item.id} - - client = Client() - url = reverse('v2:_api_project-detail', kwargs=self.url_view_kwargs) - - - client.force_login(self.view_user) - response = client.get(url) - - self.api_data = response.data - - - - - def test_api_field_exists_estimation_project(self): - """ Test for existance of API Field - - This test is a custom test of a test case with the same name. - this model does not have a model_notes_field - - estimation_project field must exist - """ - - assert 'estimation_project' in self.api_data - - - - def test_api_field_type_estimation_project(self): - """ Test for type for API Field - - estimation_project field must be int - """ - - assert type(self.api_data['estimation_project']) is int - - - - def test_api_field_exists_duration_project(self): - """ Test for existance of API Field - - This test is a custom test of a test case with the same name. - this model does not have a model_notes_field - - duration_project field must exist - """ - - assert 'duration_project' in self.api_data - - - - def test_api_field_type_duration_project(self): - """ Test for type for API Field - - duration_project field must be int - """ - - assert type(self.api_data['duration_project']) is int - - - - def test_api_field_exists_completed(self): - """ Test for existance of API Field - - This test is a custom test of a test case with the same name. - this model does not have a model_notes_field - - model_notes field must exist - """ - - assert 'completed' in self.api_data - - - - def test_api_field_type_completed(self): - """ Test for type for API Field - - description field must be str - """ - - assert type(self.api_data['completed']) is str - - - - def test_api_field_exists_model_notes(self): - """ Test for existance of API Field - - This test is a custom test of a test case with the same name. - this model does not have a model_notes_field - - model_notes field must exist - """ - - assert 'model_notes' not in self.api_data - - - def test_api_field_type_model_notes(self): - """ Test for type for API Field - - This test is a custom test of a test case with the same name. - this model does not have a model_notes_field - - model_notes field must be str - """ - - assert True - - - - def test_api_field_exists_description(self): - """ Test for existance of API Field - - model_notes field must exist - """ - - assert 'description' in self.api_data - - - def test_api_field_type_description(self): - """ Test for type for API Field - - description field must be str - """ - - assert type(self.api_data['description']) is str - - - - def test_api_field_exists_priority(self): - """ Test for existance of API Field - - priority field must exist - """ - - assert 'priority' in self.api_data - - - def test_api_field_type_priority(self): - """ Test for type for API Field - - priority field must be int - """ - - assert type(self.api_data['priority']) is int - - - - def test_api_field_exists_code(self): - """ Test for existance of API Field - - code field must exist - """ - - assert 'code' in self.api_data - - - def test_api_field_type_code(self): - """ Test for type for API Field - - code field must be str - """ - - assert type(self.api_data['code']) is str - - - - def test_api_field_exists_planned_start_date(self): - """ Test for existance of API Field - - planned_start_date field must exist - """ - - assert 'planned_start_date' in self.api_data - - - def test_api_field_type_planned_start_date(self): - """ Test for type for API Field - - planned_start_date field must be str - """ - - assert type(self.api_data['planned_start_date']) is str - - - - def test_api_field_exists_planned_finish_date(self): - """ Test for existance of API Field - - planned_finish_date field must exist - """ - - assert 'planned_finish_date' in self.api_data - - - def test_api_field_type_planned_finish_date(self): - """ Test for type for API Field - - planned_finish_date field must be str - """ - - assert type(self.api_data['planned_finish_date']) is str - - - - def test_api_field_exists_real_start_date(self): - """ Test for existance of API Field - - real_start_date field must exist - """ - - assert 'real_start_date' in self.api_data - - - def test_api_field_type_real_start_date(self): - """ Test for type for API Field - - real_start_date field must be str - """ - - assert type(self.api_data['real_start_date']) is str - - - - def test_api_field_exists_real_finish_date(self): - """ Test for existance of API Field - - real_finish_date field must exist - """ - - assert 'real_finish_date' in self.api_data - - - def test_api_field_type_real_finish_date(self): - """ Test for type for API Field - - real_finish_date field must be str - """ - - assert type(self.api_data['real_finish_date']) is str - - - - def test_api_field_exists_is_deleted(self): - """ Test for existance of API Field - - is_deleted field must exist - """ - - assert 'is_deleted' in self.api_data - - - def test_api_field_type_is_deleted(self): - """ Test for type for API Field - - is_deleted field must be bool - """ - - assert type(self.api_data['is_deleted']) is bool - - - - def test_api_field_exists_external_ref(self): - """ Test for existance of API Field - - external_ref field must exist - """ - - assert 'external_ref' in self.api_data - - - def test_api_field_type_external_ref(self): - """ Test for type for API Field - - external_ref field must be int - """ - - assert type(self.api_data['external_ref']) is int - - - - def test_api_field_exists_external_system(self): - """ Test for existance of API Field - - external_system field must exist - """ - - assert 'external_system' in self.api_data - - - def test_api_field_type_external_system(self): - """ Test for type for API Field - - external_system field must be int - """ - - assert type(self.api_data['external_system']) is int - - - - def test_api_field_exists_project_type(self): - """ Test for existance of API Field - - project_type field must exist - """ - - assert 'project_type' in self.api_data - - - def test_api_field_type_project_type(self): - """ Test for type for API Field - - project_type field must be dict - """ - - assert type(self.api_data['project_type']) is dict - - - def test_api_field_exists_project_type_id(self): - """ Test for existance of API Field - - project_type.id field must exist - """ - - assert 'id' in self.api_data['project_type'] - - - def test_api_field_type_project_type_id(self): - """ Test for type for API Field - - project_type.id field must be int - """ - - assert type(self.api_data['project_type']['id']) is int - - - def test_api_field_exists_project_type_display_name(self): - """ Test for existance of API Field - - project_type.display_name field must exist - """ - - assert 'display_name' in self.api_data['project_type'] - - - def test_api_field_type_project_type_display_name(self): - """ Test for type for API Field - - project_type.display_name field must be str - """ - - assert type(self.api_data['project_type']['display_name']) is str - - - def test_api_field_exists_project_type_url(self): - """ Test for existance of API Field - - project_type.url field must exist - """ - - assert 'url' in self.api_data['project_type'] - - - def test_api_field_type_project_type_url(self): - """ Test for type for API Field - - project_type.url field must be Hyperlink - """ - - assert type(self.api_data['project_type']['url']) is Hyperlink - - - - def test_api_field_exists_state(self): - """ Test for existance of API Field - - state field must exist - """ - - assert 'state' in self.api_data - - - def test_api_field_type_state(self): - """ Test for type for API Field - - state field must be dict - """ - - assert type(self.api_data['state']) is dict - - - def test_api_field_exists_state_id(self): - """ Test for existance of API Field - - state.id field must exist - """ - - assert 'id' in self.api_data['state'] - - - def test_api_field_type_state_id(self): - """ Test for type for API Field - - state.id field must be int - """ - - assert type(self.api_data['state']['id']) is int - - - def test_api_field_exists_state_display_name(self): - """ Test for existance of API Field - - state.display_name field must exist - """ - - assert 'display_name' in self.api_data['state'] - - - def test_api_field_type_state_display_name(self): - """ Test for type for API Field - - state.display_name field must be str - """ - - assert type(self.api_data['state']['display_name']) is str - - - def test_api_field_exists_state_url(self): - """ Test for existance of API Field - - state.url field must exist - """ - - assert 'url' in self.api_data['state'] - - - def test_api_field_type_state_url(self): - """ Test for type for API Field - - state.url field must be Hyperlink - """ - - assert type(self.api_data['state']['url']) is Hyperlink - - - - def test_api_field_exists_manager_user(self): - """ Test for existance of API Field - - manager_user field must exist - """ - - assert 'manager_user' in self.api_data - - - def test_api_field_type_manager_user(self): - """ Test for type for API Field - - manager_user field must be dict - """ - - assert type(self.api_data['manager_user']) is dict - - - def test_api_field_exists_manager_user_id(self): - """ Test for existance of API Field - - manager_user.id field must exist - """ - - assert 'id' in self.api_data['manager_user'] - - - def test_api_field_type_manager_user_id(self): - """ Test for type for API Field - - manager_user.id field must be int - """ - - assert type(self.api_data['manager_user']['id']) is int - - - def test_api_field_exists_manager_user_display_name(self): - """ Test for existance of API Field - - manager_user.display_name field must exist - """ - - assert 'display_name' in self.api_data['manager_user'] - - - def test_api_field_type_manager_user_display_name(self): - """ Test for type for API Field - - manager_user.display_name field must be str - """ - - assert type(self.api_data['manager_user']['display_name']) is str - - - def test_api_field_exists_manager_user_url(self): - """ Test for existance of API Field - - manager_user.url field must exist - """ - - assert 'url' in self.api_data['manager_user'] - - - def test_api_field_type_manager_user_url(self): - """ Test for type for API Field - - manager_user.url field must be Hyperlink - """ - - assert type(self.api_data['manager_user']['url']) is Hyperlink - - - - def test_api_field_exists_manager_team(self): - """ Test for existance of API Field - - manager_team field must exist - """ - - assert 'manager_team' in self.api_data - - - def test_api_field_type_manager_team(self): - """ Test for type for API Field - - manager_team field must be dict - """ - - assert type(self.api_data['manager_team']) is dict - - - def test_api_field_exists_manager_team_id(self): - """ Test for existance of API Field - - manager_team.id field must exist - """ - - assert 'id' in self.api_data['manager_team'] - - - def test_api_field_type_manager_team_id(self): - """ Test for type for API Field - - manager_team.id field must be int - """ - - assert type(self.api_data['manager_team']['id']) is int - - - def test_api_field_exists_manager_team_display_name(self): - """ Test for existance of API Field - - manager_team.display_name field must exist - """ - - assert 'display_name' in self.api_data['manager_team'] - - - def test_api_field_type_manager_team_display_name(self): - """ Test for type for API Field - - manager_team.display_name field must be str - """ - - assert type(self.api_data['manager_team']['display_name']) is str - - - def test_api_field_exists_manager_team_url(self): - """ Test for existance of API Field - - manager_team.url field must exist - """ - - assert 'url' in self.api_data['manager_team'] - - - def test_api_field_type_manager_team_url(self): - """ Test for type for API Field - - manager_team.url field must be str - """ - - assert type(self.api_data['manager_team']['url']) is str - - - - def test_api_field_exists_team_members(self): - """ Test for existance of API Field - - team_members field must exist - """ - - assert 'team_members' in self.api_data - - - def test_api_field_type_team_members(self): - """ Test for type for API Field - - team_members field must be dict - """ - - assert type(self.api_data['team_members']) is list - - - def test_api_field_exists_team_members_id(self): - """ Test for existance of API Field - - team_members.id field must exist - """ - - assert 'id' in self.api_data['team_members'][0] - - - def test_api_field_type_team_members_id(self): - """ Test for type for API Field - - team_members.id field must be int - """ - - assert type(self.api_data['team_members'][0]['id']) is int - - - def test_api_field_exists_team_members_display_name(self): - """ Test for existance of API Field - - team_members.display_name field must exist - """ - - assert 'display_name' in self.api_data['team_members'][0] - - - def test_api_field_type_team_members_display_name(self): - """ Test for type for API Field - - team_members.display_name field must be str - """ - - assert type(self.api_data['team_members'][0]['display_name']) is str - - - def test_api_field_exists_team_members_url(self): - """ Test for existance of API Field - - team_members.url field must exist - """ - - assert 'url' in self.api_data['team_members'][0] - - - def test_api_field_type_team_members_url(self): - """ Test for type for API Field - - team_members.url field must be Hyperlink - """ - - assert type(self.api_data['team_members'][0]['url']) is Hyperlink diff --git a/app/project_management/tests/unit/project/test_unit_project_model.py b/app/project_management/tests/unit/project/test_unit_project_model.py index 55054c82..f6dcd8fa 100644 --- a/app/project_management/tests/unit/project/test_unit_project_model.py +++ b/app/project_management/tests/unit/project/test_unit_project_model.py @@ -142,7 +142,7 @@ class ClusterModelTestCases( 'team_members': { 'blank': True, 'default': models.fields.NOT_PROVIDED, - 'field_type': models.ForeignKey, + 'field_type': models.ManyToManyField, 'null': False, 'unique': False, },