test: add more test case

[leavesster]

No description provided.

[coderabbitai]

Summary by CodeRabbit

• 测试
  • 新增用例，覆盖在缺少输入定义时的凭据替换行为，确保原有值保持不变。
  • 调整输入定义的构造方式，维持既有断言与预期结果。
  • 扩展断言，校验对是否为凭据句柄的识别逻辑，更精确反映预期。
  • 细节整理与分隔，提升相关场景的测试覆盖率与可读性。
  • 仅测试层改动，对现有功能与接口无影响。

Walkthrough

在测试层面更新与凭证处理相关的用例：调整构造 InputHandleDef 的方式，新增当输入定义为 None 时的 replace_credential 行为测试；并在 handle 测试中新增对 is_credential_handle() 的断言。

Changes

Cohort / File(s)	Summary of Changes
Credential 测试更新 executor/tests/test_credential.py	- 使用字典解包构造 InputHandleDef（替代显式关键字参数）。 - 新增测试：当输入定义为 None 时，replace_credential 不改变凭证值。 - 轻微格式调整；其余现有测试逻辑不变。
Handle 测试断言扩充 oocana/tests/test_handle.py	- 在 TestHandleData.input_handle_test 中新增对 input_def.is_credential_handle() 的断言；若期望值未提供则默认断言为 False。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: support credential input #433 — 引入 is_credential_handle 与 replace_credential 相关实现与测试，直接对应本次测试用例与断言的新增与调整。

Pre-merge checks

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description Check	⚠️ Warning	PR 未提供描述，导致无法通过描述了解本次变更的具体内容；该检查要求描述至少与变更集相关，但此处描述为空，不满足条件。	请添加一个简要的 PR 描述，概述此次变更所新增的测试用例及其预期行为，以便评审者了解更改目的。

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title Check	✅ Passed	PR 标题 “test: add more test case” 符合 <type>: <subject> 的格式要求，其中 type 定义为 test，且未指定 scope。标题准确反映了本次变更中新增测试用例的意图，与实际修改内容保持一致。

---

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

oocana/tests/test_handle.py (1)

100-100: 补齐 TypedDict 字段以匹配新断言

这里新增了对 expected["is_credential_handle"] 的访问，但 InputHandleTestResult TypedDict 目前并未声明该字段，静态类型检查时会直接报错。建议在 InputHandleTestResult 中新增一个 Optional[bool] 类型的 is_credential_handle 字段，使类型声明与断言保持一致。

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7b7b89c and 365cf82.

📒 Files selected for processing (2)

• executor/tests/test_credential.py (2 hunks)
• oocana/tests/test_handle.py (1 hunks)

🧰 Additional context used

🧬 Code graph analysis (2)

executor/tests/test_credential.py (2)

oocana/oocana/handle.py (2)

• InputHandleDef (86-114)
• get (10-12)

executor/python_executor/credential.py (1)

• replace_credential (32-48)

oocana/tests/test_handle.py (1)

oocana/oocana/handle.py (1)

• is_credential_handle (78-79)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: check
• GitHub Check: check

🔇 Additional comments (1)

executor/tests/test_credential.py (1)

56-63: 新增用例覆盖空定义场景 👍

这个用例很好地验证了 input_def 为 None 时的回退逻辑，避免未来改动误触这一路径。