No OneTemporary
Actions

Size

6 KB

Referenced Files

None

Subscribers

None

View Options

	diff --git a/src/lint/linter/__tests__/ArcanistLinterTestCase.php b/src/lint/linter/__tests__/ArcanistLinterTestCase.php
	index b933171b..2c6c2051 100644
	--- a/src/lint/linter/__tests__/ArcanistLinterTestCase.php
	+++ b/src/lint/linter/__tests__/ArcanistLinterTestCase.php
	@@ -1,193 +1,193 @@
	<?php

	/**
	* Facilitates implementation of test cases for @{class:ArcanistLinter}s.
	*/
	abstract class ArcanistLinterTestCase extends ArcanistPhutilTestCase {

	public function executeTestsInDirectory($root, ArcanistLinter $linter) {
	$files = id(new FileFinder($root))
	->withType('f')
	->withSuffix('lint-test')
	->find();

	$test_count = 0;
	foreach ($files as $file) {
	$this->lintFile($root.$file, $linter);
	$test_count++;
	}

	$this->assertTrue(
	($test_count > 0),
	pht('Expected to find some .lint-test tests in directory %s!', $root));
	}

	private function lintFile($file, ArcanistLinter $linter) {
	$linter = clone $linter;

	$contents = Filesystem::readFile($file);
	$contents = preg_split('/^~{4,}\n/m', $contents);
	if (count($contents) < 2) {
	throw new Exception(
	"Expected '~~~~~~~~~~' separating test case and results.");
	}

	list ($data, $expect, $xform, $config) = array_merge(
	$contents,
	array(null, null));

	$basename = basename($file);

	if ($config) {
	$config = phutil_json_decode($config);
	} else {
	$config = array();
	}
	PhutilTypeSpec::checkMap(
	$config,
	array(
	'hook' => 'optional bool',
	- 'config' => 'optional wild',
	+ 'config' => 'optional map<string, wild>',
	'path' => 'optional string',
	));

	$exception = null;
	$after_lint = null;
	$messages = null;
	$exception_message = false;
	$caught_exception = false;

	try {
	$tmp = new TempFile($basename);
	Filesystem::writeFile($tmp, $data);
	$full_path = (string)$tmp;

	$dir = dirname($full_path);
	$path = basename($full_path);

	$working_copy = ArcanistWorkingCopyIdentity::newFromRootAndConfigFile(
	$dir,
	null,
	'Unit Test');
	$configuration_manager = new ArcanistConfigurationManager();
	$configuration_manager->setWorkingCopyIdentity($working_copy);


	$engine = new UnitTestableArcanistLintEngine();
	$engine->setWorkingCopy($working_copy);
	$engine->setConfigurationManager($configuration_manager);
	$engine->setPaths(array($path));

	$engine->setCommitHookMode(idx($config, 'hook', false));

	$path_name = idx($config, 'path', $path);
	$linter->addPath($path_name);
	$linter->addData($path_name, $data);
	$config = idx($config, 'config', array());
	foreach ($config as $key => $value) {
	$linter->setLinterConfigurationValue($key, $value);
	}

	$engine->addLinter($linter);
	$engine->addFileData($path_name, $data);

	$results = $engine->run();

	$this->assertEqual(
	1,
	count($results),
	'Expect one result returned by linter.');

	$result = reset($results);
	$patcher = ArcanistLintPatcher::newFromArcanistLintResult($result);
	$after_lint = $patcher->getModifiedFileContent();
	} catch (ArcanistPhutilTestTerminatedException $ex) {
	throw $ex;
	} catch (Exception $exception) {
	$caught_exception = true;
	if ($exception instanceof PhutilAggregateException) {
	$caught_exception = false;
	foreach ($exception->getExceptions() as $ex) {
	if ($ex instanceof ArcanistUsageException) {
	$this->assertSkipped($ex->getMessage());
	} else {
	$caught_exception = true;
	}
	}
	} else if ($exception instanceof ArcanistUsageException) {
	$this->assertSkipped($exception->getMessage());
	}
	$exception_message = $exception->getMessage()."\n\n".
	$exception->getTraceAsString();
	}

	$this->assertEqual(false, $caught_exception, $exception_message);
	$this->compareLint($basename, $expect, $result);
	$this->compareTransform($xform, $after_lint);
	}

	private function compareLint($file, $expect, ArcanistLintResult $result) {
	$seen = array();
	$raised = array();
	$message_map = array();

	foreach ($result->getMessages() as $message) {
	$sev = $message->getSeverity();
	$line = $message->getLine();
	$char = $message->getChar();
	$code = $message->getCode();
	$name = $message->getName();
	$message_key = $sev.':'.$line.':'.$char;
	$message_map[$message_key] = $message;
	$seen[] = $message_key;
	$raised[] = " {$sev} at line {$line}, char {$char}: {$code} {$name}";
	}
	$expect = trim($expect);
	if ($expect) {
	$expect = explode("\n", $expect);
	} else {
	$expect = array();
	}
	foreach ($expect as $key => $expected) {
	$expect[$key] = head(explode(' ', $expected));
	}

	$expect = array_fill_keys($expect, true);
	$seen = array_fill_keys($seen, true);

	if (!$raised) {
	$raised = array('No messages.');
	}
	$raised = "Actually raised:\n".implode("\n", $raised);

	foreach (array_diff_key($expect, $seen) as $missing => $ignored) {
	list($sev, $line, $char) = explode(':', $missing);
	$this->assertFailure(
	"In '{$file}', ".
	"expected lint to raise {$sev} on line {$line} at char {$char}, ".
	"but no {$sev} was raised. {$raised}");
	}

	foreach (array_diff_key($seen, $expect) as $surprising => $ignored) {
	$message = $message_map[$surprising];
	$message_info = $message->getDescription();

	list($sev, $line, $char) = explode(':', $surprising);
	$this->assertFailure(
	"In '{$file}', ".
	"lint raised {$sev} on line {$line} at char {$char}, ".
	"but nothing was expected:\n\n{$message_info}\n\n{$raised}");
	}
	}

	private function compareTransform($expected, $actual) {
	if (!strlen($expected)) {
	return;
	}
	$this->assertEqual(
	$expected,
	$actual,
	'File as patched by lint did not match the expected patched file.');
	}

	}

File Metadata

Mime Type: text/x-diff
Expires: Sun, Jan 19, 15:35 (3 w, 23 h ago)
Storage Engine: blob
Storage Format: Raw Data
Storage Handle: 1126043
Default Alt Text: (6 KB)

No OneTemporaryActions

View Options

File Metadata

Event Timeline

No OneTemporary
Actions