Don't include port if port is default port of the protocol (#70)

src/linkplay/endpoint.py

@@ -37,7 +37,12 @@ def __init__(
             "http",
             "https",
         ], "Protocol must be either 'http' or 'https'"
-        self._endpoint: str = f"{protocol}://{endpoint}:{port}"
+        include_port = (protocol == "http" and port != 80) or (
+            protocol == "https" and port != 443
+        )
+        port_suffix = f":{port}" if include_port else ""
+        self._endpoint: str = f"{protocol}://{endpoint}{port_suffix}"
+
         self._session: ClientSession = session
 
     def to_dict(self):

tests/linkplay/test_bridge.py

@@ -4,7 +4,6 @@
 from unittest.mock import AsyncMock, patch
 
 import pytest
-
 from linkplay.bridge import (
     LinkPlayBridge,
     LinkPlayDevice,
@@ -31,7 +30,7 @@ def test_device_name():
         protocol="http", port=80, endpoint="1.2.3.4", session=None
     )
     bridge: LinkPlayBridge = LinkPlayBridge(endpoint=endpoint)
-    assert f"{bridge}" == "http://1.2.3.4:80"
+    assert f"{bridge}" == "http://1.2.3.4"
 
     bridge.device.properties[DeviceAttribute.DEVICE_NAME] = "TestDevice"
     assert f"{bridge}" == "TestDevice"

tests/linkplay/test_endpoint.py

@@ -1,10 +1,29 @@
 """Test endpoint functionality."""
 
 import pytest
-
 from linkplay.endpoint import LinkPlayApiEndpoint
 
 
+@pytest.mark.parametrize(
+    "protocol, endpoint, port, expected",
+    [
+        ("http", "1.2.3.4", 80, "http://1.2.3.4"),
+        ("http", "1.2.3.4", 8080, "http://1.2.3.4:8080"),
+        ("https", "1.2.3.4", 443, "https://1.2.3.4"),
+        ("https", "1.2.3.4", 8443, "https://1.2.3.4:8443"),
+    ],
+)
+def test_api_endpoint_protocol_port_combination(
+    protocol, endpoint, port, expected
+) -> None:
+    """Tests the endpoint creation"""
+
+    endpoint: LinkPlayApiEndpoint = LinkPlayApiEndpoint(
+        protocol=protocol, port=port, endpoint=endpoint, session=None
+    )
+    assert f"{endpoint}" == expected
+
+
 def test_api_endpoint_protocol_raises_assertion_error() -> None:
     """Tests whether or not instantiating the LinkPlayApiEndpoint
     with an invalid protocol raises an AssertionError."""