From 446ea7c9103beb018aaab5450aea832950d026ba Mon Sep 17 00:00:00 2001 From: Jon Date: Tue, 10 Jun 2025 12:41:45 +0930 Subject: [PATCH] chore(core): correct test for get_url_kwargs for meta abstract model ref: #813 #793 --- .../test_unit_centurion_audit_meta_model.py | 20 ++++++++++++++++++ ...st_unit_centurion_model_note_meta_model.py | 21 ++++++++++++++++++- .../check_ins/test_unit_check_ins_model.py | 15 +++++++++++++ 3 files changed, 55 insertions(+), 1 deletion(-) diff --git a/app/core/tests/unit/centurion_audit_meta/test_unit_centurion_audit_meta_model.py b/app/core/tests/unit/centurion_audit_meta/test_unit_centurion_audit_meta_model.py index b6fe6b98..f0668edd 100644 --- a/app/core/tests/unit/centurion_audit_meta/test_unit_centurion_audit_meta_model.py +++ b/app/core/tests/unit/centurion_audit_meta/test_unit_centurion_audit_meta_model.py @@ -232,3 +232,23 @@ class MetaAbstractModelPyTest( assert e.value.code == 'no_model_supplied' + + + + def test_method_get_url_kwargs(self, mocker, model_instance, settings): + """Test Class Method + + Ensure method `get_url_kwargs` returns the correct value. + """ + + model_instance.id = 1 + model_instance.model = model_instance + + url = model_instance.get_url_kwargs() + + assert model_instance.get_url_kwargs() == { + 'app_label': model_instance._meta.app_label, + 'model_name': model_instance._meta.model_name, + 'model_id': model_instance.model.id, + 'pk': model_instance.id, + } diff --git a/app/core/tests/unit/centurion_model_note_meta/test_unit_centurion_model_note_meta_model.py b/app/core/tests/unit/centurion_model_note_meta/test_unit_centurion_model_note_meta_model.py index c3bb9d64..cc362dd9 100644 --- a/app/core/tests/unit/centurion_model_note_meta/test_unit_centurion_model_note_meta_model.py +++ b/app/core/tests/unit/centurion_model_note_meta/test_unit_centurion_model_note_meta_model.py @@ -195,7 +195,26 @@ class MetaAbstractModelInheritedCases( class MetaAbstractModelPyTest( MetaAbstractModelTestCases, ): - # pass + + + def test_method_get_url_kwargs(self, mocker, model_instance, settings): + """Test Class Method + + Ensure method `get_url_kwargs` returns the correct value. + """ + + model_instance.id = 1 + model_instance.model = model_instance + + url = model_instance.get_url_kwargs() + + assert model_instance.get_url_kwargs() == { + 'app_label': model_instance._meta.app_label, + 'model_name': model_instance._meta.model_name, + 'model_id': model_instance.model.id, + 'pk': model_instance.id, + } + @pytest.mark.xfail( reason = 'This model is an abstract model') def test_model_tag_defined(self, model): diff --git a/app/devops/tests/unit/check_ins/test_unit_check_ins_model.py b/app/devops/tests/unit/check_ins/test_unit_check_ins_model.py index 03653511..7ea50722 100644 --- a/app/devops/tests/unit/check_ins/test_unit_check_ins_model.py +++ b/app/devops/tests/unit/check_ins/test_unit_check_ins_model.py @@ -97,3 +97,18 @@ class CheckInModelPyTest( """ assert model.model_tag is not None + + + def test_method_get_url_kwargs(self, mocker, model_instance, settings): + """Test Class Method + + Ensure method `get_url_kwargs` returns the correct value. + """ + + + url = model_instance.get_url_kwargs() + + assert model_instance.get_url_kwargs() == { + 'organization_id': model_instance.organization.id, + 'software_id': model_instance.software.id + }