From 6ff0811a59326739f029ec30e9d8ef5b8f16c8de Mon Sep 17 00:00:00 2001
From: mvdbeek <m.vandenbeek@gmail.com>
Date: Sat, 28 Oct 2023 20:13:36 +0200
Subject: [PATCH] Fix up tutorial test

---
 tests/test_training_tutorial.py | 38 ++++++++++++++++-----------------
 1 file changed, 18 insertions(+), 20 deletions(-)

diff --git a/tests/test_training_tutorial.py b/tests/test_training_tutorial.py
index 04d769a95..a02c23020 100644
--- a/tests/test_training_tutorial.py
+++ b/tests/test_training_tutorial.py
@@ -327,32 +327,30 @@ def test_tutorial_create_hands_on_tutorial() -> None:
     """Test :func:`planemo.training.tutorial.tutorial.create_hands_on_tutorial`."""
     tuto = Tutorial(training=training, topic=topic)
     os.makedirs(tuto.wf_dir)
-    # with init_wf_id and no Galaxy URL
-    tuto.init_wf_id = "ID"
-    tuto.training.galaxy_url = None
-    with pytest.raises(Exception, match="No Galaxy URL given"):
-        tuto.create_hands_on_tutorial(CTX)
-    # with init_wf_id and no Galaxy API key
-    tuto.init_wf_id = "ID"
-    tuto.training.galaxy_url = f"http://{KWDS['host']}:{KWDS['port']}"
-    tuto.training.galaxy_api_key = None
-    with pytest.raises(Exception, match="No API key to access the given Galaxy instance"):
-        tuto.create_hands_on_tutorial(CTX)
-    # with init_wf_id
     with engine_context(CTX, **KWDS) as galaxy_engine:
         assert isinstance(galaxy_engine, LocalManagedGalaxyEngine)
         with galaxy_engine.ensure_runnables_served([RUNNABLE]) as config:
+            # fail without Galaxy URL
+            tuto.init_wf_id = "ID"
+            tuto.training.galaxy_url = None
+            with pytest.raises(Exception, match="No Galaxy URL given"):
+                tuto.create_hands_on_tutorial(CTX)
+            # fail without Galaxy API key
+            tuto.training.galaxy_url = f"http://{KWDS['host']}:{KWDS['port']}"
             tuto.init_wf_id = config.workflow_id(WF_FP)
+            with pytest.raises(Exception, match="Workflow is not owned by or shared with current user"):
+                tuto.create_hands_on_tutorial(CTX)
+            # pass with API key
             tuto.training.galaxy_api_key = config.user_api_key
             tuto.create_hands_on_tutorial(CTX)
-    assert os.path.exists(tuto.tuto_fp)
-    os.remove(tuto.tuto_fp)
-    # with init_wf_fp
-    tuto.init_wf_id = None
-    tuto.init_wf_fp = WF_FP
-    tuto.create_hands_on_tutorial(CTX)
-    assert os.path.exists(tuto.tuto_fp)
-    shutil.rmtree("topics")
+            assert os.path.exists(tuto.tuto_fp)
+            os.remove(tuto.tuto_fp)
+            # with init_wf_fp
+            tuto.init_wf_id = None
+            tuto.init_wf_fp = WF_FP
+            tuto.create_hands_on_tutorial(CTX)
+            assert os.path.exists(tuto.tuto_fp)
+            shutil.rmtree("topics")
 
 
 @skip_if_environ("PLANEMO_SKIP_GALAXY_TESTS")