diff tests/unit/test_plugin_xep_0167.py @ 4285:f1d0cde61af7

tests (unit): fix tests + black reformatting.
author Goffi <goffi@goffi.org>
date Sun, 14 Jul 2024 17:42:53 +0200
parents 716dd791be46
children 9447796408f6
line wrap: on
line diff
--- a/tests/unit/test_plugin_xep_0167.py	Sun Jul 14 16:47:45 2024 +0200
+++ b/tests/unit/test_plugin_xep_0167.py	Sun Jul 14 17:42:53 2024 +0200
@@ -25,7 +25,10 @@
 
 from libervia.backend.plugins.plugin_xep_0166 import XEP_0166
 from libervia.backend.plugins.plugin_xep_0167 import XEP_0167, mapping
-from libervia.backend.plugins.plugin_xep_0167.constants import NS_JINGLE_RTP, NS_JINGLE_RTP_INFO
+from libervia.backend.plugins.plugin_xep_0167.constants import (
+    NS_JINGLE_RTP,
+    NS_JINGLE_RTP_INFO,
+)
 from libervia.backend.tools import xml_tools
 from libervia.backend.tools.common import data_format
 
@@ -45,8 +48,8 @@
         """Senders are mapped to SDP attribute"""
         assert mapping.senders_to_sdp("both", {"role": "initiator"}) == "a=sendrecv"
         assert mapping.senders_to_sdp("none", {"role": "initiator"}) == "a=inactive"
-        assert mapping.senders_to_sdp("initiator", {"role": "initiator"}) == "a=sendonly"
-        assert mapping.senders_to_sdp("responder", {"role": "initiator"}) == "a=recvonly"
+        assert mapping.senders_to_sdp("initiator", {"role": "initiator"}) == "a=recvonly"
+        assert mapping.senders_to_sdp("responder", {"role": "initiator"}) == "a=sendonly"
 
     def test_generate_sdp_from_session(self):
         """SDP is correctly generated from session data"""
@@ -134,35 +137,36 @@
                     "media": "audio",
                     "payload_types": {
                         96: {
+                            "clockrate": 48000,
                             "id": 96,
                             "name": "opus",
-                            "clockrate": 48000,
                             "parameters": {"sprop-stereo": "1"},
                         }
                     },
                 },
+                "id": "audio",
+                "senders": "both",
                 "transport_data": {
+                    "candidates": [
+                        {
+                            "address": "10.0.0.1",
+                            "component_id": 1,
+                            "foundation": "1",
+                            "port": 12345,
+                            "priority": 1,
+                            "transport": "UDP",
+                            "type": "host",
+                        }
+                    ],
                     "port": 9999,
                     "pwd": "pwd",
                     "ufrag": "ufrag",
-                    "candidates": [
-                        {
-                            "foundation": "1",
-                            "component_id": 1,
-                            "transport": "UDP",
-                            "priority": 1,
-                            "address": "10.0.0.1",
-                            "port": 12345,
-                            "type": "host",
-                        }
-                    ],
                 },
-                "id": "audio",
             },
             "metadata": {},
         }
 
-        assert mapping.parse_sdp(sdp) == expected_session
+        assert mapping.parse_sdp(sdp, "responder") == expected_session
 
     def test_build_description(self):
         """<description> element is generated from media data"""