Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ab9d8d6

Browse files
committed
Escape file path before searching for it in output via regex
1 parent 7d10129 commit ab9d8d6

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

Lib/test/test_site.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import builtins
1111
import os
1212
import sys
13+
import re
1314
import encodings
1415
import subprocess
1516
import sysconfig
@@ -108,7 +109,8 @@ def test_addpackage_import_bad_syntax(self):
108109
with captured_stderr() as err_out:
109110
site.addpackage(pth_dir, pth_fn, set())
110111
self.assertRegex(err_out.getvalue(), "line 1")
111-
self.assertRegex(err_out.getvalue(), os.path.join(pth_dir, pth_fn))
112+
self.assertRegex(err_out.getvalue(),
113+
re.escape(os.path.join(pth_dir, pth_fn)))
112114
# XXX: the previous two should be independent checks so that the
113115
# order doesn't matter. The next three could be a single check
114116
# but my regex foo isn't good enough to write it.
@@ -122,7 +124,8 @@ def test_addpackage_import_bad_exec(self):
122124
with captured_stderr() as err_out:
123125
site.addpackage(pth_dir, pth_fn, set())
124126
self.assertRegex(err_out.getvalue(), "line 2")
125-
self.assertRegex(err_out.getvalue(), os.path.join(pth_dir, pth_fn))
127+
self.assertRegex(err_out.getvalue(),
128+
re.escape(os.path.join(pth_dir, pth_fn)))
126129
# XXX: ditto previous XXX comment.
127130
self.assertRegex(err_out.getvalue(), 'Traceback')
128131
self.assertRegex(err_out.getvalue(), 'ImportError')
@@ -133,7 +136,8 @@ def test_addpackage_import_bad_pth_file(self):
133136
with captured_stderr() as err_out:
134137
site.addpackage(pth_dir, pth_fn, set())
135138
self.assertRegex(err_out.getvalue(), "line 1")
136-
self.assertRegex(err_out.getvalue(), os.path.join(pth_dir, pth_fn))
139+
self.assertRegex(err_out.getvalue(),
140+
re.escape(os.path.join(pth_dir, pth_fn)))
137141
# XXX: ditto previous XXX comment.
138142
self.assertRegex(err_out.getvalue(), 'Traceback')
139143
self.assertRegex(err_out.getvalue(), 'TypeError')

0 commit comments

Comments
 (0)