Fix ResourceWarning when running tests

supervisor/supervisor/http.py:581: ResourceWarning: unclosed <socket.socket fd=14, family=AddressFamily.AF_UNIX, type=SocketKind.SOCK_STREAM, proto=0>
  used = self.checkused(socketname)
ResourceWarning: Enable tracemalloc to get the object allocation traceback
Unlinking stale socket /var/folders/4l/30_vf63152bflw8hnvyvnw_00000gn/T/tmpbpxj02zu

In 1849019, uses of tempfile.mktemp() were replaced
with tempfile.NamedTemporaryFile().  Some tests did not run as intended afterward
because they expected the file to not exist on disk.

Author: mnaberez

supervisor/tests/test_http.py

@@ -620,8 +620,10 @@ def test_make_http_servers_socket_type_error(self):
             self.assertEqual(exc.args[0], 'Cannot determine socket type 999')
 
     def test_make_http_servers_noauth(self):
-        with tempfile.NamedTemporaryFile(delete=False) as f:
+        with tempfile.NamedTemporaryFile(delete=True) as f:
             socketfile = f.name
+        self.assertFalse(os.path.exists(socketfile))
+
         inet = {'family':socket.AF_INET, 'host':'localhost', 'port':17735,
                 'username':None, 'password':None, 'section':'inet_http_server'}
         unix = {'family':socket.AF_UNIX, 'file':socketfile, 'chmod':0o700,
@@ -648,8 +650,10 @@ def test_make_http_servers_noauth(self):
         self.assertEqual([x.IDENT for x in server.handlers], idents)
 
     def test_make_http_servers_withauth(self):
-        with tempfile.NamedTemporaryFile(delete=False) as f:
+        with tempfile.NamedTemporaryFile(delete=True) as f:
             socketfile = f.name
+        self.assertFalse(os.path.exists(socketfile))
+
         inet = {'family':socket.AF_INET, 'host':'localhost', 'port':17736,
                 'username':'username', 'password':'password',
                 'section':'inet_http_server'}

supervisor/tests/test_options.py

@@ -1641,8 +1641,10 @@ def test_reopenlogs(self):
         self.assertEqual(logger.data[0], 'supervisord logreopen')
 
     def test_write_pidfile_ok(self):
-        with tempfile.NamedTemporaryFile(delete=False) as f:
+        with tempfile.NamedTemporaryFile(delete=True) as f:
             fn = f.name
+        self.assertFalse(os.path.exists(fn))
+
         try:
             instance = self._makeOne()
             instance.logger = DummyLogger()