Stale Access application path-param tests expect valid account_id to raise

オープン 初心者向け
#2,747 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
88/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
python
領域
api, testing

調査の方向性

tests/api_resources/zero_trust/access/test_applications.py にあるスキップされたパスパラメータのテストから着手してください。特に test_path_params_create_overload_1 と、sync および async の test_path_params_list ケースを確認します。それらの期待値を src/cloudflare/resources/zero_trust/access/applications/applications.py と比較し、その後、対象を絞ったアプリケーションテストを実行してください。完了の条件は、空でない account_id が受け入れられ、空または欠落しているアカウントおよびゾーンの識別子だけがエラーを発生させることです。

索引モデルが issue の本文から書いたものです。

説明

Confirm this is a Python library issue and not an underlying Cloudflare API issue.
  • This is an issue with the Python library
Describe the bug

Some skipped path-param tests for zero_trust.access.applications appear to be stale after the account/zone path selection logic was fixed.

For example, these tests still expect ValueError("You must provide either account_id or zone_id") when account_id="account_id" is provided:

  • test_path_params_create_overload_1 in tests/api_resources/zero_trust/access/test_applications.py
  • test_path_params_list in both sync and async sections of the same file

That expectation now contradicts the current resource implementation, which correctly treats account_id as sufficient and builds an /accounts/{account_id}/... path. This looks related to the behavior discussed in #2705, where passing account_id should not require zone_id.

I noticed this while checking #2705. The runtime behavior on current main looks correct, but the skipped generated tests still encode the old/broken expectation.

To Reproduce
  1. Inspect the current main version of tests/api_resources/zero_trust/access/test_applications.py.
  2. See the valid-account_id branches in these tests:
  3. Compare with the current implementation in src/cloudflare/resources/zero_trust/access/applications/applications.py, where account_id selects the accounts path and does not need zone_id.
  4. A minimal local smoke check confirms the current behavior builds GET /accounts/acct_123/access/apps for client.zero_trust.access.applications.list(account_id="acct_123").

Expected: the path-param tests should only expect an error for empty/missing account and zone identifiers, not for a non-empty account_id.

Code snippets
client.zero_trust.access.applications.list(account_id="acct_123")
# current main builds: GET /accounts/acct_123/access/apps
OS

all

Python version

3.11.1

Library version

main / 5.7.0

主要言語
Python
スター
509
フォーク
150
平均マージ
3時間 15分
マージ済み PR(30日)
1

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

cloudflare/cloudflare-python のほかの issue

cloudflare/cloudflare-python の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。