• Johnny's avatar
    fix: set DRIVER=sqlite in CI to prevent TestMain from spawning child processes · e082adf7
    Johnny authored
    Problem:
    - store/test/TestMain checks if DRIVER env var is set
    - If not set, it runs tests for all 3 drivers (sqlite, mysql, postgres)
      by spawning child 'go test' processes
    - This conflicts with t.Parallel() in individual tests
    - CI workflow didn't set DRIVER, triggering multi-driver execution
    
    Solution:
    - Set DRIVER=sqlite in GitHub Actions workflow
    - TestMain will run tests once with SQLite driver
    - Tests run in parallel with t.Parallel() as intended
    - Avoids spawning child processes and race conditions
    
    Why SQLite:
    - Fastest test execution (no container startup)
    - Sufficient for CI validation
    - MySQL/Postgres can be tested locally when needed
    
    This fixes the 'table already exists' errors and test flakiness
    in CI while maintaining parallel execution benefits.
    e082adf7
Name
Last commit
Last update
.github Loading commit data...
cmd/memos Loading commit data...
internal Loading commit data...
plugin Loading commit data...
proto Loading commit data...
scripts Loading commit data...
server Loading commit data...
store Loading commit data...
web Loading commit data...
.dockerignore Loading commit data...
.gitignore Loading commit data...
.golangci.yaml Loading commit data...
AGENTS.md Loading commit data...
CODEOWNERS Loading commit data...
LICENSE Loading commit data...
README.md Loading commit data...
SECURITY.md Loading commit data...
go.mod Loading commit data...
go.sum Loading commit data...