psa-systems/mokosh-server
Watch
9
0
Fork
You've already forked mokosh-server
 1
Code Issues Pull requests Projects Releases 10 Packages Wiki Activity Actions

test(kb): format assert_eq! in top-driving test, land coverage (PMS-533) #393

Merged
Claude-Run merged 1 commit from fix/PMS-533-kb-test-rustfmt into main 2026-06-26 17:27:44 +02:00
Conversation 0 Commits 1 Files changed 1 +236
Claude-Run commented 2026-06-26 17:14:32 +02:00
Member
Copy link

CI run #2698 (check.yml) failed at the rustfmt step on the test/PMS-485-top-ticket-driving-articles-endpoint branch: tests/knowledge_base.rs:1301 held a single-line assert_eq! that rustfmt wanted wrapped across multiple lines, so cargo fmt --all --check exited 1 and the integration coverage never merged into main.

This ports that test commit (top_ticket_driving_articles integration coverage for GET /api/v1/kb/top-ticket-driving-articles) onto main with the offending assert_eq! reformatted by cargo fmt, so check.yml (compile + clippy + fmt) is green.

#PMS-533
#PMS-485

CI run #2698 (check.yml) failed at the rustfmt step on the test/PMS-485-top-ticket-driving-articles-endpoint branch: tests/knowledge_base.rs:1301 held a single-line assert_eq! that rustfmt wanted wrapped across multiple lines, so cargo fmt --all --check exited 1 and the integration coverage never merged into main. This ports that test commit (top_ticket_driving_articles integration coverage for GET /api/v1/kb/top-ticket-driving-articles) onto main with the offending assert_eq! reformatted by cargo fmt, so check.yml (compile + clippy + fmt) is green. #PMS-533 #PMS-485
test(kb): format assert_eq! in top-driving test, land coverage (PMS-533)
All checks were successful
E2E / Playwright against staging (pull_request) Successful in 1m33s
Details
Check / fmt + clippy + build + tests (pull_request) Successful in 4m22s
Details
Integration / integration tests (pull_request) Successful in 13m4s
Details
Create release / Gate (release-branch merges only) (pull_request) Successful in 1s
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
f859ec0d66 
CI run #2698 (check.yml) failed at the rustfmt step on the test/PMS-485-top-ticket-driving-articles-endpoint branch: tests/knowledge_base.rs:1301 held a single-line assert_eq! that rustfmt wanted wrapped across multiple lines, so cargo fmt --all --check exited 1 and the integration coverage never merged into main.

This ports that test commit (top_ticket_driving_articles integration coverage for GET /api/v1/kb/top-ticket-driving-articles) onto main with the offending assert_eq! reformatted by cargo fmt, so check.yml (compile + clippy + fmt) is green.

#PMS-533
#PMS-485
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/mokosh-server!393
No description provided.