fixup! Use enums instead of literals for comparisons

src/zino/tasks/bgpstatemonitortask.py

@@ -271,7 +271,7 @@ def _update_established_peer(self, data: BaseBGPRow, uptime: int):
 
     def _update_nonestablished_peer(self, data: BaseBGPRow, uptime: int):
         saved_bgp_peer_session = self.device_state.bgp_peers.get(data.peer_remote_address)
-        if data.peer_admin_status in ["stop", "halted"]:
+        if data.peer_admin_status in [BGPAdminStatus.STOP, BGPAdminStatus.HALTED]:
             if not saved_bgp_peer_session or saved_bgp_peer_session.admin_status != data.peer_admin_status:
                 self._bgp_admin_down(data)
                 _logger.debug(
@@ -286,7 +286,7 @@ def _update_peer_with_admin_status_start_or_running(
     ):
         if not saved_bgp_peer_session or saved_bgp_peer_session.admin_status != data.peer_admin_status:
             self._bgp_admin_up(data)
-        if not saved_bgp_peer_session or saved_bgp_peer_session.oper_state == "established":
+        if not saved_bgp_peer_session or saved_bgp_peer_session.oper_state == BGPOperState.ESTABLISHED:
             # First verify that we've been up longer than the required time before we flag it as an alert
             if uptime > TIME_BEFORE_OPER_DOWN_ALERT:
                 self._bgp_oper_down(data)