Improve test coverage (#117)

* improve test coverage of check_methods_priority
* replace elif with else to improve branch coverage

tests/unit/test_core/test_activation/test_prioritization.py

@@ -18,6 +18,9 @@ def test_check_methods_priority():
 
     # These should be fine
     check_methods_priority(methods_priority=None, methods=methods)
+    # methods_priority is empty list. Does not crash.
+    check_methods_priority(methods_priority=[], methods=methods)
+    # Does not make sense but should not crash.
     check_methods_priority(methods_priority=["*"], methods=methods)
     # Simple list of methods
     check_methods_priority(methods_priority=["A", "B", "F"], methods=methods)
@@ -45,6 +48,15 @@ def test_check_methods_priority():
     ):
         check_methods_priority(methods_priority=["A", "*", "B", "*"], methods=methods)
 
+    # duplicate method names
+    with pytest.raises(
+        ValueError,
+        match=re.escape('Duplicate method name "A" in methods_priority'),
+    ):
+        check_methods_priority(
+            methods_priority=["A", "*", "B", {"A", "C"}], methods=methods
+        )
+
     # Asterisk inside a set
     with pytest.raises(
         ValueError,

wakepy/core/activation.py

@@ -415,7 +415,7 @@ def get_prioritized_methods_groups(
         if item == asterisk:
             # Save the location where to add the rest of the methods ('*')
             asterisk_index = len(out)
-        elif isinstance(item, set):
+        else:  # Item is a set but not `asterisk`
             out.append({method_dct[name] for name in item})
 
     out_flattened = {m for group in out for m in group}