From 2965856aa34575104fffeb5266932a8629bea2ff Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Mon, 4 Jul 2022 18:56:24 +0200 Subject: [PATCH] Worker: add test flag to enable Blender-dependent test Add a `-withBlender` CLI argument for a unit test, to aid in debugging T99438. Run the test with `go test ./internal/worker/find_blender/ -args -withBlender` to actually fail when the file association with `.blend` files cannot be found. Note that this doesn't rely on Blender being runnable, but it does rely on _something_ being associated with .blend files. --- internal/worker/find_blender/windows_test.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/internal/worker/find_blender/windows_test.go b/internal/worker/find_blender/windows_test.go index 618cc345..c126573f 100644 --- a/internal/worker/find_blender/windows_test.go +++ b/internal/worker/find_blender/windows_test.go @@ -4,17 +4,24 @@ package find_blender import ( + "flag" "testing" "github.com/stretchr/testify/assert" ) +var withBlender = flag.Bool("withBlender", false, "run test that requires Blender to be installed") + // TestFindBlender is a "weak" test, which actually accepts both happy and unhappy flows. // It would be too fragile to always require a file association to be set up with Blender. func TestFindBlender(t *testing.T) { exe, err := FindBlender() if err != nil { assert.Empty(t, exe) + + if *withBlender { + t.Fatalf("unexpected error: %v", err) + } return } assert.NotEmpty(t, exe)