Ver Fonte

Add known issue for whitespace, mark whitespace test as skipped.

Justin Hileman há 15 anos atrás
pai
commit
582c3278fa
2 ficheiros alterados com 10 adições e 6 exclusões
  1. 1 0
      README.markdown
  2. 9 6
      test/MustacheTest.php

+ 1 - 0
README.markdown

@@ -83,6 +83,7 @@ Known Issues
 
  * Sections don't respect delimiter changes -- `delimiters` example currently fails with an
    "unclosed section" exception.
+ * Mustache isn't always very good at whitespace.
  * Test coverage is incomplete.
 
 

+ 9 - 6
test/MustacheTest.php

@@ -35,6 +35,11 @@ class MustacheTest extends PHPUnit_Framework_TestCase {
 
 	const TEST_CLASS = 'Mustache';
 
+	protected $knownIssues = array(
+		'Delimiters'     => "Known issue: sections don't respect delimiter changes",
+		'SectionsSpaces' => "Known issue: Mustache fails miserably at whitespace",
+	);
+
 	/**
 	 * Test Mustache constructor.
 	 *
@@ -191,9 +196,8 @@ class MustacheTest extends PHPUnit_Framework_TestCase {
 	 * @return void
 	 */
 	public function test__clone($class, $template, $output) {
-		if ($class == 'Delimiters') {
-			$this->markTestSkipped("Known issue: sections don't respect delimiter changes");
-			return;
+		if (isset($this->knownIssues[$class])) {
+			return $this->markTestSkipped($this->knownIssues[$class]);
 		}
 
 		$m = new $class;
@@ -222,9 +226,8 @@ class MustacheTest extends PHPUnit_Framework_TestCase {
 	 * @return void
 	 */
 	public function testExamples($class, $template, $output) {
-		if ($class == 'Delimiters') {
-			$this->markTestSkipped("Known issue: sections don't respect delimiter changes");
-			return;
+		if (isset($this->knownIssues[$class])) {
+			return $this->markTestSkipped($this->knownIssues[$class]);
 		}
 
 		$m = new $class;