From patchwork Sat Sep  1 01:55:40 2018
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [2, of,
 4] test-check-py3-compat: exclude thirdparty and cffi which are known
 to fail
From: Yuya Nishihara <yuya@tcha.org>
X-Patchwork-Id: 34226
Message-Id: <1f867d70459c469dcd78.1535766940@mimosa>
To: mercurial-devel@mercurial-scm.org
Date: Sat, 01 Sep 2018 10:55:40 +0900

# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 1535764428 -32400
#      Sat Sep 01 10:13:48 2018 +0900
# Node ID 1f867d70459c469dcd78e214b6cda41e2f2873e7
# Parent  875e635c201c1d1cf1c3f8d511ed32b791e8c419
test-check-py3-compat: exclude thirdparty and cffi which are known to fail

Otherwise the following errors would be spilled out:

  mercurial/cffi/bdiff.py: error importing: <ImportError> cannot import
  name '_bdiff' (error at bdiff.py:*)
  mercurial/cffi/bdiffbuild.py: error importing: <ModuleNotFoundError> No
  module named 'cffi' (error at bdiffbuild.py:*)
  mercurial/cffi/mpatch.py: error importing: <ImportError> cannot import
  name '_mpatch' (error at mpatch.py:*)
  mercurial/cffi/mpatchbuild.py: error importing: <ModuleNotFoundError> No
  module named 'cffi' (error at mpatchbuild.py:*)
  mercurial/cffi/osutilbuild.py: error importing: <ModuleNotFoundError> No
  module named 'cffi' (error at osutilbuild.py:*)
  mercurial/thirdparty/concurrent/futures/_base.py: invalid syntax: invalid
  syntax (<unknown>, line *)
  mercurial/thirdparty/concurrent/futures/process.py: error importing:
  <SyntaxError> invalid syntax (_base.py, line 416) (error at __init__.py:*)
  mercurial/thirdparty/concurrent/futures/thread.py: error importing:
  <SyntaxError> invalid syntax (_base.py, line 416) (error at __init__.py:*)
  mercurial/thirdparty/zope/interface/_flatten.py: error importing:
  <ImportError> cannot import name 'Declaration' (error at _flatten.py:*)

Some of them can be suppressed by building cffi modules for example, but
I don't think it's worth keeping these modules covered by the compatibility
checker.

diff --git a/tests/test-check-py3-compat.t b/tests/test-check-py3-compat.t
--- a/tests/test-check-py3-compat.t
+++ b/tests/test-check-py3-compat.t
@@ -26,7 +26,8 @@
   $ testrepohg files 'set:(**.py) - grep(pygments)' \
   > -X hgdemandimport/demandimportpy2.py \
   > -X hgext/fsmonitor/pywatchman \
-  > -X mercurial/thirdparty/cbor \
+  > -X mercurial/cffi \
+  > -X mercurial/thirdparty \
   > | sed 's|\\|/|g' | xargs python3 contrib/check-py3-compat.py \
   > | sed 's/[0-9][0-9]*)$/*)/'