Merge pull request #338 from opentok/feature/hls-broadcast-status

Status keys for broadcast object

Author: SecondeJK

src/OpenTok/Broadcast.php

@@ -44,6 +44,9 @@
 *
 * @property boolean $isDvr
 * Whether the broadcast supports DVR functionality for the HLS stream.
+
+* @property string $status
+* Broadcast state. Either `started` or `stopped`
 *
 * @property boolean $isLowLatency
 * Whether the broadcast supports low-latency mode for the HLS stream.
@@ -86,6 +89,8 @@ class Broadcast
     private $hasAudio;
     /** @ignore */
     private $hasVideo;
+    /** @ignore */
+    private $status;
 
     public function __construct($broadcastData, $options = array())
     {
@@ -107,34 +112,37 @@ public function __construct($broadcastData, $options = array())
         );
 
         $options = array_merge($defaults, array_intersect_key($options, $defaults));
-        list($apiKey, $apiSecret, $apiUrl, $client, $isStopped, $streamMode, $hasAudio, $hasVideo) = array_values($options);
 
-        // validate params
         Validators::validateBroadcastData($broadcastData);
-        Validators::validateClient($client);
-        Validators::validateHasStreamMode($streamMode);
+        Validators::validateClient($options['client']);
+        Validators::validateHasStreamMode($options['streamMode']);
 
         $this->data = $broadcastData;
 
         if (isset($this->data['multiBroadcastTag'])) {
             $this->multiBroadcastTag = $this->data['multiBroadcastTag'];
         }
 
-        $this->isStopped = $isStopped;
+        if (isset($this->data['status'])) {
+            $this->status = $this->data['status'];
+        }
+
+        $this->isStopped = $options['isStopped'];
         $this->resolution = $this->data['resolution'];
         $this->isHls = isset($this->data['settings']['hls']);
         $this->isLowLatency = $this->data['settings']['hls']['lowLatency'] ?? false;
         $this->isDvr = $this->data['settings']['hls']['dvr'] ?? false;
-        $this->hasAudio = $hasAudio;
-        $this->hasVideo = $hasVideo;
+        $this->hasAudio = $options['hasAudio'];
+        $this->hasVideo = $options['hasVideo'];
+
+        $this->client = $options['client'] ?? new Client();
 
-        $this->client = isset($client) ? $client : new Client();
         if (!$this->client->isConfigured()) {
-            Validators::validateApiKey($apiKey);
-            Validators::validateApiSecret($apiSecret);
-            Validators::validateApiUrl($apiUrl);
+            Validators::validateApiKey($options['apiKey']);
+            Validators::validateApiSecret($options['apiSecret']);
+            Validators::validateApiUrl($options['apiUrl']);
 
-            $this->client->configure($apiKey, $apiSecret, $apiUrl);
+            $this->client->configure($options['apiKey'], $options['apiSecret'], $options['apiUrl']);
         }
     }
 
@@ -148,7 +156,6 @@ public function __get($name)
             case 'partnerId':
             case 'sessionId':
             case 'broadcastUrls':
-            case 'status':
             case 'maxDuration':
             case 'streamMode':
                 return $this->data[$name];
@@ -170,6 +177,8 @@ public function __get($name)
                 return $this->hasAudio;
             case 'hasVideo':
                 return $this->hasVideo;
+            case 'status':
+                return $this->status;
             default:
                 return null;
         }

tests/OpenTokTest/BroadcastTest.php

@@ -34,6 +34,7 @@ public function setUp(): void
             'updatedAt' => 1394394801000,
             'partnerId' => 685,
             'sessionId' => '2_MX44NTQ1MTF-flR1ZSBOb3YgMTIgMDk6NDA6NTkgUFNUIDIwMTN-MC43NjU0Nzh-',
