Mercurial > public > mercurial-scm > hg-stable
diff tests/run-tests.py @ 49314:2e726c934fcd
py3: catch FileNotFoundError instead of checking errno == ENOENT
author | Manuel Jacob <me@manueljacob.de> |
---|---|
date | Tue, 31 May 2022 22:50:01 +0200 |
parents | 53e9422a9b45 |
children | 35bf7f23b84c |
line wrap: on
line diff
--- a/tests/run-tests.py Tue May 31 21:16:17 2022 +0200 +++ b/tests/run-tests.py Tue May 31 22:50:01 2022 +0200 @@ -362,9 +362,7 @@ try: path = os.path.expanduser(os.path.expandvars(filename)) f = open(path, "rb") - except IOError as err: - if err.errno != errno.ENOENT: - raise + except FileNotFoundError: if warn: print("warning: no such %s file: %s" % (listtype, filename)) continue @@ -390,9 +388,8 @@ for l in f: if l.startswith(b'#testcases '): cases.append(sorted(l[11:].split())) - except IOError as ex: - if ex.errno != errno.ENOENT: - raise + except FileNotFoundError: + pass return cases @@ -1110,12 +1107,11 @@ if os.path.exists(self.errpath): try: os.remove(self.errpath) - except OSError as e: - # We might have raced another test to clean up a .err - # file, so ignore ENOENT when removing a previous .err + except FileNotFoundError: + # We might have raced another test to clean up a .err file, + # so ignore FileNotFoundError when removing a previous .err # file. - if e.errno != errno.ENOENT: - raise + pass if self._usechg: self._chgsockdir = os.path.join( @@ -2622,9 +2618,8 @@ times.append( (m.group(1), [float(t) for t in m.group(2).split()]) ) - except IOError as err: - if err.errno != errno.ENOENT: - raise + except FileNotFoundError: + pass return times @@ -2979,9 +2974,7 @@ except KeyError: try: val = -os.stat(f).st_size - except OSError as e: - if e.errno != errno.ENOENT: - raise + except FileNotFoundError: perf[f] = -1e9 # file does not exist, tell early return -1e9 for kw, mul in slow.items(): @@ -3584,9 +3577,8 @@ if os.readlink(mypython) == sysexecutable: continue os.unlink(mypython) - except OSError as err: - if err.errno != errno.ENOENT: - raise + except FileNotFoundError: + pass if self._findprogram(pyexename) != sysexecutable: try: os.symlink(sysexecutable, mypython) @@ -3714,9 +3706,8 @@ if not self.options.verbose: try: os.remove(installerrs) - except OSError as e: - if e.errno != errno.ENOENT: - raise + except FileNotFoundError: + pass else: with open(installerrs, 'rb') as f: for line in f: