D9826.id.diff
No OneTemporary
Actions

Size

1 KB

Referenced Files

None

Subscribers

None

D9826.id.diff
View Options

	diff --git a/src/__tests__/ArcanistInfrastructureTestCase.php b/src/__tests__/ArcanistInfrastructureTestCase.php
	--- a/src/__tests__/ArcanistInfrastructureTestCase.php
	+++ b/src/__tests__/ArcanistInfrastructureTestCase.php
	@@ -1,6 +1,7 @@
	<?php

	final class ArcanistInfrastructureTestCase extends ArcanistTestCase {
	+
	/**
	* This is more of an acceptance test case instead of a unit test. It verifies
	* that all symbols can be loaded correctly. It can catch problems like
	@@ -10,4 +11,27 @@
	id(new PhutilSymbolLoader())->selectAndLoadSymbols();
	$this->assertTrue(true);
	}
	+
	+ /**
	+ * This is more of an acceptance test case instead of a unit test. It verifies
	+ * that all the library map is up-to-date.
	+ */
	+ public function testLibraryMap() {
	+ $root = phutil_get_library_root('arcanist');
	+
	+ $new_library_map = id(new PhutilLibraryMapBuilder($root))
	+ ->setQuiet(true)
	+ ->setDryRun(true)
	+ ->buildMap();
	+
	+ $bootloader = PhutilBootloader::getInstance();
	+ $old_library_map = $bootloader->getLibraryMap('arcanist');
	+ unset($old_library_map[PhutilLibraryMapBuilder::LIBRARY_MAP_VERSION_KEY]);
	+
	+ $this->assertEqual(
	+ $new_library_map,
	+ $old_library_map,
	+ 'The library map does not appear to be up-to-date. Try '.
	+ 'rebuilding the map with `arc liberate`.');
	+ }
	}