test: rename cloud_only marker as access_tokens

[quettabit]

No description provided.

[greptile-apps]

Greptile Summary

This PR renames the cloud_only pytest marker to access_tokens, reflecting a more precise semantic: the marker now specifically identifies access-token-related tests rather than broadly "cloud-only" tests. As a side effect, test_get_basin_config loses its marker entirely and will now run in the s2-lite integration test suite, which appears intentional since that test doesn't involve access tokens and is presumably supported on s2-lite.

Confidence Score: 5/5

Safe to merge — pure marker rename with intentional, low-risk behavioural change for one test.

All findings are P2 or lower. The rename is consistent across pytest.ini, the CI workflow, and all test annotations. The removal of the marker from test_get_basin_config appears deliberate and correct.

No files require special attention.

Important Files Changed

Filename	Overview
pytest.ini	Renames the cloud_only marker definition to access_tokens with an updated description; clean and accurate.
.github/workflows/ci.yml	Updates the s2-lite integration test filter from not cloud_only to not access_tokens; the s2-cloud job already runs all account-marked tests so access-token tests remain covered there.
tests/test_account_ops.py	Replaces @pytest.mark.cloud_only with @pytest.mark.access_tokens on all access-token tests; test_get_basin_config intentionally loses its marker, enabling it to run on s2-lite.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[pytest test run] --> B{Marker filter}
    B --> C["local-checks\nnot (account or basin or stream or metrics)"]
    B --> D["s2-cloud integration\naccount or basin or stream or metrics"]
    B --> E["s2-lite integration\n(account or basin or stream) and not access_tokens"]

    C --> F[Unit tests only]
    D --> G[All account/basin/stream/metrics tests\nincl. access_tokens tests]
    E --> H[Account/basin/stream tests\nexcl. access_tokens tests]

    style G fill:#d4edda
    style H fill:#fff3cd

Loading

_{Reviews (1): Last reviewed commit: "initial commit" | Re-trigger Greptile}