Skip to content

adds migration for missing columns#1628

Merged
akshaydeo merged 1 commit intomainfrom
02-12-adds_migration_for_missing_columns
Feb 12, 2026
Merged

adds migration for missing columns#1628
akshaydeo merged 1 commit intomainfrom
02-12-adds_migration_for_missing_columns

Conversation

@akshaydeo
Copy link
Contributor

@akshaydeo akshaydeo commented Feb 12, 2026

Summary

Update migration tests to set azure_scopes to NULL instead of a specific value to prevent snapshot comparison diffs during config sync.

Changes

  • Modified the migration test scripts to set azure_scopes to NULL in both PostgreSQL and SQLite databases
  • Added comments explaining that config sync resets this column on startup, so non-null values would cause snapshot comparison differences

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (Next.js)
  • Docs

How to test

Run the migration tests to verify they complete successfully without snapshot comparison diffs:

# Run migration tests
./.github/workflows/scripts/run-migration-tests.sh

Breaking changes

  • Yes
  • No

Related issues

Fixes issues with migration test failures due to config sync resetting the azure_scopes column.

Security considerations

No security implications as this only affects test data.

Checklist

  • I read docs/contributing/README.md and followed the guidelines
  • I added/updated tests where appropriate
  • I updated documentation where needed
  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 02-12-adds_migration_for_missing_columns

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Contributor

🧪 Test Suite Available

This PR can be tested by a repository admin.

Run tests for PR #1628

@akshaydeo akshaydeo marked this pull request as ready for review February 12, 2026 16:43
@akshaydeo akshaydeo merged commit 8474a7e into main Feb 12, 2026
9 checks passed
@akshaydeo akshaydeo deleted the 02-12-adds_migration_for_missing_columns branch February 12, 2026 16:44
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