-
Notifications
You must be signed in to change notification settings - Fork 2
bug(tests): Windows snapshot test failure in config_default_snapshot_lsp_policy #1957
Copy link
Copy link
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Problem
Windows test on 2026-03-18 fails with snapshot assertion error:
thread 'config::types::tests::config_default_snapshot' panicked at:
snapshot assertion for 'config_default_snapshot_lsp_policy' failed in line 55
Test: config::types::tests::config_default_snapshot
Platform: Windows (windows-latest in CI)
Severity: HIGH — blocks all subsequent tests (fail-fast)
Reproduction: Run cargo nextest run --config-file .github/nextest.toml config::types::tests::config_default_snapshot on Windows
Root Cause
LSP policy configuration differs on Windows vs macOS/Linux. Either:
- Default LSP configuration changed and snapshot wasn't updated
- Snapshot comparison is platform-specific (e.g., path separators, line endings)
- Windows-specific config behavior not matching snapshot expectations
Investigation Needed
- Review snapshot file:
.snapshots/config_default_snapshot_lsp_policy.snap - Compare actual vs expected config in snapshot assertion
- Check if LSP policy has platform-specific defaults
- Verify snapshot serialization handles Windows line endings correctly
Related
Might be related to config changes in recent PRs — need to check if SecurityConfig struct lost Copy trait (known issue from workflow patterns).
Test Failure Log
Run ID: 23218939946
Job: Test (windows-latest)
Full logs: https://github.com/bug-ops/zeph/actions/runs/23218939946
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working