minor #16155 Added more tests for PropertyAccess (pierredup)

fabpot · fabpot · commit a954fea00efd · 2015-10-07T14:25:40.000+02:00
This PR was merged into the 2.7 branch. Discussion ---------- Added more tests for PropertyAccess | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | This is a follow up for [16090#issuecomment-145183635](#16090 (comment)) Commits ------- 378db75 Added more tests for PropertyAccess
diff --git a/src/Symfony/Component/PropertyAccess/Tests/PropertyAccessorTest.php b/src/Symfony/Component/PropertyAccess/Tests/PropertyAccessorTest.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\PropertyAccess\Tests;
 
+use Symfony\Component\PropertyAccess\Exception\NoSuchIndexException;
 use Symfony\Component\PropertyAccess\PropertyAccessor;
 use Symfony\Component\PropertyAccess\Tests\Fixtures\TestClass;
 use Symfony\Component\PropertyAccess\Tests\Fixtures\TestClassMagicCall;
@@ -139,6 +140,20 @@ public function testGetValueNotModifyObject()
         $this->assertSame(array('Bernhard'), $object->firstName);
     }
 
+    public function testGetValueNotModifyObjectException()
+    {
+        $propertyAccessor = new PropertyAccessor(false, true);
+        $object = new \stdClass();
+        $object->firstName = array('Bernhard');
+
+        try {
+            $propertyAccessor->getValue($object, 'firstName[1]');
+        } catch (NoSuchIndexException $e) {
+        }
+
+        $this->assertSame(array('Bernhard'), $object->firstName);
+    }
+
     /**
      * @expectedException \Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException
      */
