[quality] fix: solver test emoji mismatch breaking go test on all PRs (P0)#19955
[quality] fix: solver test emoji mismatch breaking go test on all PRs (P0)#19955clubanderson wants to merge 1 commit into
Conversation
solver.go uses \u2746 (❆) for resolved notifications but solver_coverage_test.go expected \u2726 (✦), causing go test failures on main and all open PRs. Update both test assertions (lines 136 and 456) to match the production emoji. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: sec-check <sec-check@hive.kubestellar.io>
✅ Deploy Preview for kubestellarconsole canceled.
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 Hey @clubanderson — thanks for opening this PR!
This is an automated message. |
|
🐝 Hi @clubanderson! I'm Trusted users — org members and contributors with write access — can mention Automation may take a moment to start, and follow-up happens through workflow activity rather than chat replies. |
There was a problem hiding this comment.
Pull request overview
Fixes failing solver termination tests by aligning the “resolved” notification title emoji assertion with the production implementation in terminate().
Changes:
- Update
TestTerminateResolvedexpected notification title to use\u2746(❆) instead of\u2726(✦). - Update the “resolved status” table case in
TestTerminateAllStatusTypesto expect the same\u2746(❆) title string.
|
Closing — superseded by #19951 which includes both the emoji test fix and nil-safety guards for |
Test Fix (P0)
Aligns
solver_coverage_test.goemoji assertions with the production code insolver.go.Root cause:
solver.gowas updated to use\u2746(❆ six-pointed star) for resolved notifications, but the test file still expected\u2726(✦ four-pointed star). This causesTestTerminateResolvedandTestTerminateAllStatusTypes/resolved_statusto fail, blockinggo test ./...on all open PRs.Fix: Update both test assertions (lines 136 and 456) to match the production emoji
\u2746.Filed by quality agent (hold-gated mode). Human review required.