Refactor handlers and update changelog for 5.0.0-Beta

- Split IndexHandler into HealthHandler, VersionHandler, and CharacterHandler
- Rename index.go to version.go in backend
- Change VersionData.Changelog from string to []string
- Add changelog entries for issues #16-#23
- Remove TestDB function and related code
- Fix import ordering in several files

Closes #21, #22
References #16, #17, #18, #19, #20, #23

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>

internal/server/characterHandler.go

@@ -0,0 +1,42 @@
+package server
+
+import (
+	"net/http"
+
+	"github.com/labstack/echo/v5"
+	"music-server/internal/backend"
+)
+
+type CharacterHandler struct {
+}
+
+func NewCharacterHandler() *CharacterHandler {
+	return &CharacterHandler{}
+}
+
+// GetCharacterList godoc
+// @Summary Get list of characters
+// @Description Returns a list of all available characters
+// @Tags characters
+// @Accept json
+// @Produce json
+// @Success 200 {array} string
+// @Router /characters [get]
+func (c *CharacterHandler) GetCharacterList(ctx *echo.Context) error {
+	characters := backend.GetCharacterList()
+	return ctx.JSON(http.StatusOK, characters)
+}
+
+// GetCharacter godoc
+// @Summary Get character image
+// @Description Returns the image for a specific character
+// @Tags characters
+// @Accept json
+// @Produce image/png
+// @Param name query string true "Character name"
+// @Success 200 {file} file
+// @Router /character [get]
+func (c *CharacterHandler) GetCharacter(ctx *echo.Context) error {
+	character := ctx.QueryParam("name")
+	return ctx.File(backend.GetCharacter(character))
+}

internal/server/character_handler_test.go

@@ -5,45 +5,9 @@ import (
 	"net/http"
 	"testing"
 
-	"music-server/internal/backend"
-	"music-server/internal/db"
-
 	"github.com/stretchr/testify/assert"
 )
 
