test: update module_llm tests for system prompt handling (None -> "")

[februarysea]

Summary

Updates existing tests to align with the system prompt handling behavior introduced in the previous fix. This ensures the test suite correctly reflects the normalized handling of empty system prompts.

Bug / Issue

This PR is a follow-up to #41 (Fix Ollama tool calling crash when system prompt is None).

In PR #41, the runtime crash was resolved by normalizing the system prompt when it is None. However, some existing tests still assumed the old behavior, causing a mismatch between the implementation and the test expectations.

Implementation

• Updated existing tests in test_module_llm.py to align with the current behavior where an empty or missing system prompt is normalized to an empty string (None -> "") for Ollama models.
• No changes were made to the core implementation logic; this PR strictly updates test expectations to match the corrected behavior.

Testing

• Ran the updated test suite locally with pytest and pre-commit.

Additional Notes

N/A

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[februarysea]

It seems like The Upload coverage to Codecov step in the mesa/mesa-llm workflow failed due to a missing or invalid CODECOV_TOKEN, preventing the coverage report upload.

[colinfrisch]

LGTM, thanks !!