Add some assertions for deep constructor on object snapshots

Wisembly · Dec 25, 2015 · 30a333f · 30a333f
1 parent 3f308d0
commit 30a333f
Showing 1 changed file with 16 additions and 17 deletions.
diff --git a/test/Snapshot/ObjectSnapshotTest.php b/test/Snapshot/ObjectSnapshotTest.php
@@ -48,23 +48,6 @@ public function testConstructWithoutObject()
         new ObjectSnapshot([]);
     }
 
-    /**
-     * @dataProvider deepProvider
-     */
-    public function testDeepConstructor($value)
-    {
-        new ObjectSnapshot((object) ['foo' => $value]);
-    }
-
-    public function deepProvider()
-    {
-        return [
-            'with a sub-object' => [(object) ['bar' => 'baz']],
-            'with a sub-array' => [['bar' => 'baz']],
-            'with a scalar' => ['fubar']
-        ];
-    }
-
     public function testExportAllProperties()
     {
         $snapshot = new ObjectSnapshot(new Foo('foo', 'bar', 'baz'));
@@ -80,6 +63,22 @@ public function testExportAllProperties()
         $this->assertSame('bar', $data['bar']);
         $this->assertSame('baz', $data['baz']);
     }
+
+    public function testDeepConstructor()
+    {
+        $object = new Foo(
+            (object) ['foo' => 'bar'], // object
+            ['baz' => 'fubar'], // array
+            'fubaz' // scalar
+        );
+
+        $snapshot = new ObjectSnapshot($object);
+        $data = $snapshot->getComparableData();
+
+        $this->assertInstanceOf('Totem\\Snapshot\\ObjectSnapshot', $data['foo']);
+        $this->assertInstanceOf('Totem\\Snapshot\\ArraySnapshot', $data['bar']);
+        $this->assertNotInstanceOf('Totem\\AbstractSnapshot', $data['baz']);
+    }
 }
 
 // todo in php7 : use anon class !