-// TestHealthCheck verifies the health endpoint returns database status
-func TestHealthCheck(t *testing.T) {
-	// Setup database
-	db.TestSetupDB(t)
-	defer db.TestTearDownDB(t)
-
-	e := StartTestServer(t)
-
-	resp := MakeTestRequest(t, e, "GET", "/health")
-	assert.Equal(t, http.StatusOK, resp.Code)
-
-	var healthData map[string]string
-	err := json.Unmarshal(resp.Body.Bytes(), &healthData)
-	assert.NoError(t, err)
-	assert.NotEmpty(t, healthData)
-	assert.Equal(t, "up", healthData["status"])
-}
-
-// TestGetVersion verifies the version endpoint returns version history
-func TestGetVersion(t *testing.T) {
-	e := StartTestServer(t)
-
-	resp := MakeTestRequest(t, e, "GET", "/version")
-	assert.Equal(t, http.StatusOK, resp.Code)
-
-	var versionData backend.VersionData
-	err := json.Unmarshal(resp.Body.Bytes(), &versionData)
-	assert.NoError(t, err)
-	assert.NotEmpty(t, versionData.Version)
-	assert.NotEmpty(t, versionData.Changelog)
-	assert.NotEmpty(t, versionData.History)
-}
-
 // TestGetCharacterList verifies the characters endpoint returns list of characters
 func TestGetCharacterList(t *testing.T) {
 	e := StartTestServer(t)
@@ -81,16 +45,3 @@ func TestGetCharacterNotFound(t *testing.T) {
 	// Should return 404 or similar error
 	assert.NotEqual(t, http.StatusOK, resp.Code)
 }
-
-// TestDBTest verifies the database test endpoint
-func TestDBTest(t *testing.T) {
-	// Setup database
-	db.TestSetupDB(t)
-	defer db.TestTearDownDB(t)
-
-	e := StartTestServer(t)
-
-	resp := MakeTestRequest(t, e, "GET", "/dbtest")
-	assert.Equal(t, http.StatusOK, resp.Code)
-	assert.Contains(t, resp.Body.String(), "TestedDB")
-}

internal/server/healthHandler.go

@@ -0,0 +1,28 @@
+package server
+
+import (
+	"net/http"
+
+	"github.com/labstack/echo/v5"
+	"music-server/internal/db"
+)
+
+type HealthHandler struct {
+}
+
+func NewHealthHandler() *HealthHandler {
+	return &HealthHandler{}
+}
+
+// HealthCheck godoc
+//
+//	@Summary		Check server health
+//	@Description	Returns the health status of the server
+//	@Tags			health
+//	@Accept			json
+//	@Produce		json
+//	@Success		200	{string}	string	"OK"
+//	@Router			/health [get]
+func (h *HealthHandler) HealthCheck(ctx *echo.Context) error {
+	return ctx.JSON(http.StatusOK, db.Health())
+}

internal/server/health_handler_test.go

@@ -0,0 +1,29 @@
+package server
+
+import (
+	"encoding/json"
+	"net/http"
+	"testing"
+
+	"music-server/internal/db"
+
+	"github.com/stretchr/testify/assert"
+)
+
+// TestHealthCheck verifies the health endpoint returns database status
+func TestHealthCheck(t *testing.T) {
+	// Setup database
+	db.TestSetupDB(t)
+	defer db.TestTearDownDB(t)
+
+	e := StartTestServer(t)
+
+	resp := MakeTestRequest(t, e, "GET", "/health")
+	assert.Equal(t, http.StatusOK, resp.Code)
+
+	var healthData map[string]string
+	err := json.Unmarshal(resp.Body.Bytes(), &healthData)
+	assert.NoError(t, err)
+	assert.NotEmpty(t, healthData)
+	assert.Equal(t, "up", healthData["status"])
+}

internal/server/indexHandler.go

@@ -1,86 +0,0 @@
-package server
-
-import (
-	"music-server/internal/backend"
-	"music-server/internal/db"
-	"net/http"
-
-	"github.com/labstack/echo/v5"
-)
-
-type IndexHandler struct {
-}
-
-func NewIndexHandler() *IndexHandler {
-	return &IndexHandler{}
-}
-
-// GetVersion godoc
-//
-//	@Summary		Getting the version of the backend
-//	@Description	get string by ID
-//	@Tags			accounts
-//	@Accept			json
-//	@Produce		json
-//	@Success		200	{object}	backend.VersionData
-//	@Failure		404	{object}	string
-//	@Router			/version [get]
-func (i *IndexHandler) GetVersion(ctx *echo.Context) error {
-	versionHistory := backend.GetVersionHistory()
-	if versionHistory.Version == "" {
-		return ctx.JSON(http.StatusNotFound, "version not found")
-	}
-	return ctx.JSON(http.StatusOK, versionHistory)
-}
-
-// GetDBTest godoc
-// @Summary Test database connection
-// @Description Tests the database connection
-// @Tags database
-// @Accept json
-// @Produce json
-// @Success 200 {string} string "TestedDB"
-// @Router /dbtest [get]
-func (i *IndexHandler) GetDBTest(ctx *echo.Context) error {
-	backend.TestDB()
-	return ctx.JSON(http.StatusOK, "TestedDB")
-}
-
-// HealthCheck godoc
-// @Summary Check server health
-// @Description Returns the health status of the server
-// @Tags health
-// @Accept json
-// @Produce json
-// @Success 200 {string} string "OK"
-// @Router /health [get]
-func (i *IndexHandler) HealthCheck(ctx *echo.Context) error {
-	return ctx.JSON(http.StatusOK, db.Health())
-}
-
-// GetCharacterList godoc
-// @Summary Get list of characters
-// @Description Returns a list of all available characters
-// @Tags characters
-// @Accept json
-// @Produce json
-// @Success 200 {array} string
-// @Router /characters [get]
-func (i *IndexHandler) GetCharacterList(ctx *echo.Context) error {
-	characters := backend.GetCharacterList()
-	return ctx.JSON(http.StatusOK, characters)
-}
-
-// GetCharacter godoc
-// @Summary Get character image
-// @Description Returns the image for a specific character
-// @Tags characters
-// @Accept json
-// @Produce image/png
-// @Param name query string true "Character name"
-// @Success 200 {file} file
-// @Router /character [get]
-func (i *IndexHandler) GetCharacter(ctx *echo.Context) error {
-	character := ctx.QueryParam("name")
-	return ctx.File(backend.GetCharacter(character))
-}

internal/server/routes.go

@@ -10,8 +10,8 @@ import (
 	"github.com/labstack/echo/v5"
 	"github.com/labstack/echo/v5/middleware"
 	echoSwagger "github.com/swaggo/echo-swagger/v2"
-	"music-server/internal/logging"
 	"go.uber.org/zap"
+	"music-server/internal/logging"
 )
 
 //	@Title			MusicServer API
@@ -29,7 +29,7 @@ import (
 // @BasePath	/
 func (s *Server) RegisterRoutes() http.Handler {
 	e := echo.New()
-	
+
 	// Serve OpenAPI spec at /openapi
 	e.GET("/openapi", echo.WrapHandler(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		w.Header().Set("Content-Type", "application/json")
@@ -57,12 +57,16 @@ func (s *Server) RegisterRoutes() http.Handler {
 	// Swagger UI
 	e.GET("/swagger/*", echoSwagger.WrapHandler)
 
-	index := NewIndexHandler()
-	e.GET("/version", index.GetVersion)
-	e.GET("/dbtest", index.GetDBTest)
-	e.GET("/health", index.HealthCheck)
-	e.GET("/character", index.GetCharacter)
-	e.GET("/characters", index.GetCharacterList)
+	health := NewHealthHandler()
+	e.GET("/health", health.HealthCheck)
+
+	version := NewVersionHandler()
+	e.GET("/version", version.GetLatestVersion)
+	e.GET("/version/history", version.GetVersionHistory)
+
+	character := NewCharacterHandler()
+	e.GET("/character", character.GetCharacter)
+	e.GET("/characters", character.GetCharacterList)
 
 	download := NewDownloadHandler()
 	e.GET("/download", download.checkLatest)

internal/server/versionHandler.go

@@ -0,0 +1,51 @@
+package server
+
+import (
+	"net/http"
+
+	"github.com/labstack/echo/v5"
+	"music-server/internal/backend"
+)
+
+type VersionHandler struct {
+}
+
+func NewVersionHandler() *VersionHandler {
+	return &VersionHandler{}
+}
+
+// GetVersionHistory godoc
+//
+//	@Summary		Getting the version history of the backend
+//	@Description	get version history
+//	@Tags			version
+//	@Accept			json
+//	@Produce		json
+//	@Success		200	{array}	backend.VersionData
+//	@Failure		404	{object}	string
+//	@Router			/version/history [get]
+func (v *VersionHandler) GetVersionHistory(ctx *echo.Context) error {
+	versionHistory := backend.GetVersionHistory()
+	if len(versionHistory) == 0 {
+		return ctx.JSON(http.StatusNotFound, "version not found")
+	}
+	return ctx.JSON(http.StatusOK, versionHistory)
+}
+
+// GetLatestVersion godoc
+//
+//	@Summary		Getting the latest version of the backend
+//	@Description	get latest version info
+//	@Tags			version
+//	@Accept			json
+//	@Produce		json
+//	@Success		200	{object}	backend.VersionData
+//	@Failure		404	{object}	string
+//	@Router			/version [get]
+func (v *VersionHandler) GetLatestVersion(ctx *echo.Context) error {
+	latestVersion := backend.GetLatestVersion()
+	if latestVersion.Version == "" {
+		return ctx.JSON(http.StatusNotFound, "version not found")
+	}
+	return ctx.JSON(http.StatusOK, latestVersion)
+}

internal/server/version_handler_test.go

@@ -0,0 +1,40 @@
+package server
+
+import (
+	"encoding/json"
+	"net/http"
+	"testing"
+
+	"music-server/internal/backend"
+
+	"github.com/stretchr/testify/assert"
+)
+
+// TestGetLatestVersion verifies the version endpoint returns latest version
+func TestGetLatestVersion(t *testing.T) {
+	e := StartTestServer(t)
+
+	resp := MakeTestRequest(t, e, "GET", "/version")
+	assert.Equal(t, http.StatusOK, resp.Code)
+
+	var versionData backend.VersionData
+	err := json.Unmarshal(resp.Body.Bytes(), &versionData)
+	assert.NoError(t, err)
+	assert.NotEmpty(t, versionData.Version)
+	assert.NotEmpty(t, versionData.Changelog)
+}
+
+// TestGetVersionHistory verifies the version history endpoint returns version history
+func TestGetVersionHistory(t *testing.T) {
+	e := StartTestServer(t)
+
+	resp := MakeTestRequest(t, e, "GET", "/version/history")
+	assert.Equal(t, http.StatusOK, resp.Code)
+
+	var versionHistory []backend.VersionData
+	err := json.Unmarshal(resp.Body.Bytes(), &versionHistory)
+	assert.NoError(t, err)
+	assert.NotEmpty(t, versionHistory)
+	assert.NotEmpty(t, versionHistory[0].Version)
+	assert.NotEmpty(t, versionHistory[0].Changelog)
+}