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

weldr: messages → msgs in status route

Browse source 
composer-cli expects `msgs`, not `messages`, and throws an exception if
it doesn't exist:

    https://github.com/weldr/lorax/blob/master/src/composer/cli/status.py#L52

It is not used anywhere else, so this is a safe change.
This commit is contained in:
Lars Karlitski 2020-05-25 12:31:26 +02:00 • committed by Tom Gundersen
parent 7c0d384fde
commit 8e0dd790b7
2 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
internal/weldr/api.go
View file

@ -286,7 +286,7 @@ func (api *API) statusHandler(writer http.ResponseWriter, request *http.Request,
SchemaVersion string `json:"schema_version"`
Backend string `json:"backend"`
Build string `json:"build"`
Messages []string `json:"messages"`
Messages []string `json:"msgs"`
}
err := json.NewEncoder(writer).Encode(reply{

2
internal/weldr/api_test.go
View file

@ -46,7 +46,7 @@ func TestBasic(t *testing.T) {
ExpectedStatus int
ExpectedJSON string
}{
{"/api/status", http.StatusOK, `{"api":"1","db_supported":true,"db_version":"0","schema_version":"0","backend":"osbuild-composer","build":"devel","messages":[]}`},
{"/api/status", http.StatusOK, `{"api":"1","db_supported":true,"db_version":"0","schema_version":"0","backend":"osbuild-composer","build":"devel","msgs":[]}`},
{"/api/v0/projects/source/list", http.StatusOK, `{"sources":["test-id"]}`},