mirror of
https://github.com/amir20/dozzle.git
synced 2026-01-03 11:35:00 +01:00
4c5bd9a7be
* Reverts back previous fix and uses proper error handeling * Adds more tests and refactors existing tests
224 lines
7.4 KiB
Go
224 lines
7.4 KiB
Go
package web
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"io"
|
|
"time"
|
|
|
|
"io/ioutil"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"strings"
|
|
"testing"
|
|
|
|
"github.com/amir20/dozzle/docker"
|
|
"github.com/beme/abide"
|
|
"github.com/stretchr/testify/mock"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func Test_handler_streamLogs_happy(t *testing.T) {
|
|
id := "123456"
|
|
req, err := http.NewRequest("GET", "/api/logs/stream", nil)
|
|
q := req.URL.Query()
|
|
q.Add("id", id)
|
|
req.URL.RawQuery = q.Encode()
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
|
|
mockedClient := new(MockedClient)
|
|
reader := ioutil.NopCloser(strings.NewReader("INFO Testing logs..."))
|
|
mockedClient.On("FindContainer", id).Return(docker.Container{ID: id}, nil)
|
|
mockedClient.On("ContainerLogs", mock.Anything, mock.Anything, "").Return(reader, nil)
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamLogs)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
func Test_handler_streamLogs_happy_with_id(t *testing.T) {
|
|
id := "123456"
|
|
req, err := http.NewRequest("GET", "/api/logs/stream", nil)
|
|
q := req.URL.Query()
|
|
q.Add("id", id)
|
|
req.URL.RawQuery = q.Encode()
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
|
|
mockedClient := new(MockedClient)
|
|
reader := ioutil.NopCloser(strings.NewReader("2020-05-13T18:55:37.772853839Z INFO Testing logs..."))
|
|
mockedClient.On("FindContainer", id).Return(docker.Container{ID: id}, nil)
|
|
mockedClient.On("ContainerLogs", mock.Anything, mock.Anything, "").Return(reader, nil)
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamLogs)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
func Test_handler_streamLogs_happy_container_stopped(t *testing.T) {
|
|
id := "123456"
|
|
req, err := http.NewRequest("GET", "/api/logs/stream", nil)
|
|
q := req.URL.Query()
|
|
q.Add("id", id)
|
|
req.URL.RawQuery = q.Encode()
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
|
|
mockedClient := new(MockedClient)
|
|
mockedClient.On("FindContainer", id).Return(docker.Container{ID: id}, nil)
|
|
mockedClient.On("ContainerLogs", mock.Anything, id, "").Return(ioutil.NopCloser(strings.NewReader("")), io.EOF)
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamLogs)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
func Test_handler_streamLogs_error_finding_container(t *testing.T) {
|
|
id := "123456"
|
|
req, err := http.NewRequest("GET", "/api/logs/stream", nil)
|
|
q := req.URL.Query()
|
|
q.Add("id", id)
|
|
req.URL.RawQuery = q.Encode()
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
|
|
mockedClient := new(MockedClient)
|
|
mockedClient.On("FindContainer", id).Return(docker.Container{}, errors.New("error finding container"))
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamLogs)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
func Test_handler_streamLogs_error_reading(t *testing.T) {
|
|
id := "123456"
|
|
req, err := http.NewRequest("GET", "/api/logs/stream", nil)
|
|
q := req.URL.Query()
|
|
q.Add("id", id)
|
|
req.URL.RawQuery = q.Encode()
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
|
|
mockedClient := new(MockedClient)
|
|
mockedClient.On("FindContainer", id).Return(docker.Container{ID: id}, nil)
|
|
mockedClient.On("ContainerLogs", mock.Anything, id, "").Return(ioutil.NopCloser(strings.NewReader("")), errors.New("test error"))
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamLogs)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
func Test_handler_streamEvents_happy(t *testing.T) {
|
|
req, err := http.NewRequest("GET", "/api/events/stream", nil)
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
mockedClient := new(MockedClient)
|
|
messages := make(chan docker.ContainerEvent)
|
|
errChannel := make(chan error)
|
|
mockedClient.On("Events", mock.Anything).Return(messages, errChannel)
|
|
mockedClient.On("ListContainers").Return([]docker.Container{}, nil)
|
|
|
|
go func() {
|
|
messages <- docker.ContainerEvent{
|
|
Name: "start",
|
|
ActorID: "1234",
|
|
}
|
|
messages <- docker.ContainerEvent{
|
|
Name: "something-random",
|
|
ActorID: "1234",
|
|
}
|
|
close(messages)
|
|
}()
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamEvents)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
func Test_handler_streamEvents_error(t *testing.T) {
|
|
req, err := http.NewRequest("GET", "/api/events/stream", nil)
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
mockedClient := new(MockedClient)
|
|
messages := make(chan docker.ContainerEvent)
|
|
errChannel := make(chan error)
|
|
mockedClient.On("Events", mock.Anything).Return(messages, errChannel)
|
|
mockedClient.On("ListContainers").Return([]docker.Container{}, nil)
|
|
|
|
go func() {
|
|
errChannel <- errors.New("fake error")
|
|
close(messages)
|
|
}()
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamEvents)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
func Test_handler_streamEvents_error_request(t *testing.T) {
|
|
req, err := http.NewRequest("GET", "/api/events/stream", nil)
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
|
|
mockedClient := new(MockedClient)
|
|
|
|
messages := make(chan docker.ContainerEvent)
|
|
errChannel := make(chan error)
|
|
mockedClient.On("Events", mock.Anything).Return(messages, errChannel)
|
|
mockedClient.On("ListContainers").Return([]docker.Container{}, nil)
|
|
|
|
ctx, cancel := context.WithCancel(context.Background())
|
|
req = req.WithContext(ctx)
|
|
|
|
go func() {
|
|
cancel()
|
|
}()
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.streamEvents)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|
|
|
|
// for /api/logs
|
|
func Test_handler_between_dates(t *testing.T) {
|
|
req, err := http.NewRequest("GET", "/api/logs", nil)
|
|
require.NoError(t, err, "NewRequest should not return an error.")
|
|
|
|
from, _ := time.Parse(time.RFC3339, "2018-01-01T00:00:00Z")
|
|
to, _ := time.Parse(time.RFC3339, "2018-01-01T010:00:00Z")
|
|
|
|
q := req.URL.Query()
|
|
q.Add("from", from.Format(time.RFC3339))
|
|
q.Add("to", to.Format(time.RFC3339))
|
|
q.Add("id", "123456")
|
|
req.URL.RawQuery = q.Encode()
|
|
|
|
mockedClient := new(MockedClient)
|
|
reader := ioutil.NopCloser(strings.NewReader("2020-05-13T18:55:37.772853839Z INFO Testing logs...\n2020-05-13T18:55:37.772853839Z INFO Testing logs...\n"))
|
|
mockedClient.On("ContainerLogsBetweenDates", mock.Anything, "123456", from, to).Return(reader, nil)
|
|
|
|
h := handler{client: mockedClient, config: &Config{}}
|
|
handler := http.HandlerFunc(h.fetchLogsBetweenDates)
|
|
rr := httptest.NewRecorder()
|
|
handler.ServeHTTP(rr, req)
|
|
abide.AssertHTTPResponse(t, t.Name(), rr.Result())
|
|
mockedClient.AssertExpectations(t)
|
|
}
|