Skip to content

Commit 150ab0d

Browse files
committed
Merge pull request #137 from cystbear/improve-PropertyReadMethodsTest-testGetBoolean
added check for "thisIsYes" node
2 parents b850e3e + 68cd5f3 commit 150ab0d

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

tests/05_Reading/PropertyReadMethodsTest.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,12 @@ public function testGetBoolean()
328328
// php interprets everything as true except null, 0, '' and boolean false. thus even the string "false" is true.
329329
// we require getString to return something that evaluates to false (the empty string makes sense)
330330
$this->assertTrue(! $prop->getString(), 'boolean false returned as string should evaluate to php <false>');
331+
332+
$prop = $this->node->getNode('numberPropertyNode/jcr:content')->getProperty('thisIsYes');
333+
$this->assertTrue($prop->getBoolean());
334+
$this->assertFalse(!$prop->getString());
331335
}
336+
332337
public function testGetBooleanMulti()
333338
{
334339
$prop = $this->node->getNode('numberPropertyNode/jcr:content')->getProperty('multiBoolean');

0 commit comments

Comments
 (0)