Prepared the test file for the incoming testdata and cleaned some code in the determine_constant_parameters() function.

Author: GideonKoenig

package-parser/package_parser/commands/generate_annotations/_generate_annotations.py

@@ -121,10 +121,6 @@ def __determine_constant_parameters(usages: UsageStore) -> dict[str, str]:
         if n_total_usage == 0 and len(value_usages[parameter_qname].values()) == 0:
             continue
 
-        usage_key, usage_count = __n_not_set_to_most_common_value(
-            parameter_qname, usages.function_usages, usages.value_usages
-        )
-
         if len(usages.value_usages[parameter_qname].keys()) == 1:
             result[parameter_qname] = usages.most_common_value(parameter_qname)
 

package-parser/tests/commands/generate-annotations/test_generate_annotations.py

@@ -4,8 +4,35 @@
     UsageStore
 )
 
+# @Unused annotations should be created for the following declarations:
+#
+# test.Unused_Class
+# test.unused_global_function
+# test.Commonly_Used_Class.unused_method
+#
+# @Constant annotations should be created for the following parameters:
+#
+# test.commonly_used_global_function.useless_required_parameter (with value 'blup')
+# test.commonly_used_global_function.unused_optional_parameter (with value 'bla', i.e. the default value)
+# test.commonly_used_global_function.useless_optional_parameter (with value 'bla')
+
+
 def test_determination_of_constant_parameters():
 
-    usages = UsageStore()
+    expected = {
+        ("test.commonly_used_global_function.useless_required_parameter", "blup"),
+        ("test.commonly_used_global_function.unused_optional_parameter", "bla"),
+        ("test.commonly_used_global_function.useless_optional_parameter", "bla")
+    }
+
+    base_path_to_data = "~/."
+    usages_file = ""
+
+    with usages_file:
+        usages_json = json.load(usages_file)
+        usages = UsageStore.from_json(usages_json)
+
     constant_parameters = __determine_constant_parameters(usages)
 
+    assert constant_parameters == expected
+