Pārlūkot izejas kodu

Merge branch 'pr304'

Rob Allen 11 gadi atpakaļ
vecāks
revīzija
7884400401

+ 1 - 1
library/Zend/Test/PHPUnit/Constraint/DomQuery37.php

@@ -193,7 +193,7 @@ class Zend_Test_PHPUnit_Constraint_DomQuery extends PHPUnit_Framework_Constraint
             case self::ASSERT_CONTENT_COUNT:
             case self::ASSERT_CONTENT_COUNT_MIN:
             case self::ASSERT_CONTENT_COUNT_MAX:
-                if (!$match) {
+                if ($match === false) {
                     require_once 'Zend/Test/PHPUnit/Constraint/Exception.php';
                     throw new Zend_Test_PHPUnit_Constraint_Exception('No count provided against which to compare');
                 }

+ 28 - 0
tests/Zend/Test/PHPUnit/Constraint/DomQueryTest.php

@@ -63,4 +63,32 @@ class Zend_Test_PHPUnit_Constraint_DomQueryTest extends PHPUnit_Framework_TestCa
             Zend_Test_PHPUnit_Constraint_DomQuery::ASSERT_CONTENT_CONTAINS, 'value1');
         $this->assertTrue($result);
     }
+    
+    /**
+     * @group issue-303
+     */
+    public function testShouldAllowMatchingZeroCounts()
+    {
+        $html = '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
+<html>
+    <head>
+        <title>ZF Issue ZF-4010</title>
+    </head>
+    <body>
+        <form>
+            <fieldset id="fieldset-input"><legend>Inputs</legend>
+                <ol>
+                    <li><input type="text" name="input[0]" id="input-0" value="value1" /></li>
+                    <li><input type="text" name="input[1]" id="input-1" value="value2" /></li>
+                    <li><input type="text" name="input[2]" id="input-2" value="" /></li>
+                </ol>
+            </fieldset>
+        </form>
+    </body>
+</html>';
+        $assertion = new Zend_Test_PHPUnit_Constraint_DomQuery('input#input-3 @value');
+        $result = $assertion->evaluate($html,
+            Zend_Test_PHPUnit_Constraint_DomQuery::ASSERT_CONTENT_COUNT, 0);
+        $this->assertTrue($result);
+    }
 }