From: Jeremy Stanley <fungi@yuggoth.org>
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?p=mudpy.git;a=commitdiff_plain;h=787d1bc42c8b9cde309709f520e254e66a23ff8d

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