Skip to content

feat(ticketing): add sla policies CUD, reorder, definitions#87

Merged
lmeilibr merged 1 commit into
mainfrom
feature/sla-policies-cud
Apr 21, 2026
Merged

feat(ticketing): add sla policies CUD, reorder, definitions#87
lmeilibr merged 1 commit into
mainfrom
feature/sla-policies-cud

Conversation

@lmeilibr
Copy link
Copy Markdown
Contributor

Summary

  • Adds `CreateSlaPolicyCmd` / `UpdateSlaPolicyCmd` and a mapper that skips unset optionals
  • Extends `SlaPolicyApiClient` with `create`, `update`, `delete`, `reorder`, `list_definitions`
  • `SlaPoliciesService` exposes a `**fields` kwargs API consistent with the rest of the CUD series
  • 100% unit coverage across all five SLA policy modules (122 stmts); integration tests exercise every endpoint against a live tenant

Refs #79

Test plan

  • `uv run pytest tests/unit/ticketing/test_sla_policy.py tests/unit/ticketing/test_sla_policy_mapper.py tests/unit/ticketing/test_sla_policies_service.py` — 40 pass
  • Full unit suite: 1893 pass
  • Coverage: 122/122 statements
  • Integration tests against live tenant (gated, verified locally)

🤖 Generated with Claude Code

Adds CreateSlaPolicyCmd/UpdateSlaPolicyCmd dataclasses, mapper,
and API client methods for create, update, delete, reorder, and
list_definitions. Service exposes a **fields kwargs API consistent
with macros/triggers/automations/views. Integration tests cover
every endpoint end-to-end; unit tests achieve 100% coverage across
all five SLA policy modules.

Refs #79

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
position: int | None = None


SlaPolicyCmd: TypeAlias = CreateSlaPolicyCmd | UpdateSlaPolicyCmd
@sonarqubecloud
Copy link
Copy Markdown

@lmeilibr lmeilibr merged commit fabe35e into main Apr 21, 2026
3 checks passed
@lmeilibr lmeilibr deleted the feature/sla-policies-cud branch April 21, 2026 23:15
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