675376 - Add better support for logging in test support

Status: RESOLVED FIXED

(Cloud Services :: Server: Core, defect)

Description

[Toby Elliott [:telliott]]

It would be nice to have a way to easily make logging seamless in tests, particularly those that use WebTest (so are talking to an "independent" server. Right now, they aren't in the app config (and tarek is correct that it's not really an app function). Even something as simple as initlog(product, file) at the start of the tests might be sufficient, though there's a lot we could do past that.

Comment 1

[Tarek Ziadé (:tarek)]

Attachment: context manager to catch logs

Here's the log catcher

the usage is very simple:

  with catch_logs() as logs:
      ..your test..
   
  logs = logs.read()

by default it catches the syncserver logger at the error level, but you can catch any level, and any logger -- see the tests

This supersede the need to set up the logger via a config file or programmatically

Comment 2

[Toby Elliott [:telliott]]

Comment on attachment 550992 [details] [diff] [review]
context manager to catch logs

Looks good to me, though I'll defer to rmiller as to how it fits into the stuff he's working on.

Comment 3

[Tarek Ziadé (:tarek)]

https://hg.mozilla.org/services/server-core/rev/2c1465171bf8