Action no longer sets an ActionID internally

Author: clue

src/Protocol/Action.php

@@ -10,8 +10,6 @@ public function __construct($action, array $parts = array())
     {
         $this->action = $action;
         $this->parts = $parts;
-
-        $this->parts['ActionID'] = (string)mt_rand();
     }
 
     public function getMessageSerialized()

tests/Protocol/ActionTest.php

@@ -1,37 +1,48 @@
 <?php
 
 use Clue\React\Ami\Protocol\Action;
+
 class ActionTest extends TestCase
 {
+    public function testIdDefaultsToNull()
+    {
+        $action = new Action('name');
+
+        $this->assertNull($action->getActionId());
+    }
+
+    public function testIdCanBeSet()
+    {
+        $action = new Action('name', array('ActionID' => '123'));
+
+        $this->assertEquals('123', $action->getActionId());
+    }
+
     public function testSerializeSimple()
     {
         $action = new Action('name');
-        $id = $action->getActionId();
 
-        $this->assertEquals("Action: name\r\nActionID: $id\r\n\r\n", $action->getMessageSerialized());
+        $this->assertEquals("Action: name\r\n\r\n", $action->getMessageSerialized());
     }
 
     public function testSerializeKeySingle()
     {
         $action = new Action('name', array('Key' => 'Value'));
-        $id = $action->getActionId();
 
-        $this->assertEquals("Action: name\r\nKey: Value\r\nActionID: $id\r\n\r\n", $action->getMessageSerialized());
+        $this->assertEquals("Action: name\r\nKey: Value\r\n\r\n", $action->getMessageSerialized());
     }
 
     public function testSerializeKeyMultipleValues()
     {
         $action = new Action('name', array('Key' => array('Value1', 'Value2')));
-        $id = $action->getActionId();
 
-        $this->assertEquals("Action: name\r\nKey: Value1\r\nKey: Value2\r\nActionID: $id\r\n\r\n", $action->getMessageSerialized());
+        $this->assertEquals("Action: name\r\nKey: Value1\r\nKey: Value2\r\n\r\n", $action->getMessageSerialized());
     }
 
     public function testSerializeKeyMultipleKeyValues()
     {
         $action = new Action('name', array('Variables' => array('first' => 'on', 'second' => 'off')));
-        $id = $action->getActionId();
 
-        $this->assertEquals("Action: name\r\nVariables: first=on\r\nVariables: second=off\r\nActionID: $id\r\n\r\n", $action->getMessageSerialized());
+        $this->assertEquals("Action: name\r\nVariables: first=on\r\nVariables: second=off\r\n\r\n", $action->getMessageSerialized());
     }
 }