Skip to content

Commit 7bc65b6

Browse files
authored
Parser: raise exception instead of sys.exit() (#58)
* raise exception instead of sys.exit() * remove pylint notes
1 parent aa74f1f commit 7bc65b6

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

search_query/parser.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@
22
"""Query parser."""
33
from __future__ import annotations
44

5-
import sys
65
import typing
76

87
from search_query.constants import PLATFORM
98
from search_query.ebsco.parser import EBSCOListParser
109
from search_query.ebsco.parser import EBSCOParser
10+
from search_query.exception import QuerySyntaxError
1111
from search_query.pubmed.parser import PubmedListParser
1212
from search_query.pubmed.parser import PubmedParser
1313
from search_query.query import Query
@@ -50,8 +50,8 @@ def parse(
5050
query_list=query_str,
5151
field_general=field_general,
5252
).parse()
53-
except Exception: # pylint: disable=broad-except
54-
sys.exit(1)
53+
except QuerySyntaxError as exc:
54+
raise exc
5555
return query
5656

5757
if platform not in PARSERS: # pragma: no cover
@@ -62,8 +62,8 @@ def parse(
6262
query = parser_class(
6363
query_str, field_general=field_general
6464
).parse() # type: ignore
65-
except Exception: # pylint: disable=broad-except
66-
sys.exit(1)
65+
except QuerySyntaxError as exc:
66+
raise exc
6767

6868
return query
6969

0 commit comments

Comments
 (0)