tests: add some more actions to test-issue-6028, describe what's being done This patch does two things: it explains what's happening in the test file so it's easier to understand, and also it adds checks that make sure something like swapping merge parents is not accidentally breaking anything. The primary reason to touch this test file was that it was broken by a change in core's merge.graft() that erroneously swapped merge parents. Since only evolve uses merge.graft() for merge commits and there aren't any tests in core for it, let's test it here. Plus, this test case is pretty simple and these additional checks don't make it too complicated.

     1

import sys

     2

     3

formatted_args = []

     4

     5

UNSAFE_CHARACTERS = [" ", "!", "\"", "#", "$", "&", "'", "(", ")", "*", ",", ";", "<", ">", "?", "[", "\\", "]", "^", "`", "{", "|", "}", ":", "~", "/"]

     6

     7

     8

def find_unsafe(arg):

     9

    for unsafe in UNSAFE_CHARACTERS:

    10

        if unsafe in arg:

    11

            return True

    12

    13

    return False

    14

    15

    16

for arg in sys.argv[1:]:

    17

    if find_unsafe(arg):

    18

        formatted_args.append('"%s"' % arg)

    19

    else:

    20

        formatted_args.append(arg)

    21

    22

print("$ " + " ".join(formatted_args))