merge_rft_ertobs: validate OBSDIR input

src/subscript/merge_rft_ertobs/merge_rft_ertobs.py

@@ -1,5 +1,6 @@
 import argparse
 import logging
+from os.path import abspath, isdir
 from pathlib import Path
 from typing import Tuple
 
@@ -154,6 +155,8 @@ def get_observations(obsdir: str = "", filepattern: str = "*.obs") -> pd.DataFra
     """  # noqa
     if "/" in filepattern:
         raise ValueError("Do not include paths in filepattern")
+    if not isdir(obsdir):
+        raise ValueError(f'Observation directory "{abspath(obsdir)}" doesn\'t exist')
 
     obs_dfs = []
     for obsfilename in Path(obsdir).glob(filepattern):

tests/test_merge_rft_ertobs.py

@@ -45,6 +45,12 @@ def fixture_drogondata(tmp_path):
         os.chdir(cwd)
 
 
+def test_invalid_obsdir():
+    """Test exception when obsdir is not valid"""
+    with pytest.raises(ValueError, match="Observation directory"):
+        get_observations("/path/to/random/non-existing/directory")
+
+
 def test_get_observations(drogondata):
     """Try to parse observations"""
     dframe = get_observations("rft")