Skip to content

Commit 2de3eaa

Browse files
author
Tjaz Erzen
committed
Display module does not exist error when module does not exist
1 parent 3aa65a5 commit 2de3eaa

3 files changed

Lines changed: 8 additions & 4 deletions

File tree

plain2code.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
MissingAPIKey,
2929
MissingPreviousFunctionalitiesError,
3030
MissingResource,
31+
ModuleDoesNotExistError,
3132
NetworkConnectionError,
3233
OutdatedClientVersion,
3334
PlainSyntaxError,
@@ -318,6 +319,9 @@ def main(): # noqa: C901
318319
exc_info = sys.exc_info()
319320
console.error(f"Connection error: {str(e)}\n")
320321
console.error("Please check that your internet connection is working.")
322+
except ModuleDoesNotExistError as e:
323+
exc_info = sys.exc_info()
324+
console.error(f"Module does not exist: {str(e)}\n")
321325
console.debug(f"Render ID: {run_state.render_id}")
322326
except Exception as e:
323327
exc_info = sys.exc_info()

plain_file.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
import concept_utils
1818
import file_utils
1919
import plain_spec
20-
from plain2code_exceptions import PlainSyntaxError
20+
from plain2code_exceptions import ModuleDoesNotExistError, PlainSyntaxError
2121
from plain2code_nodes import Plain2CodeIncludeTag, Plain2CodeLoaderMixin
2222

2323
RESOURCE_MARKER = "[resource]"
@@ -513,7 +513,7 @@ def parse_plain_source( # noqa: C901
513513
def read_module_plain_source(module_name: str, template_dirs: list[str]) -> str:
514514
plain_source_text = file_utils.open_from(template_dirs, module_name + PLAIN_SOURCE_FILE_EXTENSION)
515515
if plain_source_text is None:
516-
raise PlainSyntaxError(f"Module does not exist ({module_name}).")
516+
raise ModuleDoesNotExistError(f"Module does not exist ({module_name}).")
517517
return plain_source_text
518518

519519

tests/test_requires.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
import pytest
22

33
import plain_file
4-
from plain2code_exceptions import PlainSyntaxError
4+
from plain2code_exceptions import ModuleDoesNotExistError
55

66

77
def test_non_existent_require(get_test_data_path):
8-
with pytest.raises(PlainSyntaxError, match="Module does not exist"):
8+
with pytest.raises(ModuleDoesNotExistError, match="Module does not exist"):
99
plain_file.plain_file_parser("non_existent_require.plain", [get_test_data_path("data/requires")])
1010

1111

0 commit comments

Comments
 (0)