Skip to content

Commit 92b28ee

Browse files
committed
fix: stop parsing PlayReqs
1 parent ba51ab5 commit 92b28ee

2 files changed

Lines changed: 4 additions & 7 deletions

File tree

hslog/parser.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
from aniso8601 import parse_time
66
from hearthstone.enums import (
77
BlockType, ChoiceType, FormatType, GameTag, GameType,
8-
MetaDataType, Mulligan, OptionType, PlayReq, PowerType
8+
MetaDataType, Mulligan, OptionType, PowerType
99
)
1010

1111
from . import packets, tokens
@@ -177,8 +177,6 @@ def clean_option_errors(error, error_param):
177177

178178
if error == "NONE":
179179
error = None
180-
else:
181-
error = parse_enum(PlayReq, error)
182180

183181
if not error_param:
184182
error_param = None

tests/test_parser.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@
55
import pytest
66
from aniso8601 import parse_datetime, parse_time
77
from hearthstone.enums import (
8-
CardType, ChoiceType, GameTag, OptionType,
9-
PlayReq, PlayState, PowerType, State, Step, Zone
8+
CardType, ChoiceType, GameTag, OptionType, PlayState, PowerType, State, Step, Zone
109
)
1110

1211
from hslog import LogParser, packets
@@ -303,7 +302,7 @@ def test_options_packet_with_errors(self):
303302
assert op0.id == 0
304303
assert op0.type == OptionType.END_TURN
305304
assert op0.entity is None
306-
assert op0.error == PlayReq.INVALID
305+
assert op0.error == "INVALID"
307306
assert op0.error_param is None
308307

309308
op1 = options_packet.options[1]
@@ -317,7 +316,7 @@ def test_options_packet_with_errors(self):
317316
target = op1.options[11]
318317
assert target.id == 11
319318
assert target.entity == 37
320-
assert target.error == PlayReq.REQ_TARGET_MAX_ATTACK
319+
assert target.error == "REQ_TARGET_MAX_ATTACK"
321320
assert target.error_param == 3
322321

323322
def test_options_no_option_packet(self):

0 commit comments

Comments
 (0)