- Dec 11, 2017
-
-
Matt Harbison authored
# skip-blame because this was mechanically rewritten the following script. I ran it on both *.t and *.py, but none of the *.py changes were proper. All *.t ones appear to be, and they run without addition failures on both Windows and Linux. import argparse import os import re ap = argparse.ArgumentParser() ap.add_argument('path', nargs='+') opts = ap.parse_args() globre = re.compile(r'^(.*) \(glob\)(.*)$') for p in opts.path: tmp = p + '.tmp' with open(p, 'rb') as src, open(tmp, 'wb') as dst: for line in src: m = globre.match(line) if not m or '$LOCALIP' in line or '*' in line: dst.write(line) continue if '?' in line[:-3] or ('?' in line[:-3] and line[-3:] != '(?)'): dst.write(line) continue dst.write(m.group(1) + m.group(2) + '\n') os.unlink(p) os.rename(tmp, p)
-
Matt Harbison authored
I'll probably resurrect at least some of these as replacement patterns to switch '\' to '/' when creating tests at some point. But since this subset of globs isn't needed anymore after dfae14354660, and the rules are blocking the glob removal, just drop them for now. It probably isn't worth turning them into checks for unnecessary globs being present.
-
Matt Harbison authored
This is the first step to dropping the existing globs for '\' matches, now that it is handled automatically. Instead of just dropping it, this pattern is now used to convert to '/' paths, to reduce the amount of manual cleanup required when creating tests on Windows.
-
- Dec 07, 2017
-
-
Anton Shestakov authored
It's essentially a copy of json-log with graph-related things added (col, row, color, edges).
-
- Dec 12, 2017
-
-
Phil Cohen authored
Differential Revision: https://phab.mercurial-scm.org/D1666
-
- Dec 11, 2017
-
-
Augie Fackler authored
Hopefully this will give us a shot at fixing it on the Windows builder. Differential Revision: https://phab.mercurial-scm.org/D1653
-
Phil Cohen authored
Commit hashes are a useful way to ensure the content of commits made in the tests are not changing, even if we don't query every aspect of every commit. (And some properties, like extras, are rarely printed at all.) Many of the rebase log -G calls didn't show hashes; by adding hashes to places that weren't showing them we can help protect those tests from unwanted changes. Differential Revision: https://phab.mercurial-scm.org/D1650
-
Phil Cohen authored
This is just a very simple start, but verifies some of the basic cases of an in-memory rebase. Differential Revision: https://phab.mercurial-scm.org/D1652
-
Yuya Nishihara authored
This changes {\W} to {\W - any 8bit characters} so that multibyte sequences are taken as words. Since we don't know the encoding of user content, this is the most sensible definition of a non-word.
-
- Dec 10, 2017
-
-
Matt Harbison authored
Having to constantly adjust these is a hassle. It's easy for this to slip by when not testing on Windows, and then when it happens on stable, the tests fail for the next 3 months if we follow the rules for stable. This works the same way the EOL differences are ignored, namely to adjust on the fly and recheck on Windows. I can't think of any situation where there would be a '\' on Windows, a '/' elsewhere, and the '/' should be considered a failure on Windows. This fixes the obvious output problems where (glob) is missing. Without this, test-alias.t, test-remotenames.t and test-largefiles-misc.t are failing. The flip side (not handled by this) is the case where an unnecessary glob is present. There seems to be two separate behaviors. cf300c1ad7bf is an example of where the test has been autocorrecting (with output differences), and d4ec69ff652a is an example where the test fails and reports 'no result code from test'. Hopefully those cases will become even more rare if people don't need to guess at when a glob is needed for a Windows path. It's probably unreasonable to submit a single patch that wipes out all of the (glob) instances that were only used to hide path differences, given the churn from other contributors. Since their presence isn't harming the tests, these can be removed through attrition.
-
Matt Harbison authored
We already do this for lines ending in '\n', such that the test only needs to be run with --interactive and the changes accepted at the end. But that wasn't working with list-tree.py output for example, and required manual fixup.
-
Matt Harbison authored
The test-largefiles-misc.t test was moving 'dir2\' before 'dir\' because while '/' precedes most of the printable ASCII characters, '\' comes after numbers and capital letters, among other symbols.
-
Yuya Nishihara authored
Copied from commandserver.py.
-
Yuya Nishihara authored
-
Yuya Nishihara authored
-
durin42 authored
The shell construct here appears to be unevenly supported: it works in /bin/sh on FreeBSD, but it doesn't seem to work when /bin/sh is dash. Using a Python inline directive works fine, so let's just do that instead. Differential Revision: https://phab.mercurial-scm.org/D1636
-
Pulkit Goyal authored
Yay, we have crossed 100 in number of tests passing on Python 3. There are 662 tests in our test suite, so there is a lot more which is need to be done. Differential Revision: https://phab.mercurial-scm.org/D1648
-
- Dec 09, 2017
-
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1647
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1646
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1645
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1644
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1643
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1642
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1641
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1640
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1639
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1638
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1637
-
- Dec 08, 2017
-
-
Matt Harbison authored
This also ends up testing the local extension wrapping for dstrepo during upgrade, which was fixed in 06987c6971be.
-
Matt Harbison authored
In order to fix the missing lfs store after an upgrade, I attempted to walk the store vfs to hardlink to the upgraded repo's store. But the custom join() clashes with the default walk() implementation. First, 'path=None' blew up in the regex matcher, because it wanted a string. But even if that is fixed, the join to walk the root of the vfs wouldn't match the required xx/xx...xx pattern. The first cut of this was a copy/paste/tweak of the base implementation, but this version of walk() hides the internal directories, and treats the vfs as a flat store. I think this makes sense because most vfs methods call join() on input paths, which wants the simple oid format. It also relieves the caller from having to deal with bogus files/directories in the store.
-
Matt Harbison authored
The test also shows that the local blob store is erroneously lost.
-
Matt Harbison authored
-
- Dec 09, 2017
-
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1635
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1634
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1633
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1632
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1631
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1630
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1629
-
Pulkit Goyal authored
Differential Revision: https://phab.mercurial-scm.org/D1628
-