Skip to content

Fix stale Access application path parameter tests - #2748

Open
FranciscoMaxwell wants to merge 1 commit into
cloudflare:mainfrom
FranciscoMaxwell:fix-stale-access-applications-path-tests
Open

FranciscoMaxwell wants to merge 1 commit into
cloudflare:mainfrom
FranciscoMaxwell:fix-stale-access-applications-path-tests

Conversation

@FranciscoMaxwell

Copy link
Copy Markdown
  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

This updates stale Access application path-parameter tests so they match the current account/zone routing behavior.

  • Keeps the empty account_id validation assertion.
  • Stops expecting a valid account_id call to raise ValueError.
  • Applies the same correction to the sync and async list path-parameter tests.

Additional context & links

Related to #2705 and #2747.

Testing

  • python -m compileall -q tests\api_resources\zero_trust\access\test_applications.py
  • git diff --check

I could not run the focused pytest command locally because pytest is not installed in my environment. The touched tests are also currently marked skipped in the generated suite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant