test(oocana): add comprehensive tests for Context output/finish methods

[leavesster]

Summary

Add 17 unit tests for Context class covering:

• finish() with result, error, and empty cases
• Error takes priority over result in finish()
• output() sends BlockOutput message
• output() with undefined handle warns
• outputs() sends batch BlockOutputs message
• send_warning() and error() methods
• Property accessors (session_id, job_id, node_id, tmp_dir, session_dir)
• inputs_def and outputs_def are read-only

Problem

Context class (912 lines) had zero test coverage for core methods.

Solution

Created tests/test_context.py with mocked Mainframe to test message sending behavior.

Test Results

Ran 17 tests in 0.032s
OK

Files Added

• tests/test_context.py (247 lines)

[coderabbitai]

Summary by CodeRabbit

发布说明

• 测试
  • 为 Context 类添加了全面的单元测试套件，包括消息传递、属性访问和错误处理等场景验证

_{✏️ Tip: You can customize this high-level summary in your review settings.}

概览

为 Context 类添加全面的单元测试套件，涵盖 finish()、output()、send_warning()、error() 等方法，以及 session_id、job_id、node_id 等属性的行为验证，使用 mocked mainframe 作为依赖。

变更

组织/文件	摘要
Context 类单元测试 oocana/tests/test_context.py	添加 247 行单元测试，覆盖 Context 类的核心功能，包括完成处理（带/不带结果和错误）、输出消息、警告/错误报告、会话属性访问、临时目录管理及只读属性验证。

预估代码审查工作量

🎯 2 (简单) | ⏱️ ~10 分钟

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	标题遵循了所需的格式 <type>(<scope>): <subject>，清晰描述了拉取请求的主要改动。
Description check	✅ Passed	描述与变更集紧密相关，详细说明了新增测试的内容、测试覆盖范围和测试结果。

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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

[Copilot]

Pull request overview

This PR adds comprehensive unit tests for the Context class, which previously had zero test coverage despite being 912 lines long. The tests focus on core message-sending methods and property accessors.

Changes:

• Added 17 unit tests covering finish(), output(), outputs(), send_warning(), error(), send_message() methods
• Added tests for property accessors (session_id, job_id, node_id, tmp_dir, session_dir)
• Added tests verifying inputs_def and outputs_def are read-only properties

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The tearDown method imports shutil inside the method rather than at the module level. For consistency with other imports (unittest.mock, tempfile, os) which are imported at the top, shutil should also be imported at the module level. This is a minor style issue but keeps imports consistent and makes dependencies clear.