py-mine · PerchunPak · Feb 11, 2024 · Feb 5, 2024 · Feb 11, 2024
diff --git a/mcstatus/motd/__init__.py b/mcstatus/motd/__init__.py
@@ -20,7 +20,7 @@
 MOTD_COLORS_RE = re.compile(r"([\xA7|&][0-9A-FK-OR])", re.IGNORECASE)
 
 
-@dataclass
+@dataclass(frozen=True)
 class Motd:
     """Represents parsed MOTD."""
 

diff --git a/mcstatus/motd/components.py b/mcstatus/motd/components.py
@@ -55,7 +55,7 @@ class MinecraftColor(Enum):
     MINECOIN_GOLD = "g"
 
 
-@dataclass
+@dataclass(frozen=True)
 class WebColor:
     """Raw HTML color from MOTD.
 
@@ -110,7 +110,7 @@ def _check_rgb(rgb: tuple[int, int, int]) -> None:
                 raise ValueError(f"RGB color byte out of its 8-bit range (0-255) for {color_name} ({value=})")
 
 
-@dataclass
+@dataclass(frozen=True)
 class TranslationTag:
     """Represents a ``translate`` field in server's answer.
 

diff --git a/mcstatus/status_response.py b/mcstatus/status_response.py
@@ -68,7 +68,7 @@ class RawJavaResponse(TypedDict):
 ]
 
 
-@dataclass
+@dataclass(frozen=True)
 class BaseStatusResponse(ABC):
     """Class for storing shared data from a status response."""
 
@@ -101,7 +101,7 @@ def build(cls, *args, **kwargs) -> Self:
         raise NotImplementedError("You can't use abstract methods.")
 
 
-@dataclass
+@dataclass(frozen=True)
 class JavaStatusResponse(BaseStatusResponse):
     """The response object for :meth:`JavaServer.status() <mcstatus.server.JavaServer.status>`."""
 
@@ -159,7 +159,7 @@ def build(cls, raw: RawJavaResponse, latency: float = 0) -> Self:
         )
 
 
-@dataclass
+@dataclass(frozen=True)
 class BedrockStatusResponse(BaseStatusResponse):
     """The response object for :meth:`BedrockServer.status() <mcstatus.server.BedrockServer.status>`."""
 
@@ -205,7 +205,7 @@ def build(cls, decoded_data: list[Any], latency: float) -> Self:
         )
 
 
-@dataclass
+@dataclass(frozen=True)
 class BaseStatusPlayers(ABC):
     """Class for storing information about players on the server."""
 
@@ -215,7 +215,7 @@ class BaseStatusPlayers(ABC):
     """The maximum allowed number of players (aka server slots)."""
 
 
-@dataclass
+@dataclass(frozen=True)
 class JavaStatusPlayers(BaseStatusPlayers):
     """Class for storing information about players on the server."""
 
@@ -252,12 +252,12 @@ def build(cls, raw: RawJavaResponsePlayers) -> Self:
         )
 
 
-@dataclass
+@dataclass(frozen=True)
 class BedrockStatusPlayers(BaseStatusPlayers):
     """Class for storing information about players on the server."""
 
 
-@dataclass
+@dataclass(frozen=True)
 class JavaStatusPlayer:
     """Class with information about a single player."""
 
@@ -284,7 +284,7 @@ def build(cls, raw: RawJavaResponsePlayer) -> Self:
         return cls(name=raw["name"], id=raw["id"])
 
 
-@dataclass
+@dataclass(frozen=True)
 class BaseStatusVersion(ABC):
     """A class for storing version information."""
 
@@ -301,7 +301,7 @@ class BaseStatusVersion(ABC):
     """
 
 
-@dataclass
+@dataclass(frozen=True)
 class JavaStatusVersion(BaseStatusVersion):
     """A class for storing version information."""
 
@@ -318,7 +318,7 @@ def build(cls, raw: RawJavaResponseVersion) -> Self:
         return cls(name=raw["name"], protocol=raw["protocol"])
 
 
-@dataclass
+@dataclass(frozen=True)
 class BedrockStatusVersion(BaseStatusVersion):
     """A class for storing version information."""
 

diff --git a/tests/status_response/test_java.py b/tests/status_response/test_java.py
@@ -89,11 +89,10 @@ def build(self) -> JavaStatusPlayer:
         return JavaStatusPlayer.build({"name": "foo", "id": "0b3717c4-f45d-47c8-b8e2-3d9ff6f93a89"})
 
     def test_id_field_the_same_as_uuid(self) -> None:
-        build = Jav
6A8B
aStatusPlayer.build({"name": "foo", "id": "0b3717c4-f45d-47c8-b8e2-3d9ff6f93a89"})
+        unique = object()
+        build = JavaStatusPlayer.build({"name": "foo", "id": unique})  # type: ignore[assignment]
         assert build.id is build.uuid
-
-        build.id = unique = object()  # type: ignore[assignment]
-        assert unique is build.uuid
+        assert build.uuid is unique
 
 
 @BaseStatusResponseTest.construct