From 488102039cb5e79114954ad91663ce28c99153c8 Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Sat, 7 Mar 2020 00:46:26 -0800 Subject: Log internal HTTP requests This restores the previous behavior of logging the success and failures of internal HTTP requests. Part of https://gitlab.com/gitlab-org/gitlab/issues/207916 --- internal/logger/logger_test.go | 42 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 42 insertions(+) create mode 100644 internal/logger/logger_test.go (limited to 'internal/logger/logger_test.go') diff --git a/internal/logger/logger_test.go b/internal/logger/logger_test.go new file mode 100644 index 0000000..e14d01c --- /dev/null +++ b/internal/logger/logger_test.go @@ -0,0 +1,42 @@ +package logger + +import ( + "io/ioutil" + "os" + "strings" + "testing" + "time" + + log "github.com/sirupsen/logrus" + "github.com/stretchr/testify/require" + "gitlab.com/gitlab-org/gitlab-shell/internal/config" +) + +func TestConfigure(t *testing.T) { + tmpFile, err := ioutil.TempFile(os.TempDir(), "logtest-") + require.NoError(t, err) + defer tmpFile.Close() + + config := config.Config{ + LogFile: tmpFile.Name(), + LogFormat: "json", + } + + err = Configure(&config) + + require.NoError(t, err) + + log.Info("this is a test") + + tmpFile.Close() + + data, err := ioutil.ReadFile(tmpFile.Name()) + require.NoError(t, err) + require.True(t, strings.Contains(string(data), `msg":"this is a test"`)) +} + +func TestElapsedTime(t *testing.T) { + start := time.Now() + start = start.Add(1234567800 * time.Nanosecond) + require.InDelta(t, -1234.568, ElapsedTime(start), 0.01) +} -- cgit v1.2.1