assure product is list. Add test

mapgen/modules/satellite_satpy_quicklook.py

@@ -59,6 +59,8 @@ def _get_satpy_products(satpy_products, full_request, default_dataset):
                         ms_satpy_products = full_request['LAYER']
                     except KeyError:
                         pass
+    if not isinstance(ms_satpy_products, list):
+        ms_satpy_products = [ms_satpy_products]
     return ms_satpy_products
 
 def _get_mapfiles_path(regexp_pattern_module):

tests/test_satellite_satpy_quicklook.py

@@ -0,0 +1,25 @@
+"""Test arome arctic"""
+import os
+import logging
+import datetime
+import xarray as xr
+import numpy as np
+import pandas as pd
+from unittest.mock import patch, MagicMock
+from mapgen.modules.get_quicklook import get_quicklook
+
+def test_get_satpy_products_is_list():
+    from mapgen.modules.satellite_satpy_quicklook import _get_satpy_products
+    assert isinstance(_get_satpy_products(None,{'layers': 'overview'}, 'overview'), list)
+    assert isinstance(_get_satpy_products(None,{'layers': ['overview']}, 'overview'), list)
+    assert isinstance(_get_satpy_products(None,{'LAYERS': 'overview'}, 'overview'), list)
+    assert isinstance(_get_satpy_products(None,{'LAYERS': ['overview']}, 'overview'), list)
+
+    assert isinstance(_get_satpy_products(None,{'layer': 'overview'}, 'overview'), list)
+    assert isinstance(_get_satpy_products(None,{'layer': ['overview']}, 'overview'), list)
+    assert isinstance(_get_satpy_products(None,{'LAYER': 'overview'}, 'overview'), list)
+    assert isinstance(_get_satpy_products(None,{'LAYER': ['overview']}, 'overview'), list)
+
+    assert isinstance(_get_satpy_products(None,{'NA': 'overview'}, 'overview'), list)
+
+    assert isinstance(_get_satpy_products('overview',{'NA': 'overview'}, 'overview'), list)