make chatbot configurations model selection not nullable (#1574)

* make chatbot configurations model selection not nullable

* fix race condition in image uploading tests

Author: HeljaeRaeisaenen

services/headless-lms/chatbot/src/azure_chatbot.rs

@@ -237,7 +237,7 @@ impl LLMRequest {
         let serialized_messages = serde_json::to_string(&api_chat_messages)?;
         let request_estimated_tokens = estimate_tokens(&serialized_messages);
 
-        let params = if let Some(true) = configuration.thinking_model {
+        let params = if configuration.thinking_model {
             LLMRequestParams::Thinking(ThinkingParams {
                 max_completion_tokens: Some(configuration.max_completion_tokens),
                 reasoning_effort: Some(configuration.reasoning_effort),

services/headless-lms/migrations/20251027064052_chatbot_configurations_model_not_null.down.sql

@@ -0,0 +1,6 @@
+ALTER TABLE chatbot_configurations
+ALTER COLUMN model_id DROP NOT NULL,
+  ALTER COLUMN thinking_model DROP NOT NULL;
+
+ALTER TABLE chatbot_configurations
+  RENAME COLUMN model_id TO model;

services/headless-lms/migrations/20251027064052_chatbot_configurations_model_not_null.up.sql

@@ -0,0 +1,8 @@
+ALTER TABLE chatbot_configurations
+ALTER COLUMN model
+SET NOT NULL,
+  ALTER COLUMN thinking_model
+SET NOT NULL;
+
+ALTER TABLE chatbot_configurations
+  RENAME COLUMN model TO model_id;