jfraeysd/fetch_ml
0
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

test(e2e): skip gracefully when Redis unavailable, fix cross-device link
Some checks failed
Checkout test / test (push) Successful in 4s
Details
CI/CD Pipeline / Test (push) Failing after 1s
Details
CI/CD Pipeline / Dev Compose Smoke Test (push) Has been skipped
Details
CI/CD Pipeline / Build (push) Has been skipped
Details
CI/CD Pipeline / Test Scripts (push) Has been skipped
Details
CI/CD Pipeline / Test Native Libraries (push) Has been skipped
Details
Documentation / build-and-publish (push) Failing after 33s
Details
CI/CD Pipeline / Docker Build (push) Has been skipped
Details
Security Scan / Security Analysis (push) Has been cancelled
Details
Security Scan / Native Library Security (push) Has been cancelled
Details

Browse source 
- StartTemporaryRedis now skips tests instead of failing when redis-server unavailable

- Fix homelab_e2e_test cross-device link issue using CopyDir instead of Rename
This commit is contained in:
Jeremie Fraeys 2026-02-21 21:20:47 -05:00
parent bf4a8bcf78
commit 47ad62648d
No known key found for this signature in database
2 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tests/e2e/homelab_e2e_test.go
View file

@ -328,11 +328,11 @@ func TestConfigurationScenariosE2E(t *testing.T) {
// Backup original configs if they exist // Backup original configs if they exist
if _, err := os.Stat(originalConfigDir); err == nil { if _, err := os.Stat(originalConfigDir); err == nil {
if err := os.Rename(originalConfigDir, tempConfigDir); err != nil { if err := tests.CopyDir(originalConfigDir, tempConfigDir); err != nil {
t.Fatalf("Failed to backup configs: %v", err) t.Fatalf("Failed to backup configs: %v", err)
} }
defer func() { defer func() {
_ = os.Rename(tempConfigDir, originalConfigDir) _ = tests.CopyDir(tempConfigDir, originalConfigDir)
}() }()
} }

6
tests/fixtures/test_utils.go vendored
View file

@ -128,6 +128,12 @@ func EnsureRedis(t *testing.T) (cleanup func()) {
// Start temporary Redis // Start temporary Redis
t.Logf("Starting temporary Redis on %s", redisAddr) t.Logf("Starting temporary Redis on %s", redisAddr)
// Check if redis-server is available
if _, err := exec.LookPath("redis-server"); err != nil {
t.Skip("Skipping: redis-server not available in PATH")
}
cmd := exec.CommandContext( cmd := exec.CommandContext(
context.Background(), context.Background(),
"redis-server", "redis-server",