Differential D15675 Diff 38899 src/xsprintf/__tests__/PhutilCsprintfTestCase.php

Changeset View

Standalone View

src/xsprintf/tests/PhutilCsprintfTestCase.php

Show All 10 Lines	$inputs = array(
// For arguments which have any characters which are not safe in some		// For arguments which have any characters which are not safe in some
// context, %R should apply standard escaping.		// context, %R should apply standard escaping.
'a b' => false,		'a b' => false,


'http://domain.com/path/' => true,		'http://domain.com/path/' => true,
'svn+ssh://domain.com/path/' => true,		'svn+ssh://domain.com/path/' => true,
'`rm -rf`' => false,		'`rm -rf`' => false,
'$VALUE' => false,		'$VALUE' => phutil_is_windows(),
);		);

foreach ($inputs as $input => $expect_same) {		foreach ($inputs as $input => $expect_same) {
$actual = (string)csprintf('%R', $input);		$actual = (string)csprintf('%R', $input);
if ($expect_same) {		if ($expect_same) {
$this->assertEqual($input, $actual);		$this->assertEqual($input, $actual);
} else {		} else {
$this->assertFalse($input === $actual);		$this->assertFalse($input === $actual);
}		}
}		}
}		}

public function testPowershell() {		public function testPowershell() {
$cmd = csprintf('%s', "\n");		$cmd = csprintf('%s', "\n");
$cmd->setEscapingMode(PhutilCommandString::MODE_POWERSHELL);		$cmd->setEscapingMode(PhutilCommandString::MODE_POWERSHELL);

$this->assertEqual(		$this->assertEqual(
'"`n"',		'"`n"',
(string)$cmd);		(string)$cmd);
}		}

public function testNoPowershell() {		public function testNoPowershell() {
if (!phutil_is_windows()) {		$cmd = csprintf('%s', '#"');
$cmd = csprintf('%s', '#');
$cmd->setEscapingMode(PhutilCommandString::MODE_DEFAULT);		$cmd->setEscapingMode(PhutilCommandString::MODE_DEFAULT);

$this->assertEqual(		$this->assertEqual(
'\'#\'',		phutil_is_windows() ? '"#\""' : '\'#"\'',
		BYKAuthorUnsubmitted Not Done Inline Actions Well, this is not really accurate. I should probably branch and use the expected Windows version at this point. What do you think? BYK: Well, this is not really accurate. I should probably branch and use the expected Windows…
(string)$cmd);		(string)$cmd
}		);
}		}

public function testPasswords() {		public function testPasswords() {
// Normal "%s" doesn't do anything special.		// Normal "%s" doesn't do anything special.
$command = csprintf('echo %s', 'hunter2trustno1');		$command = csprintf('echo %s', 'hunter2trustno1');
$this->assertTrue(strpos($command, 'hunter2trustno1') !== false);		$this->assertTrue(strpos($command, 'hunter2trustno1') !== false);

// "%P" takes a PhutilOpaqueEnvelope.		// "%P" takes a PhutilOpaqueEnvelope.
Show All 13 Lines	public function testPasswords() {

// Executing the command works as expected.		// Executing the command works as expected.
list($out) = execx('%C', $command);		list($out) = execx('%C', $command);
$this->assertTrue(strpos($out, 'hunter2trustno1') !== false);		$this->assertTrue(strpos($out, 'hunter2trustno1') !== false);
}		}

public function testEscapingIsRobust() {		public function testEscapingIsRobust() {
if (phutil_is_windows()) {		if (phutil_is_windows()) {
		// NOTE: The reason we can't run this test on Windows is two fold:
		// 1. We don't have a sane process starter in Windows
		// 2. We can't use CMD because its escaping is unreliable
$this->assertSkipped(pht("This test doesn't work on Windows."));		$this->assertSkipped(pht("This test doesn't work on Windows."));
}		}

// Escaping should be robust even when used to escape commands which take		// Escaping should be robust even when used to escape commands which take
// other commands.		// other commands.
list($out) = execx(		list($out) = execx(
'sh -c %s',		'sh -c %s',
csprintf(		csprintf(
'sh -c %s',		'sh -c %s',
csprintf(		csprintf(
'sh -c %s',		'sh -c %s',
csprintf(		csprintf(
'echo %P',		'echo %P',
new PhutilOpaqueEnvelope('!@#$%^&*()')))));		new PhutilOpaqueEnvelope('!@#$%^&*()')))));
$this->assertTrue(strpos($out, '!@#$%^&*()') !== false);		$this->assertTrue(strpos($out, '!@#$%^&*()') !== false);
}		}

		public function testEdgeCases() {
		epriestleyUnsubmitted Not Done Inline Actions These test cases do not test what they intend to test. In PHP, `'\0'` is the string literal "backslash, zero", not a null byte. Likewise, `'\n'` and `'\r'` are not newline characters. When these strings are changed to use double quotes, the test cases fail. epriestley: These test cases do not test what they intend to test. In PHP, `'\0'` is the string literal…
		BYKAuthorUnsubmitted Not Done Inline Actions Ah, okay I see. Will fix. BYK: Ah, okay I see. Will fix.
		BYKAuthorUnsubmitted Not Done Inline Actions Turns out it is not possible to pass a NULL BYTE through the command line anyways so gonna drop that case. Working on the new lines which are the other two problems. BYK: Turns out it is not possible to pass a NULL BYTE through the command line anyways so gonna drop…
		$edgeCases = array(
		"\\",
		"%",
		"%%",
		" ", // space
		"", // empty string
		"-",
		"/flag",
		"\\\^\%\\\'\ \\",
		'%PATH%',
		'%XYZ%',
		'%%HOMEDIR%',
		"a\nb", // newline
		"a\rb", // newline
		"a\r\n\n\rb", // newline
		"a b",
		'"a b"',
		'"%%$HOMEDIR%^^"',
		'\'a b\'',
		'^%HO ^"M\'EDIR^%^%\'',
		'"\'a\0\r\nb%PATH%%`\'"\'`\'`\'',
		);

		foreach ($edgeCases as $edgeCase) {
		list($output) = execx('php -r %s %s', 'echo $argv[1];', $edgeCase);
		BYKAuthorUnsubmitted Not Done Inline Actions Should probably do list($output) = execx('php -r %s -- %s', 'echo $argv[1];', $edgeCase); instead. BYK: Should probably do list($output) = execx('php -r %s -- %s', 'echo $argv[1];', $edgeCase)…
		$this->assertEqual($edgeCase, $output);
		}
		}
}		}