particle-os/debian-forge-composer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

cloudapi: Fix listing composes with empty result

Browse source 
It was returning a null body instead of an empty list.

Related: RHEL-60120
This commit is contained in:
Brian C. Lane 2025-05-06 17:03:19 -07:00 • committed by Tomáš Hozza
parent b33f86783f
commit 12dd0b0be4
2 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
internal/cloudapi/v2/handler.go
View file

@ -313,7 +313,7 @@ func (h *apiHandlers) GetComposeList(ctx echo.Context) error {
} }
// Gather up the details of each job // Gather up the details of each job
var stats []ComposeStatus stats := []ComposeStatus{}
for _, jid := range jobs { for _, jid := range jobs {
s, err := h.getJobIDComposeStatus(jid) s, err := h.getJobIDComposeStatus(jid)
if err != nil { if err != nil {

4
internal/cloudapi/v2/v2_test.go
View file

@ -1887,6 +1887,10 @@ func TestComposesRoute(t *testing.T) {
srv, _, _, cancel := newV2Server(t, t.TempDir(), false, false) srv, _, _, cancel := newV2Server(t, t.TempDir(), false, false)
defer cancel() defer cancel()
// List empty root composes
test.TestRoute(t, srv.Handler("/api/image-builder-composer/v2"), false, "GET", "/api/image-builder-composer/v2/composes/", ``,
http.StatusOK, `[]`)
// Make a compose so it has something to list // Make a compose so it has something to list
reply := test.TestRouteWithReply(t, srv.Handler("/api/image-builder-composer/v2"), false, "POST", "/api/image-builder-composer/v2/compose", fmt.Sprintf(` reply := test.TestRouteWithReply(t, srv.Handler("/api/image-builder-composer/v2"), false, "POST", "/api/image-builder-composer/v2/compose", fmt.Sprintf(`
{ {