From ae6831ce6e8a0323f046471a5bc9f9974fe1f2ff Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Thu, 2 Jun 2022 12:13:25 +0200 Subject: [PATCH] Manager: fix unit test rFcfb17b178da2055ef12b2aa2ad8f7f778a952bc3 changed the semantics of `SocketIOWorkerUpdate`, in the sense that any update that doesn't change the worker status can omit `previous_status`. This commit adjusts the unit test for this. --- internal/manager/api_impl/worker_mgt_test.go | 1 - 1 file changed, 1 deletion(-) diff --git a/internal/manager/api_impl/worker_mgt_test.go b/internal/manager/api_impl/worker_mgt_test.go index 56e7c901..ab182cfc 100644 --- a/internal/manager/api_impl/worker_mgt_test.go +++ b/internal/manager/api_impl/worker_mgt_test.go @@ -143,7 +143,6 @@ func TestRequestWorkerStatusChange(t *testing.T) { mf.broadcaster.EXPECT().BroadcastWorkerUpdate(api.SocketIOWorkerUpdate{ Id: worker.UUID, Nickname: worker.Name, - PreviousStatus: &prevStatus, Status: prevStatus, StatusRequested: &requestStatus, LazyStatusRequest: ptr(true),