include module name in unsupported-resource error message

When a .plain file references a binary file, the error now names the
module the reference came from, e.g. "Referenced resource 'icon.png'
in module 'main' is a binary file."

Author: pedjaradenkovic

file_utils.py

@@ -223,7 +223,7 @@ def open_from(dirs, file_name):
     return None
 
 
-def load_linked_resources(template_dirs: list[str], resources_list):
+def load_linked_resources(template_dirs: list[str], resources_list, module_name: str):
     linked_resources = {}
 
     for resource in resources_list:
@@ -235,7 +235,7 @@ def load_linked_resources(template_dirs: list[str], resources_list):
             content = open_from(template_dirs, file_name)
         except UnicodeDecodeError:
             raise UnsupportedResourceType(
-                f"Referenced resource '{file_name}' is a binary file. "
+                f"Referenced resource '{file_name}' in module '{module_name}' is a binary file. "
                 f"Only text files (e.g. .md, .txt, .json, .yaml) can be referenced from a .plain file."
             )
 

render_machine/render_context.py

@@ -85,7 +85,7 @@ def __init__(
 
         resources_list = []
         plain_spec.collect_linked_resources(plain_source_tree, resources_list, None, True)
-        self.all_linked_resources = file_utils.load_linked_resources(template_dirs, resources_list)
+        self.all_linked_resources = file_utils.load_linked_resources(template_dirs, resources_list, module_name)
 
         # Initialize context objects
         self.frid_context: Optional[FridContext] = None

tests/test_file_utils.py

@@ -18,7 +18,7 @@ def test_load_linked_resources_text_file(template_dir):
     with open(file_path, "w") as f:
         f.write("# hello")
 
-    result = load_linked_resources([template_dir], [{"text": "Notes", "target": "notes.md"}])
+    result = load_linked_resources([template_dir], [{"text": "Notes", "target": "notes.md"}], "my_thing")
 
     assert result == {"notes.md": "# hello"}
 
@@ -29,12 +29,13 @@ def test_load_linked_resources_binary_file_raises_unsupported_resource_type(temp
         f.write(b"\x89PNG\r\n\x1a\n\x00\x00\x00\rIHDR\xff\xfe\xfd")
 
     with pytest.raises(UnsupportedResourceType) as exc_info:
-        load_linked_resources([template_dir], [{"text": "Icon", "target": "icon.png"}])
+        load_linked_resources([template_dir], [{"text": "Icon", "target": "icon.png"}], "my_thing")
 
     assert "icon.png" in str(exc_info.value)
     assert "binary file" in str(exc_info.value)
+    assert "my_thing" in str(exc_info.value)
 
 
 def test_load_linked_resources_missing_file_raises_file_not_found(template_dir):
     with pytest.raises(FileNotFoundError):
-        load_linked_resources([template_dir], [{"text": "Missing", "target": "missing.md"}])
+        load_linked_resources([template_dir], [{"text": "Missing", "target": "missing.md"}], "my_thing")