Skip to content

test: improve estimator and migration script coverage#57

Merged
phantom5099 merged 1 commit into
mainfrom
fork-pr-413-1776912961
Apr 23, 2026
Merged

test: improve estimator and migration script coverage#57
phantom5099 merged 1 commit into
mainfrom
fork-pr-413-1776912961

Conversation

@fennoai
Copy link
Copy Markdown

@fennoai fennoai Bot commented Apr 23, 2026

Requested by @phantom5099

Summary

  • add unit tests for provider token estimation helpers: EstimateSerializedPayloadTokens, EstimateTextTokens, and request signature stability
  • add unit tests for migration script output branches: changed dry-run, changed with backup, not changed with notes, and HOME-based default dir path
  • run go test ./... to verify no regression

…output

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>
@phantom5099 phantom5099 merged commit 4e88726 into main Apr 23, 2026
1 check passed
@fennoai fennoai Bot deleted the fork-pr-413-1776912961 branch April 23, 2026 05:03
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.

2 participants