azure/group: when deleting group, consider 404 success

Co-authored-by: Christer Edvartsen <[email protected]>

internal/azureclient/azureclient.go

@@ -283,6 +283,10 @@ func (s *client) DeleteGroup(ctx context.Context, grpID uuid.UUID) error {
 	}
 	defer resp.Body.Close()
 
+	if resp.StatusCode == http.StatusNotFound {
+		return nil
+	}
+
 	if resp.StatusCode != http.StatusNoContent {
 		body, _ := io.ReadAll(resp.Body)
 		return fmt.Errorf("remove azure group with ID: %q: %q: %q", grpID, resp.Status, string(body))

internal/azureclient/azureclient_test.go

@@ -706,4 +706,18 @@ func Test_DeleteGroup(t *testing.T) {
 			t.Errorf("Expected error to contain %q, got %q", contains, err)
 		}
 	})
+
+	t.Run("404 is considered OK", func(t *testing.T) {
+		httpClient := test.NewTestHttpClient(
+			func(req *http.Request) *http.Response {
+				return test.Response("404 Not Found", "{}")
+			},
+		)
+
+		client := azureclient.New(httpClient)
+		err := client.DeleteGroup(ctx, grpID)
+		if err != nil {
+			t.Fatalf("Expected no error, got %v", err)
+		}
+	})
 }

internal/reconcilers/manager.go

@@ -171,7 +171,7 @@ func (m *Manager) ListenForEvents(ctx context.Context) {
 		if err == nil || errors.Is(err, context.Canceled) {
 			// Receive will return nil error when canceled
 			return
-		} else if err != nil {
+		} else {
 			m.log.WithError(err).Error("error while receiving pubsub message")
 		}