fix(set case priority research bug) (#3830) (patch)

# Description

Fixes a bug in set case

Author: ChristianOertlin

cg/cli/set/case.py

@@ -1,6 +1,7 @@
 """Set case attributes in the status database."""
 
 import logging
+from typing import Callable
 
 import click
 
@@ -57,7 +58,7 @@ def set_case(
         data_analysis,
         data_delivery,
     ]
-    abort_on_empty_options(options=options)
+    abort_on_empty_options(options=options, priority=priority)
 
     status_db: Store = context.status_db
     case: Case = get_case(case_id=case_id, status_db=status_db)
@@ -77,14 +78,15 @@ def set_case(
     if panel_abbreviations:
         update_panels(case=case, panel_abbreviations=panel_abbreviations, status_db=status_db)
 
-    if priority:
+    if isinstance(priority, Priority):
         update_priority(case=case, priority=priority)
 
     status_db.session.commit()
 
 
-def abort_on_empty_options(options: list[str]) -> None:
-    if not any(options):
+def abort_on_empty_options(options: list[str], priority: Priority | None) -> None:
+    """Abort if options are empty and bypass_check for priority returns False"""
+    if not any(options) and not priority == Priority.research:
         LOG.error("Nothing to change")
         raise click.Abort
 

tests/cli/set/test_cli_set_case.py

@@ -98,6 +98,28 @@ def test_set_case_priority(
     assert case_query.first().priority_human == priority
 
 
+def test_set_case_priority_research(
+    cli_runner: CliRunner, base_context: CGConfig, base_store: Store, helpers: StoreHelpers
+):
+    """Test that the added case gets the priority we send in."""
+    # GIVEN a database with a case
+    case_id: str = helpers.add_case(base_store).internal_id
+    priority: str = "research"
+    case_query = base_store._get_query(table=Case)
+
+    assert case_query.first().priority_human != priority
+
+    # WHEN setting a case
+    result = cli_runner.invoke(
+        set_case, [case_id, "--priority", priority], obj=base_context, catch_exceptions=False
+    )
+
+    # THEN it should have been set
+    assert result.exit_code == EXIT_SUCCESS
+    assert case_query.count() == 1
+    assert case_query.first().priority_human == priority
+
+
 def test_set_case_customer(
     cli_runner: CliRunner, base_context: CGConfig, base_store: Store, helpers: StoreHelpers
 ):