Skip to content

Vcst restapi phase3 coverage#140

Merged
Lenajava1 merged 1 commit into
devfrom
vcst-restapi-phase3-coverage
Apr 17, 2026
Merged

Vcst restapi phase3 coverage#140
Lenajava1 merged 1 commit into
devfrom
vcst-restapi-phase3-coverage

Conversation

@Lenajava1
Copy link
Copy Markdown
Collaborator

No description provided.

…alisation tag CRUD

orders/test_orders.py:
- add make_order factory fixture with auto-teardown
- refactor test_order_create to use factory (drops inline cleanup)
- add test_order_get_by_id (GET by id, verify fields)
- add test_order_update_status (PUT status=Processing, verify via GET)
- add test_order_get_not_found (idempotent — assert empty or 404)

contacts/test_contact.py:
- add test_contact_get_not_found (HTTPError → 404 or empty result)

platform/test_user_password.py:
- add test_user_password_change_wrong_old — verifies wrong-old-password is rejected and original password still works

catalog_personalisation/test_personalisation.py:
- add test_tag_add_to_member_groups (serial) — real round-trip through /api/platform/settings: add tag, verify in GET, restore via try/finally

Results: 265 passed, 5 skipped, 2 destructive deselected. Delta: +6 tests.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Lenajava1 Lenajava1 force-pushed the vcst-restapi-phase3-coverage branch from bf37b77 to 53d3be1 Compare April 17, 2026 12:07
@github-actions
Copy link
Copy Markdown

REST API Test Results

271 tests   266 ✅  23s ⏱️
  3 suites    5 💤
  3 files      0 ❌

Results for commit 53d3be1.

@Lenajava1 Lenajava1 merged commit 5b090ab into dev Apr 17, 2026
3 checks passed
@Lenajava1 Lenajava1 deleted the vcst-restapi-phase3-coverage branch April 29, 2026 18:52
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