chore: exclude jq mock review finding
This commit is contained in:
@@ -26,6 +26,11 @@
|
||||
"title": "log function definition unclear",
|
||||
"reason": "腳本中廣泛使用了 `log` 函式,但此 diff 並未包含 `log` 的定義,表示 `log` 可能來自基礎映像或另一個未包含的腳本。建議:在 entrypoint.sh 明確定義 `log` 或在文件/腳本中註明其來源,提升可維護性與自包含性。"
|
||||
},
|
||||
{
|
||||
"location": "tests/entrypoint.sh",
|
||||
"title": "jq mock tightly coupled to entrypoint.sh expressions",
|
||||
"reason": "測試中的 `jq` 模擬實作與 `entrypoint.sh` 中使用的特定 `jq` 表達式緊密耦合。若 `entrypoint.sh` 中的 jq 邏輯變更,Mock 也必須同步更新。這可能增加測試維護成本。建議:使用更通用的 jq 模擬方式,或在測試中直接使用真正的 `jq` 工具(若測試環境允許),以降低 Mock 與實際行為不同步的風險。"
|
||||
},
|
||||
{
|
||||
"location": "entrypoint.sh:46",
|
||||
"title": "resolve_token empty-token test suggestion",
|
||||
|
||||
Reference in New Issue
Block a user