Skip to content

refactor(tests): Split PromptsActivityTest into GET/PUT test classes#106414

Merged
dcramer merged 1 commit intomasterfrom
refactor/prompts-activity-tests-split-classes
Jan 16, 2026
Merged

refactor(tests): Split PromptsActivityTest into GET/PUT test classes#106414
dcramer merged 1 commit intomasterfrom
refactor/prompts-activity-tests-split-classes

Conversation

@dcramer
Copy link
Member

@dcramer dcramer commented Jan 15, 2026

Splits the single PromptsActivityTest class into separate test classes per HTTP method, following the pattern used elsewhere (e.g., test_organization_search_details.py).

Structure:

  • PromptsActivityTestBase - shared setUp logic
  • GetPromptsActivityTest - GET endpoint tests (2 tests)
  • PutPromptsActivityTest - PUT endpoint tests (13 tests)

This addresses pre-existing technical debt, not introduced by the original PR.

Follow-up to #104990.

Splits the single test class into GetPromptsActivityTest and
PutPromptsActivityTest with a shared base class, matching the pattern
used elsewhere (e.g., test_organization_search_details.py).

Follow-up to #104990.
@dcramer dcramer merged commit 20f5183 into master Jan 16, 2026
64 checks passed
@dcramer dcramer deleted the refactor/prompts-activity-tests-split-classes branch January 16, 2026 16:56
BYK pushed a commit that referenced this pull request Jan 19, 2026
…106414)

Splits the single `PromptsActivityTest` class into separate test classes
per HTTP method, following the pattern used elsewhere (e.g.,
`test_organization_search_details.py`).

Structure:
- `PromptsActivityTestBase` - shared setUp logic
- `GetPromptsActivityTest` - GET endpoint tests (2 tests)
- `PutPromptsActivityTest` - PUT endpoint tests (13 tests)

This addresses pre-existing technical debt, not introduced by the
original PR.

Follow-up to #104990.
@github-actions github-actions bot locked and limited conversation to collaborators Feb 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants