Skip to content

Instantly share code, notes, and snippets.

@logie17
Created January 20, 2016 23:40
Show Gist options
  • Save logie17/3e79f010db4c6b4c714d to your computer and use it in GitHub Desktop.
Save logie17/3e79f010db4c6b4c714d to your computer and use it in GitHub Desktop.
var logLines []string
type logMock struct{}
func (l *logMock) Write(p []byte) (int, error) {
logLines = append(logLines, string(p))
return 0, nil
}
//CaptureLogOutput A utility to capture the log output of a function and pass it along
func CaptureLogOutput(toBeCaptured func(), handleResults func([]string)) {
logLines = []string{}
currentFlags := log.Flags()
log.SetFlags(0)
log.SetOutput(&logMock{})
toBeCaptured()
log.SetOutput(os.Stderr)
log.SetFlags(currentFlags)
handleResults(logLines)
}
@belden
Copy link

belden commented Jan 21, 2016

It would also be handy if the return value of CaptureLogOutput were the return value of the func that it encloses. That would allow you to do things like:

got := CaptureLogOutput(func() rune {
    log.Print("about to return something")
    return "☃"
  },
  func(logs []string) {
    // assert something about what got logged
  })

// assert something about the return value "☃"

This may be a can of worms since the return value of CaptureLogOutput then becomes subject to the whims of the code that's being wrapped. So maybe another callback?

CaptureLogOutput(func() rune {
    log.Print("about to return something")
    return "☃"
  },
  func(logs []string) {
    // assert something about what got logged
  },
  func(frosty rune) {
    // assert frosty is "☃"
  })

Making the callback to test the return value of the wrapped function be the third argument to CaptureLogOutput feels sensible since not all code run in CaptureLogOutput will actually have a return value, so that avoids needing to pass in an empty function as a second argument.

@logie17
Copy link
Author

logie17 commented Jan 29, 2016

Great idea, all ideas implemented: https://github.com/MediaMath/auth_go_common_libs

@logie17
Copy link
Author

logie17 commented Jan 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment