Differential D529

test suite: use full log formatter format
ClosedPublic
Actions

Authored by kparal on Aug 25 2015, 2:07 PM.

Details

Reviewers
jskladan
Group Reviewers
libtaskotron
Commits
rLTRNabf7af1350d1: test suite: use full log formatter format
Summary

We used a simplified log format while running the test suite, which made
debugging harder. Ensure we always have full format.

Test Plan

py.test -s testing/test_check.py before and after this patch (make sure you don't run test_logger.py before that, it affects it)

Diff Detail

Repository
rLTRN libtaskotron
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kparal retitled this revision from to test suite: use full log formatter format.Aug 25 2015, 2:07 PM
kparal updated this object.
kparal edited the test plan for this revision. (Show Details)
kparal added a reviewer: libtaskotron.
jskladan requested changes to this revision.Aug 25 2015, 2:08 PM
jskladan added a reviewer: jskladan.
This revision now requires changes to proceed.Aug 25 2015, 2:08 PM
jskladan accepted this revision.Aug 25 2015, 2:08 PM
This revision is now accepted and ready to land.Aug 25 2015, 2:08 PM
Closed by commit rLTRNabf7af1350d1: test suite: use full log formatter format (authored by kparal). · Explain WhyAug 25 2015, 2:10 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathPackages
Mtesting/conftest.py (2 lines)

Diff 1384

View Options

testing/conftest.py

Show First 20 Lines • Show All 46 Lines • ▼ Show 20 Line(s)39def pytest_configure(config):
47 # enable debug logging for taskotron 47​ # enable debug logging for taskotron
48 # This will allow us to use libtaskotron-logged output for failed tests. 48​ # This will allow us to use libtaskotron-logged output for failed tests.
49 # You can also use "py.test -s" option to see all output even for passed 49​ # You can also use "py.test -s" option to see all output even for passed
50 # tests (of course, it's better to run the suite with just a single file 50​ # tests (of course, it's better to run the suite with just a single file
51 # rather than the whole directory in this case) 51​ # rather than the whole directory in this case)
52 libtaskotron.logger.init_prior_config() 52​ libtaskotron.logger.init_prior_config()
53 # and this will show debug messages from many other libraries as well 53​ # and this will show debug messages from many other libraries as well
54 logging.getLogger().setLevel(logging.NOTSET) 54​ logging.getLogger().setLevel(logging.NOTSET)
55​ # always use full formatter format
56​ libtaskotron.logger.stream_handler.setFormatter(libtaskotron.logger._formatter_full)