From: Jeremy Stanley Date: Sat, 17 Apr 2021 15:59:18 +0000 (+0000) Subject: Don't leave dangling file descriptor in selftest X-Git-Tag: 0.4.0~3 X-Git-Url: https://mudpy.org/gitweb?a=commitdiff_plain;h=787d1bc42c8b9cde309709f520e254e66a23ff8d;p=mudpy.git Don't leave dangling file descriptor in selftest Use context management when opening the configuration file in selftests so that its file descriptor will be cleaned up as soon as loading completes. --- diff --git a/mudpy/tests/selftest.py b/mudpy/tests/selftest.py index c14baff..88b95c0 100644 --- a/mudpy/tests/selftest.py +++ b/mudpy/tests/selftest.py @@ -1,4 +1,4 @@ -# Copyright (c) 2004-2020 mudpy authors. Permission to use, copy, +# Copyright (c) 2004-2021 mudpy authors. Permission to use, copy, # modify, and distribute this software is granted under terms # provided in the LICENSE file distributed with this software. @@ -546,7 +546,8 @@ def option_callback(telnet_socket, command, option): def check_debug(): if len(sys.argv) > 1: - config = yaml.safe_load(open(sys.argv[1])) + with open(sys.argv[1]) as config_fd: + config = yaml.safe_load(config_fd) return config.get(".mudpy.limit.debug", False) return False