Patchwork [2,of,2] tests: adapt test-check-pylint to run on Windows

login
register
mail settings
Submitter Matt Harbison
Date June 1, 2018, 2:35 a.m.
Message ID <7164b44b0dbef0923363.1527820501@Envy>
Download mbox | patch
Permalink /patch/31923/
State Accepted
Headers show

Comments

Matt Harbison - June 1, 2018, 2:35 a.m.
# HG changeset patch
# User Matt Harbison <matt_harbison@yahoo.com>
# Date 1527819352 14400
#      Thu May 31 22:15:52 2018 -0400
# Node ID 7164b44b0dbef0923363f9f3f493bdfd37592e27
# Parent  0de423ecd560edaa0f1eb25cc265c3b2e8caf0f0
tests: adapt test-check-pylint to run on Windows

The line endings are explicitly converted because this was ending up with
'\r (no-eol) (esc)' lines, in addition to the usual '\r (esc)' lines.  I've seen
the fakerc output on other recently installed systems though (10.13 and/or
Fedora 26).  Unfortunately, the output here uses '\\' on Windows, so glob away
the whole path.

Patch

diff --git a/tests/test-check-pylint.t b/tests/test-check-pylint.t
--- a/tests/test-check-pylint.t
+++ b/tests/test-check-pylint.t
@@ -14,7 +14,8 @@  Current checks:
   >   --enable=W0102,C0321 \
   >   --reports=no \
   >   --ignore=thirdparty \
-  >   mercurial hgdemandimport hgext hgext3rd
+  >   mercurial hgdemandimport hgext hgext3rd | sed 's/\r$//'
+  Using config file *fakerc (glob) (?)
    (?)
   ------------------------------------ (?)
   Your code has been rated at 10.00/10 (?)