diff options
author | Jon Dufresne <jon.dufresne@gmail.com> | 2017-10-06 08:02:05 -0400 |
---|---|---|
committer | Jon Dufresne <jon.dufresne@gmail.com> | 2017-10-11 18:48:03 -0700 |
commit | aea4d6a0dd052b3fa28e1b00c9922bf31326673c (patch) | |
tree | ea256e7191cd9b5acb287d737bd701e43982e3d5 /tests/test_31_config.py | |
parent | 847e970c5af6a09de61307be3a3d0305eee131f7 (diff) | |
download | pysaml2-aea4d6a0dd052b3fa28e1b00c9922bf31326673c.tar.gz |
Clean up unclosed files causing ResourceWarnings in tests
* Use better subprocess functions to correctly close pipes.
* When opening files, use a context manager to ensure file is closed
in a deterministic way.
* Close logging handlers
Diffstat (limited to 'tests/test_31_config.py')
-rw-r--r-- | tests/test_31_config.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tests/test_31_config.py b/tests/test_31_config.py index eb8480c6..189d3109 100644 --- a/tests/test_31_config.py +++ b/tests/test_31_config.py @@ -322,7 +322,11 @@ def test_conf_syslog(): # otherwise the logger setting is not changed root_logger.level = logging.NOTSET - root_logger.handlers = [] + while root_logger.handlers: + handler = root_logger.handlers[-1] + root_logger.removeHandler(handler) + handler.flush() + handler.close() print(c.logger) c.setup_logger() |