Patchwork D7371: tests: add optional output when wheel is missing

login
register
mail settings
Submitter phabricator
Date Nov. 12, 2019, 3:05 a.m.
Message ID <differential-rev-PHID-DREV-vgsqflh3uv2zmueqxwid-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/43142/
State Superseded
Headers show

Comments

phabricator - Nov. 12, 2019, 3:05 a.m.
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  If the `wheel` package isn't installed, pip will emit a warning
  on stderr complaining about that. Alternatively, we could
  redirect stderr to stdout and swallow the warning. But I like
  tests having visibility of warnings, as it helps keep us honest
  regarding edge cases.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D7371

AFFECTED FILES
  tests/test-install.t

CHANGE DETAILS




To: indygreg, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/tests/test-install.t b/tests/test-install.t
--- a/tests/test-install.t
+++ b/tests/test-install.t
@@ -255,6 +255,7 @@ 
 Note: we use this weird path to run pip and hg to avoid platform differences,
 since it's bin on most platforms but Scripts on Windows.
   $ ./installenv/*/pip install --no-index $TESTDIR/.. >> pip.log
+    Failed building wheel for mercurial (?)
   $ ./installenv/*/hg debuginstall || cat pip.log
   checking encoding (ascii)...
   checking Python executable (*) (glob)