forked from go-gitea/gitea
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
migrations: add test for importing pull requests in gitea uploader (g…
…o-gitea#18752) * logs: add the buffer logger to inspect logs during testing Signed-off-by: Loïc Dachary <loic@dachary.org> * migrations: add test for importing pull requests in gitea uploader Signed-off-by: Loïc Dachary <loic@dachary.org> * for each git.OpenRepositoryCtx, call Close * Content is expected to return the content of the log * test for errors before defer Co-authored-by: Loïc Dachary <loic@dachary.org> Co-authored-by: zeripath <art27@cantab.net> Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
- Loading branch information
1 parent
fdab87e
commit 4dc95d0
Showing
12 changed files
with
512 additions
and
20 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,72 @@ | ||
// Copyright 2022 The Gitea Authors. All rights reserved. | ||
// Use of this source code is governed by a MIT-style | ||
// license that can be found in the LICENSE file. | ||
|
||
package log | ||
|
||
import ( | ||
"bytes" | ||
"sync" | ||
) | ||
|
||
type bufferWriteCloser struct { | ||
mu sync.Mutex | ||
buffer bytes.Buffer | ||
} | ||
|
||
func (b *bufferWriteCloser) Write(p []byte) (int, error) { | ||
b.mu.Lock() | ||
defer b.mu.Unlock() | ||
return b.buffer.Write(p) | ||
} | ||
|
||
func (b *bufferWriteCloser) Close() error { | ||
return nil | ||
} | ||
|
||
func (b *bufferWriteCloser) String() string { | ||
b.mu.Lock() | ||
defer b.mu.Unlock() | ||
return b.buffer.String() | ||
} | ||
|
||
// BufferLogger implements LoggerProvider and writes messages in a buffer. | ||
type BufferLogger struct { | ||
WriterLogger | ||
} | ||
|
||
// NewBufferLogger create BufferLogger returning as LoggerProvider. | ||
func NewBufferLogger() LoggerProvider { | ||
log := &BufferLogger{} | ||
log.NewWriterLogger(&bufferWriteCloser{}) | ||
return log | ||
} | ||
|
||
// Init inits connection writer | ||
func (log *BufferLogger) Init(string) error { | ||
log.NewWriterLogger(log.out) | ||
return nil | ||
} | ||
|
||
// Content returns the content accumulated in the content provider | ||
func (log *BufferLogger) Content() (string, error) { | ||
return log.out.(*bufferWriteCloser).String(), nil | ||
} | ||
|
||
// Flush when log should be flushed | ||
func (log *BufferLogger) Flush() { | ||
} | ||
|
||
// ReleaseReopen does nothing | ||
func (log *BufferLogger) ReleaseReopen() error { | ||
return nil | ||
} | ||
|
||
// GetName returns the default name for this implementation | ||
func (log *BufferLogger) GetName() string { | ||
return "buffer" | ||
} | ||
|
||
func init() { | ||
Register("buffer", NewBufferLogger) | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,64 @@ | ||
// Copyright 2022 The Gitea Authors. All rights reserved. | ||
// Use of this source code is governed by a MIT-style | ||
// license that can be found in the LICENSE file. | ||
|
||
package log | ||
|
||
import ( | ||
"fmt" | ||
"strings" | ||
"testing" | ||
"time" | ||
|
||
"github.com/stretchr/testify/assert" | ||
) | ||
|
||
func TestBufferLogger(t *testing.T) { | ||
logger := NewBufferLogger() | ||
bufferLogger := logger.(*BufferLogger) | ||
assert.NotNil(t, bufferLogger) | ||
|
||
err := logger.Init("") | ||
assert.NoError(t, err) | ||
|
||
location, _ := time.LoadLocation("EST") | ||
date := time.Date(2019, time.January, 13, 22, 3, 30, 15, location) | ||
|
||
msg := "TEST MSG" | ||
event := Event{ | ||
level: INFO, | ||
msg: msg, | ||
caller: "CALLER", | ||
filename: "FULL/FILENAME", | ||
line: 1, | ||
time: date, | ||
} | ||
logger.LogEvent(&event) | ||
content, err := bufferLogger.Content() | ||
assert.NoError(t, err) | ||
assert.Contains(t, content, msg) | ||
logger.Close() | ||
} | ||
|
||
func TestBufferLoggerContent(t *testing.T) { | ||
level := INFO | ||
logger := NewLogger(0, "console", "console", fmt.Sprintf(`{"level":"%s"}`, level.String())) | ||
|
||
logger.SetLogger("buffer", "buffer", "{}") | ||
defer logger.DelLogger("buffer") | ||
|
||
msg := "A UNIQUE MESSAGE" | ||
Error(msg) | ||
|
||
found := false | ||
for i := 0; i < 30000; i++ { | ||
content, err := logger.GetLoggerProviderContent("buffer") | ||
assert.NoError(t, err) | ||
if strings.Contains(content, msg) { | ||
found = true | ||
break | ||
} | ||
time.Sleep(1 * time.Millisecond) | ||
} | ||
assert.True(t, found) | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.