Support serializing action keys with multiple values

Author: clue

src/Protocol/Action.php

@@ -17,8 +17,16 @@ public function __construct($action, array $parts = array())
     public function getMessageSerialized()
     {
         $message = 'Action: ' . $this->action . "\r\n";
-        foreach ($this->parts as $key => $value) {
-            $message .= $key . ': ' . $value . "\r\n";
+        foreach ($this->parts as $key => $values) {
+            if (!is_array($values)) {
+                $values = array($values);
+            }
+            foreach ($values as $i => $value) {
+                if (!is_int($i)) {
+                    $value = $i . '=' . $value;
+                }
+                $message .= $key . ': ' . $value . "\r\n";
+            }
         }
         $message .= "\r\n";
 

tests/Protocol/ActionTest.php

@@ -0,0 +1,37 @@
+<?php
+
+use Clue\React\Ami\Protocol\Action;
+class ActionTest extends TestCase
+{
+    public function testSerializeSimple()
+    {
+        $action = new Action('name');
+        $id = $action->getActionId();
+
+        $this->assertEquals("Action: name\r\nActionID: $id\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());
+    }
+
+    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());
+    }
+
+    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());
+    }
+}