Skip to content

Commit

Permalink
dont display desktop in test, just manage process
Browse files Browse the repository at this point in the history
  • Loading branch information
James-Pickett committed Sep 24, 2024
1 parent 9524d0d commit cd65542
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion ee/desktop/runner/runner_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand Down

0 comments on commit cd65542

Please sign in to comment.