Browse Source

Fix race in integration testlogger (#6556)

* Fix race in logger

* At testing end drop the reference to the last test.
tags/v1.21.12.1
zeripath Lunny Xiao 6 years ago
parent
commit
c02c6a144f
2 changed files with 23 additions and 0 deletions
  1. +2
    -0
      integrations/integration_test.go
  2. +21
    -0
      integrations/testlogger.go

+ 2
- 0
integrations/integration_test.go View File

@@ -81,6 +81,8 @@ func TestMain(m *testing.M) {
}
exitCode := m.Run()

writerCloser.t = nil

if err = os.RemoveAll(setting.Indexer.IssuePath); err != nil {
fmt.Printf("os.RemoveAll: %v\n", err)
os.Exit(1)


+ 21
- 0
integrations/testlogger.go View File

@@ -33,6 +33,27 @@ func (w *testLoggerWriterCloser) Write(p []byte) (int, error) {
if len(p) > 0 && p[len(p)-1] == '\n' {
p = p[:len(p)-1]
}

defer func() {
err := recover()
if err == nil {
return
}
var errString string
errErr, ok := err.(error)
if ok {
errString = errErr.Error()
} else {
errString, ok = err.(string)
}
if !ok {
panic(err)
}
if !strings.HasPrefix(errString, "Log in goroutine after ") {
panic(err)
}
}()

w.t.Log(string(p))
return len(p), nil
}


Loading…
Cancel
Save