Add a bunch of tests on Set::changes property

src/AbstractSnapshot.php

@@ -11,9 +11,10 @@
 
 namespace Totem;
 
-use ArrayAccess;
+use ArrayAccess,
+    ReflectionClass,
 
-use BadMethodCallException,
+    BadMethodCallException,
     InvalidArgumentException;
 
 use Totem\Exception\IncomparableDataException;

src/Set.php

@@ -17,6 +17,7 @@
     ArrayIterator,
     IteratorAggregate,
 
+    RuntimeException,
     OutOfBoundsException,
     BadMethodCallException;
 

test/AbstractSnapshotTest.php

@@ -91,5 +91,21 @@ public function testDiff()
 
         $this->assertInstanceOf('Totem\\Set', $snapshot->diff($snapshot));
     }
+
+    public function testCorrectSetClass()
+    {
+        $snapshot = new Snapshot(['data' => []]);
+        $snapshot->setSetClass('Totem\\Set');
+    }
+
+    /**
+     * @expectedException        InvalidArgumentException
+     * @expectedExceptionMessage A Set Class should be instantiable and implement Totem\SetInterface
+     */
+    public function testWrongSetClass()
+    {
+        $snapshot = new Snapshot(['data' => []]);
+        $snapshot->setSetClass('stdclass');
+    }
 }
 

test/SetTest.php

@@ -136,5 +136,50 @@ public function testForbidenUnsetter()
 
         unset($set[0]);
     }
+
+    /**
+     * @expectedException         RuntimeException
+     * @expectedExceptionMessage  The changeset was not computed yet !
+     */
+    public function testHasChangedNotComputedShouldThrowException()
+    {
+        $set = new Set;
+        $set->hasChanged('foo');
+    }
+
+    /**
+     * @expectedException         RuntimeException
+     * @expectedExceptionMessage  The changeset was not computed yet !
+     */
+    public function testNotComputedCountShouldThrowException()
+    {
+        $set = new Set;
+        count($set);
+    }
+
+    /**
+     * @expectedException         RuntimeException
+     * @expectedExceptionMessage  The changeset was not computed yet !
+     */
+    public function testNotComputedIteratorShouldThrowException()
+    {
+        $set = new Set;
+        $set->getIterator();
+    }
+
+    public function testAlreadyComputedSetShouldNotRecompute()
+    {
+        $old = new Snapshot(['data' => ['foo']]);
+        $new = new Snapshot(['data' => ['bar']]);
+
+        $set = new Set($old, $new); // implicitly compute the set in the constructor
+
+        $this->assertCount(1, $set);
+        $this->assertTrue($set->hasChanged(0));
+        $this->assertEquals('foo', $set->getChange(0)->getOld());
+        $this->assertEquals('bar', $set->getChange(0)->getNew());
+
+        $set->compute($old, $new);
+    }
 }