Merge pull request #58 from opendatacube/fix-s2

fix s2 notebook

docker-compose.yml

@@ -30,7 +30,6 @@ services:
     volumes:
       - ./notebooks:/notebooks
       - ./tests:/tests
-      - ./utils:/utils
       - ./products.csv:/products.csv
     restart: always
   checkdb:

notebooks/Sentinel_2.ipynb

@@ -157,10 +157,18 @@
    "outputs": [],
    "source": [
     "import sys\n",
+    "sys.path.insert(1, '../utils/')"
+   ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
     "import datacube\n",
-    "\n",
-    "from notebooks.utils.deafrica_datahandling import load_ard\n",
-    "from notebooks.utils.deafrica_plotting import rgb"
+    "from utils.deafrica_datahandling import load_ard\n",
+    "from utils.deafrica_plotting import rgb"
    ]
   },
   {
@@ -498,7 +506,7 @@
  ],
  "metadata": {
   "kernelspec": {
-   "display_name": "Python 3 (ipykernel)",
+   "display_name": "Python 3",
    "language": "python",
    "name": "python3"
   },
@@ -512,7 +520,12 @@
    "name": "python",
    "nbconvert_exporter": "python",
    "pygments_lexer": "ipython3",
-   "version": "3.8.10"
+   "version": "3.8.15 | packaged by conda-forge | (default, Nov 22 2022, 08:49:35) \n[GCC 10.4.0]"
+  },
+  "vscode": {
+   "interpreter": {
+    "hash": "36f1ec0750f296e3257d4c290a6c8ed00b14af3513565f9d7fc44fe655b3805b"
+   }
   },
   "widgets": {
    "application/vnd.jupyter.widget-state+json": {

tests/test_notebooks.py

@@ -18,8 +18,14 @@ def test_esri_land_cover(tb):
 def test_nasadem(tb):
     assert True  # ok
 
-@pytest.mark.skip()
-@pytest.mark.xfail(reason="Index error")
-@testbook(f'{NB_DIR}/notebooks/Sentinel_2.ipynb', execute=True, timeout=180)
+@testbook(f'{NB_DIR}/notebooks/Sentinel_2.ipynb',timeout=180)
 def test_sentinel_2(tb):
+    tb.execute_cell(1)
+    tb.inject(
+        """
+        import sys
+        sys.path.insert(1, '/notebooks/')
+        """
+    )
+    tb.execute_cell(slice(3, 10))
     assert True  # ok