Test Orders API tools parameters
まだ誰も着手していません。
評価
調査の方向性
Start in planet/order_request.py at _tool() and harmonize_tool(), then inspect the planet.specs validate_tool implementation and the related issue #793. Trace how Orders API tool parameters are checked against the OpenAPI spec. Done means invalid parameter values fail validation without relying on manually maintained parameter lists.
索引モデルが issue の本文から書いたものです。
説明
Expected behavior
When testing tools for the Orders API, I'd expect the tool name and the parameters to be validated.
Actual behavior (describe the problem)
When testing tools for the Orders API, only the tool name appears to be validated.
This seems to happen in the function _tool(). It looks like our tools get passed to _tool(), which validates the tool name, but not the tool’s parameters.
For instance, for harmonize_tool() could get passed any sort of target sensor name and the test won’t fail, because it’s only looking to see if the tool name, ‘harmonize’, is a valid tool, not if the parameter, sensor_name, is valid.
Related Issues
I created a ticket a month ago (#793), which sort of fixes this issue for harmonize_tool(), but doesn’t really do so.
Workaround
None.
Minimum, Complete, Viable Code Sample
Current solution in planet.specs:
def validate_tool(tool):
return _validate_field(tool, SUPPORTED_TOOLS, 'tool')
Proposed solution in planet.specs:
def validate_tool_name(tool):
return _validate_field(tool, SUPPORTED_TOOLS, 'tool')
def validate_tool_parameter(parameter):
return _validate_field(parameter, SUPPORTED_PARAMS, 'parameter')
However, the issue here is that we'd be manually supplying SUPPORTED_PARAMS, which is it's own issue, because we need to continually manually check these against the openAPI spec.
- 主要言語
- Python
- スター
- 300
- フォーク
- 99
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
planetlabs/planet-client-python のほかの issue
-
better-testing
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
planetlabs/planet-client-python#1110 ·
-
FeaturesClient list_items does not support filtering対応中かも @steps-re が 97 日前に担当しました。 オープンproposal
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
planetlabs/planet-client-python#1221 ·
-
FeaturesClient does not implement Update Feature対応中かも @tbarsballe が 126 日前に担当しました。 オープンproposal
難易度 3/5 1〜2日 初心者へのやさしさ 57/100
planetlabs/planet-client-python#1214 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
planetlabs/planet-client-python#1197 · コメント 1 件 · リアクション 1 件 ·
-
Quota API再び着手できるかも @asonnenschein が 425 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンai-claude
planetlabs/planet-client-python#1191 · コメント 1 件 · 担当者 1 名 ·
planetlabs/planet-client-python の issue をすべて見る
似ている issue
-
enhancement from-review
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
khuisman/mcp-gee-sweet#1010 ·
メンテナーはふだん 1 日以内に返信
-
bug confirmed issue
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
open-webui/open-webui#32333 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
ll7/robot_sf_ll7#10368 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
flop-labs/technocore-chat#982 ·
メンテナーはふだん 1 日以内に返信