|
Packit |
63bb0d |
package worker_test
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
import (
|
|
Packit |
63bb0d |
"fmt"
|
|
Packit |
63bb0d |
"net/http"
|
|
Packit |
63bb0d |
"testing"
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
"github.com/google/uuid"
|
|
Packit |
63bb0d |
"github.com/stretchr/testify/require"
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
"github.com/osbuild/osbuild-composer/internal/distro"
|
|
Packit |
63bb0d |
"github.com/osbuild/osbuild-composer/internal/distro/fedoratest"
|
|
Packit |
63bb0d |
"github.com/osbuild/osbuild-composer/internal/jobqueue/testjobqueue"
|
|
Packit |
63bb0d |
"github.com/osbuild/osbuild-composer/internal/test"
|
|
Packit |
63bb0d |
"github.com/osbuild/osbuild-composer/internal/worker"
|
|
Packit |
63bb0d |
)
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
// Ensure that the status request returns OK.
|
|
Packit |
63bb0d |
func TestStatus(t *testing.T) {
|
|
Packit |
63bb0d |
server := worker.NewServer(nil, testjobqueue.New(), "")
|
|
Packit |
63bb0d |
test.TestRoute(t, server, false, "GET", "/status", ``, http.StatusOK, `{"status":"OK"}`, "message")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
func TestErrors(t *testing.T) {
|
|
Packit |
63bb0d |
var cases = []struct {
|
|
Packit |
63bb0d |
Method string
|
|
Packit |
63bb0d |
Path string
|
|
Packit |
63bb0d |
Body string
|
|
Packit |
63bb0d |
ExpectedStatus int
|
|
Packit |
63bb0d |
}{
|
|
Packit |
63bb0d |
// Bogus path
|
|
Packit |
63bb0d |
{"GET", "/foo", ``, http.StatusNotFound},
|
|
Packit |
63bb0d |
// Create job with invalid body
|
|
Packit |
63bb0d |
{"POST", "/job-queue/v1/jobs", ``, http.StatusBadRequest},
|
|
Packit |
63bb0d |
// Wrong method
|
|
Packit |
63bb0d |
{"GET", "/job-queue/v1/jobs", ``, http.StatusMethodNotAllowed},
|
|
Packit |
63bb0d |
// Update job with invalid ID
|
|
Packit |
63bb0d |
{"PATCH", "/job-queue/v1/jobs/foo", `{"status":"FINISHED"}`, http.StatusBadRequest},
|
|
Packit |
63bb0d |
// Update job that does not exist, with invalid body
|
|
Packit |
63bb0d |
{"PATCH", "/job-queue/v1/jobs/aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa", ``, http.StatusBadRequest},
|
|
Packit |
63bb0d |
// Update job that does not exist
|
|
Packit |
63bb0d |
{"PATCH", "/job-queue/v1/jobs/aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa", `{"status":"FINISHED"}`, http.StatusNotFound},
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
for _, c := range cases {
|
|
Packit |
63bb0d |
server := worker.NewServer(nil, testjobqueue.New(), "")
|
|
Packit |
63bb0d |
test.TestRoute(t, server, false, c.Method, c.Path, c.Body, c.ExpectedStatus, "{}", "message")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
func TestCreate(t *testing.T) {
|
|
Packit |
63bb0d |
distroStruct := fedoratest.New()
|
|
Packit |
63bb0d |
arch, err := distroStruct.GetArch("x86_64")
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error getting arch from distro")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
imageType, err := arch.GetImageType("qcow2")
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error getting image type from arch")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
manifest, err := imageType.Manifest(nil, distro.ImageOptions{Size: imageType.Size(0)}, nil, nil, nil)
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error creating osbuild manifest")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
server := worker.NewServer(nil, testjobqueue.New(), "")
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
id, err := server.Enqueue(manifest, nil)
|
|
Packit |
63bb0d |
require.NoError(t, err)
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
test.TestRoute(t, server, false, "POST", "/job-queue/v1/jobs", `{}`, http.StatusCreated,
|
|
Packit |
63bb0d |
`{"id":"`+id.String()+`","manifest":{"sources":{},"pipeline":{}}}`, "created")
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
test.TestRoute(t, server, false, "GET", fmt.Sprintf("/job-queue/v1/jobs/%s", id), `{}`, http.StatusOK,
|
|
Packit |
63bb0d |
`{"id":"`+id.String()+`","canceled":false}`)
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
func TestCancel(t *testing.T) {
|
|
Packit |
63bb0d |
distroStruct := fedoratest.New()
|
|
Packit |
63bb0d |
arch, err := distroStruct.GetArch("x86_64")
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error getting arch from distro")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
imageType, err := arch.GetImageType("qcow2")
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error getting image type from arch")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
manifest, err := imageType.Manifest(nil, distro.ImageOptions{Size: imageType.Size(0)}, nil, nil, nil)
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error creating osbuild manifest")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
server := worker.NewServer(nil, testjobqueue.New(), "")
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
id, err := server.Enqueue(manifest, nil)
|
|
Packit |
63bb0d |
require.NoError(t, err)
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
test.TestRoute(t, server, false, "POST", "/job-queue/v1/jobs", `{}`, http.StatusCreated,
|
|
Packit |
63bb0d |
`{"id":"`+id.String()+`","manifest":{"sources":{},"pipeline":{}}}`, "created")
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
err = server.Cancel(id)
|
|
Packit |
63bb0d |
require.NoError(t, err)
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
test.TestRoute(t, server, false, "GET", fmt.Sprintf("/job-queue/v1/jobs/%s", id), `{}`, http.StatusOK,
|
|
Packit |
63bb0d |
`{"id":"`+id.String()+`","canceled":true}`)
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
func testUpdateTransition(t *testing.T, from, to string, expectedStatus int) {
|
|
Packit |
63bb0d |
distroStruct := fedoratest.New()
|
|
Packit |
63bb0d |
arch, err := distroStruct.GetArch("x86_64")
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error getting arch from distro")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
imageType, err := arch.GetImageType("qcow2")
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error getting image type from arch")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
server := worker.NewServer(nil, testjobqueue.New(), "")
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
id := uuid.Nil
|
|
Packit |
63bb0d |
if from != "VOID" {
|
|
Packit |
63bb0d |
manifest, err := imageType.Manifest(nil, distro.ImageOptions{Size: imageType.Size(0)}, nil, nil, nil)
|
|
Packit |
63bb0d |
if err != nil {
|
|
Packit |
63bb0d |
t.Fatalf("error creating osbuild manifest")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
id, err = server.Enqueue(manifest, nil)
|
|
Packit |
63bb0d |
require.NoError(t, err)
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
if from != "WAITING" {
|
|
Packit |
63bb0d |
test.SendHTTP(server, false, "POST", "/job-queue/v1/jobs", `{}`)
|
|
Packit |
63bb0d |
if from != "RUNNING" {
|
|
Packit |
63bb0d |
test.SendHTTP(server, false, "PATCH", "/job-queue/v1/jobs/"+id.String(), `{"status":"`+from+`"}`)
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
test.TestRoute(t, server, false, "PATCH", "/job-queue/v1/jobs/"+id.String(), `{"status":"`+to+`"}`, expectedStatus, "{}", "message")
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
func TestUpdate(t *testing.T) {
|
|
Packit |
63bb0d |
var cases = []struct {
|
|
Packit |
63bb0d |
From string
|
|
Packit |
63bb0d |
To string
|
|
Packit |
63bb0d |
ExpectedStatus int
|
|
Packit |
63bb0d |
}{
|
|
Packit |
63bb0d |
{"VOID", "WAITING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"VOID", "RUNNING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"VOID", "FINISHED", http.StatusNotFound},
|
|
Packit |
63bb0d |
{"VOID", "FAILED", http.StatusNotFound},
|
|
Packit |
63bb0d |
{"WAITING", "WAITING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"WAITING", "RUNNING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"WAITING", "FINISHED", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"WAITING", "FAILED", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"RUNNING", "WAITING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"RUNNING", "RUNNING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"RUNNING", "FINISHED", http.StatusOK},
|
|
Packit |
63bb0d |
{"RUNNING", "FAILED", http.StatusOK},
|
|
Packit |
63bb0d |
{"FINISHED", "WAITING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"FINISHED", "RUNNING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"FINISHED", "FINISHED", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"FINISHED", "FAILED", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"FAILED", "WAITING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"FAILED", "RUNNING", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"FAILED", "FINISHED", http.StatusBadRequest},
|
|
Packit |
63bb0d |
{"FAILED", "FAILED", http.StatusBadRequest},
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
|
|
Packit |
63bb0d |
for _, c := range cases {
|
|
Packit |
63bb0d |
t.Log(c)
|
|
Packit |
63bb0d |
testUpdateTransition(t, c.From, c.To, c.ExpectedStatus)
|
|
Packit |
63bb0d |
}
|
|
Packit |
63bb0d |
}
|