+            'multiBroadcastTag' => 'broadcast-1234b',
             'layout' => [
                 'type' => 'custom',
                 'stylesheet' => 'a layout stylesheet',
@@ -42,8 +43,9 @@ public function setUp(): void
             'maxDuration' => 5400,
             'resolution' => '640x480',
             'streamMode' => StreamMode::AUTO,
-            'isAudio' => true,
-            'isVideo' => true
+            'status' => 'started',
+            'hasAudio' => true,
+            'hasVideo' => true
         ];
     }
 
@@ -134,12 +136,12 @@ private function setupOT()
         return $this->setupOTWithMocks([]);
     }
 
-    public function testInitializes()
+    public function testInitializes(): void
     {
-        // Arrange
         $this->setupOT();
         $this->setupBroadcasts(StreamMode::AUTO);
         $this->assertInstanceOf(Broadcast::class, $this->broadcast);
+
     }
 
     public function testCannotAddStreamToBroadcastInAutoMode(): void
@@ -238,5 +240,20 @@ public function testCannotRemoveStreamFromBroadcastOnAuto(): void
             '5dfds4-asdda4asf4'
         );
     }
+
+    public function testGetters(): void
+    {
+        $broadcastObject = new Broadcast($this->broadcastData, [
+            'apiKey' => 'abc',
+            'apiSecret' => 'efg',
+            'client' => $this->client
+        ]);
+
+        $this->assertTrue($broadcastObject->hasAudio);
+        $this->assertTrue($broadcastObject->hasVideo);
+        $this->assertEquals('broadcast-1234b', $broadcastObject->multiBroadcastTag);
+        $this->assertEquals('started', $broadcastObject->status);
+        $this->assertNull($broadcastObject->wrongKey);
+    }
 }
 

tests/OpenTokTest/OpenTokTest.php

@@ -1640,6 +1640,7 @@ public function testCanStartBroadcastWithDefaultHlsOptions(): void
 		$this->assertTrue($broadcast->isHls);
 		$this->assertFalse($broadcast->isDvr);
 		$this->assertFalse($broadcast->isLowLatency);
+        $this->assertEquals('live', $broadcast->broadcastUrls['rtmp']['foo']['status']);
 	}
 
 	public function testCanStartBroadcastWithDvrEnabled(): void

tests/mock/v2/project/APIKEY/broadcast/BROADCASTID/start_default

@@ -5,14 +5,17 @@
   "createdAt":1472435659497,
   "broadcastUrls":{
     "hls":"https://cdn-broadcast001-dub.tokbox.com/29908/29908_6706b658-2eba-42cc-b4d2-7d01a104d182.smil/playlist.m3u8",
+    "hlsStatus": "ready",
     "rtmp": {
       "foo": {
         "serverUrl": "rtmp://myfooserver/myfooapp",
-        "streamName": "myfoostreamname"
+        "streamName": "myfoostreamname",
+        "status": "live"
       },
       "bar": {
         "serverUrl": "rtmp://mybarserver/mybarapp",
-        "streamName": "mybarstreamname"
+        "streamName": "mybarstreamname",
+        "status": "offline"
       }
     }
   },

tests/mock/v2/project/APIKEY/broadcast/BROADCASTID/start_ll

@@ -5,14 +5,17 @@
   "createdAt":1472435659497,
   "broadcastUrls":{
     "hls":"https://cdn-broadcast001-dub.tokbox.com/29908/29908_6706b658-2eba-42cc-b4d2-7d01a104d182.smil/playlist.m3u8",
+    "hlsStatus": "ready",
     "rtmp": {
       "foo": {
         "serverUrl": "rtmp://myfooserver/myfooapp",
-        "streamName": "myfoostreamname"
+        "streamName": "myfoostreamname",
+        "status": "live"
       },
       "bar": {
         "serverUrl": "rtmp://mybarserver/mybarapp",
-        "streamName": "mybarstreamname"
+        "streamName": "mybarstreamname",
+        "status": "offline"
       }
     }
   },