Use struct instead text, cover by tests

utils/http/helpers/helpers.go

@@ -1,6 +1,7 @@
 package helpers
 
 import (
+	"encoding/json"
 	"errors"
 	"fmt"
 	"log"
@@ -61,15 +62,41 @@ func HandleAppStatus() http.Handler {
 			RespondAsMethodNotAllowed(w, r)
 			return
 		}
+
 		var m runtime.MemStats
 		runtime.ReadMemStats(&m)
-		memory := fmt.Sprintf(
-			`{"alloc":"%v","total_alloc":"%v","sys":"%v","num_gc":"%v"}`,
-			m.Alloc, m.TotalAlloc, m.Sys, m.NumGC,
-		)
+
+		type respMemory struct {
+			Alloc      uint64 `json:"alloc"`
+			NumGC      uint32 `json:"num_gc"`
+			Sys        uint64 `json:"sys"`
+			TotalAlloc uint64 `json:"total_alloc"`
+		}
+
+		type respRoot struct {
+			Memory   respMemory `json:"memory"`
+			Routines int        `json:"routines"`
+		}
+
+		resp := respRoot{
+			Memory: respMemory{
+				Alloc:      m.Alloc,
+				NumGC:      m.NumGC,
+				Sys:        m.Sys,
+				TotalAlloc: m.TotalAlloc,
+			},
+			Routines: runtime.NumGoroutine(),
+		}
+
+		j, err := json.Marshal(resp)
+		if err != nil {
+			RespondAsBadRequest(w, r, err)
+			return
+		}
+
 		w.Header().Set("Content-Type", "application/json")
-		if _, err := w.Write([]byte(fmt.Sprintf(`{"routines":%d,"memory":%s}`, runtime.NumGoroutine(), memory))); err != nil {
-			log.Printf("%s\n", err.Error())
+		if _, err := w.Write(j); err != nil {
+			fmt.Printf("%s\n", err.Error())
 		}
 	})
 }

utils/http/helpers/helpers_test.go

@@ -1,7 +1,9 @@
 package helpers_test
 
 import (
+	"io"
 	"net/http"
+	"net/http/httptest"
 	"testing"
 
 	. "github.com/onsi/ginkgo"
@@ -10,6 +12,9 @@ import (
 )
 
 var _ = Describe("helpers", func() {
+	var srv *httptest.Server
+	var client *http.Client
+
 	Context("ClientIP", func() {
 		It("return client IP", func() {
 			Expect(helpers.ClientIP(&http.Request{
@@ -68,6 +73,31 @@ var _ = Describe("helpers", func() {
 		})
 	})
 
+	Context("HandleAppStatus", func() {
+		BeforeEach(func() {
+			srv = httptest.NewServer(helpers.HandleAppStatus())
+			client = srv.Client()
+		})
+
+		AfterEach(func() {
+			srv.Close()
+		})
+
+		It("handle app status", func() {
+			resp, err := client.Get(srv.URL + "/")
+			Expect(err).To(Succeed())
+			defer resp.Body.Close()
+
+			Expect(resp.StatusCode).To(Equal(http.StatusOK))
+			Expect(resp.Header.Get("Content-Type")).To(Equal("application/json"))
+
+			body, err := io.ReadAll(resp.Body)
+			Expect(err).To(Succeed())
+
+			Expect(string(body)).To(MatchRegexp(`{"memory":{"alloc":[0-9]+,"num_gc":[0-9]+,"sys":[0-9]+,"total_alloc":[0-9]+},"routines":[0-9]+}`))
+		})
+	})
+
 	Context("MinifyHtmlCode", func() {
 		It("minify Html code", func() {
 			Expect(helpers.MinifyHtmlCode(`