Skip to content

Commit c642854

Browse files
snowmeadclaude
andcommitted
fix(test): only run external migrations when indexer is enabled
MSP nodes have SH_INDEXER_DB_AUTO_MIGRATE enabled by default and handle their own database migrations internally during startup. External migrations using the diesel CLI should only run for user nodes with embedded indexer (which have auto-migrate disabled via SH_INDEXER_DB_AUTO_MIGRATE="false"). This fixes CI test failures where diesel CLI is not installed on the runner, causing "Error running Diesel CLI" when fullnet tests try to run external migrations that are redundant (MSPs already migrated themselves). Before: External migrations ran for fullnet OR indexer After: External migrations only run for indexer 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 45c8b86 commit c642854

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

test/util/netLaunch/index.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -380,8 +380,9 @@ export class NetworkLauncher {
380380
log: verbose
381381
});
382382

383-
// Run migrations for fullnet (MSPs need the schema) or when indexer is enabled
384-
if (this.type === "fullnet" || this.config.indexer) {
383+
// Only run external migrations when indexer enabled
384+
// (MSPs and standalone indexer auto-migrate themselves)
385+
if (this.config.indexer) {
385386
await this.runMigrations();
386387
}
387388

0 commit comments

Comments
 (0)