Add ParsingError exception

isort/api.py

@@ -32,6 +32,7 @@
     FileSkipComment,
     FileSkipSetting,
     IntroducedSyntaxErrors,
+    ParsingError,
 )
 from .format import ask_whether_to_apply_changes_to_file, create_terminal_printer, show_unified_diff
 from .io import Empty, File
@@ -217,6 +218,9 @@ def sort_stream(
     except FileSkipComment:
         raise FileSkipComment(content_source)
 
+    except ParsingError:
+        raise ParsingError(file_path)
+
     if config.atomic:
         _internal_output.seek(0)
         try:

isort/core.py

@@ -7,7 +7,7 @@
 from isort.settings import DEFAULT_CONFIG, Config
 
 from . import output, parse
-from .exceptions import ExistingSyntaxErrors, FileSkipComment
+from .exceptions import ExistingSyntaxErrors, FileSkipComment, ISortError, ParsingError
 from .format import format_natural, remove_whitespace
 from .settings import FILE_SKIP_COMMENTS
 
@@ -417,8 +417,12 @@ def process(
                         import_section = "".join(
                             line[len(indent) :] for line in import_section.splitlines(keepends=True)
                         )
-
-                    parsed_content = parse.file_contents(import_section, config=config)
+                    try:
+                        parsed_content = parse.file_contents(import_section, config=config)
+                    except ISortError as error:
+                        raise error
+                    except Exception:
+                        raise ParsingError(file_path=None)
                     verbose_output += parsed_content.verbose_output
 
                     sorted_import_section = output.sorted_imports(

isort/exceptions.py

@@ -1,7 +1,7 @@
 """All isort specific exception classes should be defined here"""
 from functools import partial
 from pathlib import Path
-from typing import Any, Dict, List, Type, Union
+from typing import Any, Dict, List, Optional, Type, Union
 
 from .profiles import profiles
 
@@ -195,3 +195,11 @@ def __init__(self, import_module: str, section: str):
             "See https://pycqa.github.io/isort/#custom-sections-and-ordering "
             "for more info."
         )
+
+
+class ParsingError(ISortError):
+    """Raised when there's a failure with parsing the imports"""
+
+    def __init__(self, file_path: Optional[Path]):
+        super().__init__(f"isort couldn't parse imports for {file_path}")
+        self.file_path = file_path

tests/unit/test_exceptions.py

@@ -126,3 +126,11 @@ def setup_class(self):
 
     def test_variables(self):
         assert self.instance.filename == "file.py"
+
+
+class TestParsingError(TestISortError):
+    def setup_class(self):
+        self.instance: exceptions.ParsingError = exceptions.ParsingError(file_path=None)
+
+    def test_variables(self):
+        assert self.instance.file_path is None

tests/unit/test_isort.py

@@ -16,7 +16,7 @@
 
 import isort
 from isort import api, files, sections
-from isort.exceptions import ExistingSyntaxErrors, FileSkipped, MissingSection
+from isort.exceptions import ExistingSyntaxErrors, FileSkipped, MissingSection, ParsingError
 from isort.settings import Config
 from isort.utils import exists_case_sensitive
 
@@ -5671,3 +5671,16 @@ def test_reexport_not_last_line() -> None:
     meme = "rickroll"
 """
     assert isort.code(test_input, config=Config(sort_reexports=True)) == expd_output
+
+
+def test_doesnot_break_for_syntax_error() -> None:
+    """Test to ensure that isort doesn't break for syntax error while parsing import"""
+    test_input = """
+from pathlib import Path
+from typing import Optional, Union
+import numpy as np
+
+from os.path.import Path
+"""
+    with pytest.raises(ParsingError):
+        isort.code(test_input)