From patchwork Mon May  2 12:28:46 2022
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: D12598: tests: make sure .js files stay in ASCII encoding (issue6559)
From: phabricator <phabricator@mercurial-scm.org>
X-Patchwork-Id: 50947
Message-Id: <differential-rev-PHID-DREV-j7aihilxmjaluo2fchye-req@mercurial-scm.org>
To: Phabricator <phabricator@mercurial-scm.org>
Cc: mercurial-devel@mercurial-scm.org
Date: Mon, 2 May 2022 12:28:46 +0000

av6 created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REPOSITORY
  rHG Mercurial

BRANCH
  default

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

AFFECTED FILES
  tests/test-check-encoding.t

CHANGE DETAILS




To: av6, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

diff --git a/tests/test-check-encoding.t b/tests/test-check-encoding.t
new file mode 100644
--- /dev/null
+++ b/tests/test-check-encoding.t
@@ -0,0 +1,17 @@
+#require test-repo hg10
+
+  $ . "$TESTDIR/helpers-testrepo.sh"
+
+There are some web servers in the wild that can serve static files with an
+incorrect encoding (e.g. https://bz.mercurial-scm.org/show_bug.cgi?id=6559).
+One way to prevent any issues is to not use any non-ASCII characters, e.g.
+URL-encoding them or using HTML entities.
+
+check charset of all tracked files ending in .js except vendored dependencies
+
+  $ cd "`dirname "$TESTDIR"`"
+
+  $ testrepohg locate 'set:**.js' \
+  > 2>/dev/null \
+  > | xargs file -i \
+  > | grep -v 'charset=us-ascii'