Patchwork [2,of,2] tests: enforce use of $PYTHON in tests

login
register
mail settings
Submitter Augie Fackler
Date June 19, 2017, 2:16 a.m.
Message ID <d12ed9ad7fd38bc40806.1497838573@augie-macbookpro2.roam.corp.google.com>
Download mbox | patch
Permalink /patch/21497/
State Accepted
Headers show

Comments

Augie Fackler - June 19, 2017, 2:16 a.m.
# HG changeset patch
# User Augie Fackler <augie@google.com>
# Date 1497550959 14400
#      Thu Jun 15 14:22:39 2017 -0400
# Node ID d12ed9ad7fd38bc40806666709228cd458e9c136
# Parent  6b14e1abca3dcb131c7765c41f30e903d72e1faa
tests: enforce use of $PYTHON in tests

Patch

diff --git a/tests/test-check-shbang.t b/tests/test-check-shbang.t
--- a/tests/test-check-shbang.t
+++ b/tests/test-check-shbang.t
@@ -5,9 +5,21 @@ 
 
 look for python scripts that do not use /usr/bin/env
 
-  $ hg files 'set:grep(r"^#!.*?python") and not grep(r"^#!/usr/bi{1}n/env python")'
+  $ hg files 'set:grep(r"^#!.*?python") and not grep(r"^#!/usr/bi{1}n/env python") - **/*.t'
   [1]
 
+In tests, enforce $PYTHON and *not* /usr/bin/env python or similar:
+  $ hg files 'set:grep(r"#!.*?python") and **/*.t' \
+  > -X tests/test-check-execute.t \
+  > -X tests/test-check-module-imports.t \
+  > -X tests/test-check-pyflakes.t \
+  > -X tests/test-check-shbang.t
+  [1]
+
+The above exclusions are because they're looking for files that
+contain Python but don't end in .py - please avoid adding more.
+
+
 look for shell scripts that do not use /bin/sh
 
   $ hg files 'set:grep(r"^#!.*/bi{1}n/sh") and not grep(r"^#!/bi{1}n/sh")'