Improve exception message for failed setId in Statement

RusticiSoftware · May 28, 2014 · 2fc4614 · 2fc4614
1 parent 7ef39e9
commit 2fc4614
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/Statement.php b/src/Statement.php
@@ -89,7 +89,7 @@ public function stamp() {
 
     public function setId($value) {
         if (isset($value) && ! preg_match(Util::UUID_REGEX, $value)) {
-            throw new \InvalidArgumentException('arg1 must be a UUID');
+            throw new \InvalidArgumentException('arg1 must be a UUID "' . $value . '"');
         }
         $this->id = $value;
         return $this;

diff --git a/tests/StatementTest.php b/tests/StatementTest.php
@@ -65,6 +65,16 @@ public function testStamp() {
         $this->assertRegExp(TinCan\Util::UUID_REGEX, $obj->getId(), 'id is UUId');
     }
 
+    public function testSetId() {
+        $this->setExpectedException(
+            'InvalidArgumentException',
+            'arg1 must be a UUID "some invalid id"'
+        );
+
+        $obj = new Statement();
+        $obj->setId('some invalid id');
+    }
+
     /*
     // TODO: need to loop possible configs
     public function testFromJSONInstantiations() {