From cd655421e9c7312fddeef927ae4a40b01865f5a8 Mon Sep 17 00:00:00 2001 From: James Pickett Date: Tue, 24 Sep 2024 11:32:53 -0700 Subject: [PATCH] dont display desktop in test, just manage process --- ee/desktop/runner/runner_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ee/desktop/runner/runner_test.go b/ee/desktop/runner/runner_test.go index 211a9ace5..9e1cfcbf9 100644 --- a/ee/desktop/runner/runner_test.go +++ b/ee/desktop/runner/runner_test.go @@ -123,7 +123,10 @@ func TestDesktopUserProcessRunner_Execute(t *testing.T) { mockKnapsack.On("DesktopUpdateInterval").Return(time.Millisecond * 250) mockKnapsack.On("DesktopMenuRefreshInterval").Return(time.Millisecond * 250) mockKnapsack.On("KolideServerURL").Return("somewhere-over-the-rainbow.example.com") - mockKnapsack.On("DesktopEnabled").Return(true) + + // don't try to display desktop under test, causes CI flakeyness + // (we still test process management, just without trying to display the desktop) + mockKnapsack.On("DesktopEnabled").Return(false) mockKnapsack.On("Slogger").Return(slogger) mockKnapsack.On("InModernStandby").Return(false) mockKnapsack.On("SystrayRestartEnabled").Return(false).Maybe()