Merge pull request #146 from OpenSourceBrain/test_jnml

Fix for issue with: omv all -V --engine=jNeuroML_NEURON

Author: pgleeson

.github/workflows/ci_versions.yml

@@ -44,7 +44,7 @@ jobs:
 
     - name: Run simple OMV tests
       run: |
-        omv install pyNeuroML
+        omv all -V --engine=jNeuroML_NEURON
         omv test -V utilities/tests/.test.exIzh.jnmlnetpyne.omt
 
 

omv/engines/jneuroml.py

@@ -16,7 +16,7 @@ def get_environment():
         if "JNML_HOME" in os.environ:
             jnmlhome = os.environ["JNML_HOME"]
         elif shutil.which(JNeuroMLEngine.e_name) is not None:
-            jnmlhome = Path(shutil.which(JNeuroMLEngine.e_name)).parent
+            jnmlhome = str(Path(shutil.which(JNeuroMLEngine.e_name)).parent)
         else:
             jnmlhome = ""