From: Senthil Kumaran <senthil@uthcode.com>
Date: Sat, 4 Jan 2020 02:14:18 +0000 (-0800)
Subject: bpo-27973 - Use test.support.temp_dir instead of NamedTemporaryFile for the (#17774)
X-Git-Tag: v2.7.18rc1~6
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5bba60290b4ac8c95ac46cfdaba5deee37be1fab;p=thirdparty%2FPython%2Fcpython.git

bpo-27973 - Use test.support.temp_dir instead of NamedTemporaryFile for the (#17774)

desired behavior under windows platform.

Suggestion by David Bolen
---

diff --git a/Lib/test/test_urllibnet.py b/Lib/test/test_urllibnet.py
index df118dc75d1b..ef33e3a0ea61 100644
--- a/Lib/test/test_urllibnet.py
+++ b/Lib/test/test_urllibnet.py
@@ -1,4 +1,3 @@
-import tempfile
 import unittest
 from test import test_support
 from test.test_urllib2net import skip_ftp_test_on_travis
@@ -224,9 +223,10 @@ class urlopen_FTPTest(unittest.TestCase):
 
         with test_support.transient_internet(self.FTP_TEST_FILE):
             try:
-                for _ in range(self.NUM_FTP_RETRIEVES):
-                    with tempfile.NamedTemporaryFile() as fp:
-                        urllib.FancyURLopener().retrieve(self.FTP_TEST_FILE, fp.name)
+                for file_num in range(self.NUM_FTP_RETRIEVES):
+                    with test_support.temp_dir() as td:
+                        urllib.FancyURLopener().retrieve(self.FTP_TEST_FILE,
+                                                         os.path.join(td, str(file_num)))
             except IOError as e:
                 self.fail("Failed FTP retrieve while accessing ftp url "
                           "multiple times.\n Error message was : %s" % e)