Address comment

Author: BriungRi

hyperliquid/info.py

@@ -598,22 +598,24 @@ def query_sub_accounts(self, user: str) -> Any:
     def query_user_to_multi_sig_signers(self, multi_sig_user: str) -> Any:
         return self.post("/info", {"type": "userToMultiSigSigners", "user": multi_sig_user})
 
-    def subscribe(self, subscription: Subscription, callback: Callable[[Any], None]) -> int:
+    def _validate_subscription(self, subscription: Subscription) -> None:
         if (
             subscription["type"] == "l2Book"
             or subscription["type"] == "trades"
             or subscription["type"] == "candle"
             or subscription["type"] == "bbo"
         ):
             subscription["coin"] = self.name_to_coin[subscription["coin"]]
+
+    def subscribe(self, subscription: Subscription, callback: Callable[[Any], None]) -> int:
+        self._validate_subscription(subscription)
         if self.ws_manager is None:
             raise RuntimeError("Cannot call subscribe since skip_ws was used")
         else:
             return self.ws_manager.subscribe(subscription, callback)
 
     def unsubscribe(self, subscription: Subscription, subscription_id: int) -> bool:
-        if subscription["type"] == "l2Book" or subscription["type"] == "trades" or subscription["type"] == "candle":
-            subscription["coin"] = self.name_to_coin[subscription["coin"]]
+        self._validate_subscription(subscription)
         if self.ws_manager is None:
             raise RuntimeError("Cannot call unsubscribe since skip_ws was used")
         else: