Add GET /api/v1/download/linux-package
Redirects to the newest MusicPlayer release's _linux.tar.gz, which has the icon, a menu entry and install.sh. /download/linux keeps serving the bare .x86_64, which updating in place needs. Releases without the package get 404. Closes #34 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
1 parent
bb8de5548d
commit
c70eb4d13c
4 files changed
+44
No files matched your search
@@ -222,7 +222,27 @@ func TestDownloadEndpoints(t *testing.T) {
|
||||
assert.Equal(t, http.StatusFound, rec.Code)
|
||||
assert.Equal(t, "https://example.com/Player.exe", rec.Header().Get("Location"))
|
||||
|
||||
// A release without Linux files, like those before 1.9.
|
||||
assert.Equal(t, http.StatusNotFound, e.do(http.MethodGet, "/api/v1/download/linux").Code)
|
||||
assert.Equal(t, http.StatusNotFound, e.do(http.MethodGet, "/api/v1/download/linux-package").Code)
|
||||
}
|
||||
|
||||
func TestDownloadLinuxBinaryAndPackage(t *testing.T) {
|
||||
gitea := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
||||
w.Write([]byte(`{"id":1,"name":"v1.9.0","assets":[` +
|
||||
`{"name":"MusicPlayer_1.9.0_linux.tar.gz","browser_download_url":"https://example.com/package"},` +
|
||||
`{"name":"MusicPlayer_1.9.0.x86_64","browser_download_url":"https://example.com/binary"}]}`))
|
||||
}))
|
||||
t.Cleanup(gitea.Close)
|
||||
e := newTestEnv(t, func(c *config.Config) { c.ReleasesURL = gitea.URL })
|
||||
|
||||
rec := e.do(http.MethodGet, "/api/v1/download/linux")
|
||||
assert.Equal(t, http.StatusFound, rec.Code)
|
||||
assert.Equal(t, "https://example.com/binary", rec.Header().Get("Location"), "the bare binary, for updating in place")
|
||||
|
||||
rec = e.do(http.MethodGet, "/api/v1/download/linux-package")
|
||||
assert.Equal(t, http.StatusFound, rec.Code)
|
||||
assert.Equal(t, "https://example.com/package", rec.Header().Get("Location"))
|
||||
}
|
||||
|
||||
func TestDownloadUpstreamDown(t *testing.T) {
|
||||
|
||||
@@ -102,4 +102,6 @@ func (s *Server) registerMisc(a huma.API) {
|
||||
}
|
||||
download("download-client-windows", "/download/windows", "Windows", ".exe")
|
||||
download("download-client-linux", "/download/linux", "Linux", ".x86_64")
|
||||
// The package has the icon, a menu entry and install.sh. Releases before 1.9 have none (404).
|
||||
download("download-client-linux-package", "/download/linux-package", "Linux (package with installer)", "_linux.tar.gz")
|
||||
}
|
||||
@@ -81,6 +81,7 @@ func TestOpenAPIDocumentsTheV1API(t *testing.T) {
|
||||
"GET /api/v1/character/list",
|
||||
"GET /api/v1/download",
|
||||
"GET /api/v1/download/linux",
|
||||
"GET /api/v1/download/linux-package",
|
||||
"GET /api/v1/download/list",
|
||||
"GET /api/v1/download/windows",
|
||||
"GET /api/v1/health",
|
||||
@@ -235,6 +236,7 @@ func TestResponsesMatchOpenAPI(t *testing.T) {
|
||||
v.check(http.MethodGet, "/api/v1/download/list", "", http.StatusOK)
|
||||
v.check(http.MethodGet, "/api/v1/download/windows", "", http.StatusFound)
|
||||
v.check(http.MethodGet, "/api/v1/download/linux", "", http.StatusNotFound)
|
||||
v.check(http.MethodGet, "/api/v1/download/linux-package", "", http.StatusNotFound)
|
||||
v.check(http.MethodGet, "/api/v1/health", "", http.StatusOK)
|
||||
v.check(http.MethodGet, "/api/v1/version", "", http.StatusOK)
|
||||
v.check(http.MethodGet, "/api/v1/version/history", "", http.StatusOK)
|
||||
|
||||
Reference in new issue
Block a user