feat(bindgen): Check that input image files exists in Python/WASI

packages/dicom/python/itkwasm-dicom-wasi/itkwasm_dicom_wasi/apply_presentation_state_to_image.py

@@ -63,7 +63,11 @@ def apply_presentation_state_to_image(
 
     args: List[str] = ['--memory-io',]
     # Inputs
+    if not Path(image_in).exists():
+        raise FileNotFoundError("image_in does not exist")
     args.append(str(PurePosixPath(image_in)))
+    if not Path(presentation_state_file).exists():
+        raise FileNotFoundError("presentation_state_file does not exist")
     args.append(str(PurePosixPath(presentation_state_file)))
     # Outputs
     args.append('0')

packages/dicom/python/itkwasm-dicom-wasi/itkwasm_dicom_wasi/read_dicom_encapsulated_pdf.py

@@ -98,6 +98,8 @@ def read_dicom_encapsulated_pdf(
 
     args: List[str] = ['--memory-io',]
     # Inputs
+    if not Path(dicom_file).exists():
+        raise FileNotFoundError("dicom_file does not exist")
     args.append(str(PurePosixPath(dicom_file)))
     # Outputs
     args.append('0')

packages/dicom/python/itkwasm-dicom-wasi/itkwasm_dicom_wasi/structured_report_to_html.py

@@ -199,6 +199,8 @@ def structured_report_to_html(
 
     args: List[str] = ['--memory-io',]
     # Inputs
+    if not Path(dicom_file).exists():
+        raise FileNotFoundError("dicom_file does not exist")
     args.append(str(PurePosixPath(dicom_file)))
     # Outputs
     args.append('0')

packages/dicom/python/itkwasm-dicom-wasi/itkwasm_dicom_wasi/structured_report_to_text.py

@@ -106,6 +106,8 @@ def structured_report_to_text(
 
     args: List[str] = ['--memory-io',]
     # Inputs
+    if not Path(dicom_file).exists():
+        raise FileNotFoundError("dicom_file does not exist")
     args.append(str(PurePosixPath(dicom_file)))
     # Outputs
     args.append('0')

packages/dicom/python/itkwasm-dicom-wasi/test/test_apply_presentation_state_to_image.py

@@ -1,4 +1,4 @@
-from pathlib import Path 
+from pathlib import Path
 import numpy as np
 import json
 
@@ -43,3 +43,18 @@ def test_apply_presentation_state_to_dicom_image():
     # slice to get only the pixel buffer from the baseline image (pgm file)
     baseline_pixels = baseline_buffer[15:]
     assert np.array_equal(np.frombuffer(baseline_pixels, dtype=np.uint8), output_image.data.ravel())
+
+def test_apply_presentation_state_to_dicom_image_input_does_not_exist():
+    from itkwasm_dicom_wasi import apply_presentation_state_to_image
+
+    input_file = 'gsps-pstate-test-input-image-does-not-exist.dcm'
+    input_file_path = Path('..', '..', 'test', 'data', 'input', input_file)
+
+    p_state_file = 'gsps-pstate-test-input-pstate.dcm'
+    p_state_file_path = Path('..', '..', 'test', 'data', 'input', p_state_file)
+
+    try:
+        p_state_json_out, output_image = apply_presentation_state_to_image(input_file_path, p_state_file_path)
+        assert False
+    except FileNotFoundError:
+        pass

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/apply_presentation_state_to_image.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/apply_presentation_state_to_image_async.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/read_dicom_encapsulated_pdf.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/read_dicom_encapsulated_pdf_async.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/structured_report_to_html.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/structured_report_to_html_async.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/structured_report_to_text.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

packages/dicom/python/itkwasm-dicom/itkwasm_dicom/structured_report_to_text_async.py

@@ -1,7 +1,7 @@
 # Generated file. Do not edit.
 
 import os
-from typing import Dict, Tuple, Optional
+from typing import Dict, Tuple, Optional, List
 
 from itkwasm import (
     environment_dispatch,

src/bindgen/python.js

@@ -325,14 +325,18 @@ from itkwasm import (
   let inputCount = 0
   args += "    # Inputs\n"
   interfaceJson.inputs.forEach((input) => {
+    const snakeName = snakeCase(input.name)
     if (interfaceJsonTypeToInterfaceType.has(input.type)) {
       const interfaceType = interfaceJsonTypeToInterfaceType.get(input.type)
-      const name = interfaceType.includes('File') ? `str(PurePosixPath(${snakeCase(input.name)}))` : `'${inputCount.toString()}'`
+      if (interfaceType.includes('File')) {
+        args += `    if not Path(${snakeName}).exists():\n`
+        args += `        raise FileNotFoundError("${snakeName} does not exist")\n`
+      }
+      const name = interfaceType.includes('File') ? `str(PurePosixPath(${snakeName}))` : `'${inputCount.toString()}'`
       args += `    args.append(${name})\n`
       inputCount++
     } else {
-      const snake = snakeCase(input.name)
-      args += `    args.append(str(${snake}))\n`
+      args += `    args.append(str(${snakeName}))\n`
     }
   })