Merge pull request #7 from Open-EO/hotfix

added code and test for checking file existence;

src/openeo_pg_parser_python/translate_process_graph.py

@@ -216,10 +216,12 @@ def link_nodes(graph):
 
 def translate_graph(pg_filepath):
     if isinstance(pg_filepath, str):
-        if os.path.isfile(pg_filepath):
-            pg_dict = load(open(pg_filepath))
+        pg_dict = load(open(pg_filepath))
     elif isinstance(pg_filepath, dict):
         pg_dict = pg_filepath
+    else:
+        raise ValueError("'pg_filepath must either be file path to a JSON file or a dictionary.'")
+
     nodes = OrderedDict()
     nodes, _, _, _ = walk_pg_graph(nodes, pg_dict)
     # create graph object

tests/test_translate_pg.py

@@ -0,0 +1,21 @@
+import unittest
+from openeo_pg_parser_python.translate_process_graph import translate_graph
+
+
+class PGTranslateTester(unittest.TestCase):
+    """ Responsible for testing the translation of an openEO process graph. """
+
+    def setUp(self):
+        """ Specifies paths to the test data. """
+
+        self.pg_test_1_filepath = r"process_graphs/test_1.json"
+
+    def test_pg_not_found(self):
+        pg_filepath = r"process_graphs/does_not_exist.json"
+        try:
+            translate_graph(pg_filepath)
+        except FileNotFoundError:
+            assert True
+
+if __name__ == '__main__':
+    unittest.main()