Skip to content

Comments

test: add tests for migrate command#2822

Open
ivanauth wants to merge 1 commit intoauthzed:mainfrom
ivanauth:fix/issue-2758-migrate-tests
Open

test: add tests for migrate command#2822
ivanauth wants to merge 1 commit intoauthzed:mainfrom
ivanauth:fix/issue-2758-migrate-tests

Conversation

@ivanauth
Copy link
Contributor

Description

Fixes #2758 by refactoring migrate execution for testability and adding coverage.

Changes

  • Extract MigrateConfig
  • Add executeMigrate helper
  • Add migrate tests

@ivanauth ivanauth requested a review from a team as a code owner January 15, 2026 14:16
@github-actions github-actions bot added area/cli Affects the command line area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) labels Jan 15, 2026
@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

❌ Patch coverage is 92.50000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.04%. Comparing base (0b5d8c5) to head (1d3699c).
⚠️ Report is 57 commits behind head on main.

Files with missing lines Patch % Lines
pkg/cmd/migrate.go 92.50% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2822      +/-   ##
==========================================
- Coverage   78.46%   75.04%   -3.42%     
==========================================
  Files         459      484      +25     
  Lines       44674    58046   +13372     
==========================================
+ Hits        35048    43553    +8505     
- Misses       6769    11445    +4676     
- Partials     2857     3048     +191     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ivanauth ivanauth force-pushed the fix/issue-2758-migrate-tests branch from 853f50b to ba5810a Compare February 9, 2026 20:16
@ivanauth ivanauth force-pushed the fix/issue-2758-migrate-tests branch from ba5810a to 1d3699c Compare February 9, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli Affects the command line area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: write tests for "migrateRun"

1 participant