Skip to content

Commit c734e0d

Browse files
committed
str instead of enum to remain conssitent
1 parent b9678ea commit c734e0d

File tree

3 files changed

+5
-44
lines changed

3 files changed

+5
-44
lines changed

bandwidth/models/bxml/verbs/start_stream.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,13 @@
99

1010
from ..nestable_verb import NestableVerb
1111
from ..verbs.stream_param import StreamParam
12-
from ...streaming_mode_enum import StreamingModeEnum
1312

1413

1514
class StartStream(NestableVerb):
1615

1716
def __init__(
1817
self, destination: str, stream_params: List[StreamParam] = [],
19-
name: str=None, mode: StreamingModeEnum=None, tracks: str=None,
18+
name: str=None, mode: str=None, tracks: str=None,
2019
stream_event_url: str=None,
2120
stream_event_method: str=None,
2221
username: str=None, password: str=None,
@@ -56,7 +55,7 @@ def _attributes(self):
5655
return {
5756
"destination": self.destination,
5857
"name": self.name,
59-
"mode": self.mode.value if self.mode else None,
58+
"mode": self.mode,
6059
"tracks": self.tracks,
6160
"streamEventUrl": self.stream_event_url,
6261
"streamEventMethod": self.stream_event_method,

bandwidth/models/streaming_mode_enum.py

Lines changed: 0 additions & 33 deletions
This file was deleted.

test/unit/models/bxml/test_start_stream.py

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
"""
88
import unittest
99

10-
from bandwidth.models.streaming_mode_enum import StreamingModeEnum
1110
from bandwidth.models.bxml import StartStream, StreamParam, Verb, NestableVerb
1211

1312

@@ -27,6 +26,7 @@ def setUp(self):
2726
self.start_stream = StartStream(
2827
stream_params=[self.stream_param1],
2928
name = "stream1",
29+
mode = "bidirectional",
3030
tracks = "inbound",
3131
destination = "testurl.com",
3232
stream_event_url="eventurl.com",
@@ -41,15 +41,10 @@ def test_instance(self):
4141
assert isinstance(self.start_stream, Verb)
4242

4343
def test_to_bxml(self):
44-
expected = '<StartStream destination="testurl.com" name="stream1" tracks="inbound" streamEventUrl="eventurl.com" streamEventMethod="POST" username="user" password="pass"><StreamParam name="name1" value="value1" /></StartStream>'
44+
expected = '<StartStream destination="testurl.com" name="stream1" mode="bidirectional" tracks="inbound" streamEventUrl="eventurl.com" streamEventMethod="POST" username="user" password="pass"><StreamParam name="name1" value="value1" /></StartStream>'
4545
assert expected == self.start_stream.to_bxml()
4646

4747
def test_add_verb(self):
48-
expected = '<StartStream destination="testurl.com" name="stream1" tracks="inbound" streamEventUrl="eventurl.com" streamEventMethod="POST" username="user" password="pass"><StreamParam name="name1" value="value1" /><StreamParam name="name2" value="value2" /></StartStream>'
48+
expected = '<StartStream destination="testurl.com" name="stream1" mode="bidirectional" tracks="inbound" streamEventUrl="eventurl.com" streamEventMethod="POST" username="user" password="pass"><StreamParam name="name1" value="value1" /><StreamParam name="name2" value="value2" /></StartStream>'
4949
self.start_stream.add_verb(self.stream_param2)
5050
assert expected == self.start_stream.to_bxml()
51-
52-
def test_bidirectional(self):
53-
self.start_stream.mode = StreamingModeEnum.BIDIRECTIONAL
54-
expected = '<StartStream destination="testurl.com" name="stream1" mode="bidirectional" tracks="inbound" streamEventUrl="eventurl.com" streamEventMethod="POST" username="user" password="pass"><StreamParam name="name1" value="value1" /></StartStream>'
55-
assert expected == self.start_stream.to_bxml()

0 commit comments

Comments
 (0)