From f522a7686de82607a98fe2b2058413f4543dc913 Mon Sep 17 00:00:00 2001
From: bbimber <bbimber@gmail.com>
Date: Wed, 23 Jul 2025 12:36:23 -0700
Subject: [PATCH 1/2] Correct typo

---
 jbrowse/resources/views/begin.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/jbrowse/resources/views/begin.html b/jbrowse/resources/views/begin.html
index 9032801a3..50c216bb6 100644
--- a/jbrowse/resources/views/begin.html
+++ b/jbrowse/resources/views/begin.html
@@ -1,9 +1,9 @@
 The JBrowse module is part of DISCVR-Seq.  It provides a wrapper around the JBrowse Genome Browser, which lets users rapidly take data generated or uploaded into DISCRV-Seq and view it using JBrowse.
-The module ships with a version of JBrowse, meaning very little extra configuration is required in order to start using these tools.  The primary reasons we created this wrapper around JBrowse are:
+The module ships with a version of JBrowse 2, meaning very little extra configuration is required in order to start using these tools.  The primary reasons we created this wrapper around JBrowse are:
 <p></p>
 <ul>
     <li>We selected JBrowse as the browser because it is a modern web-based genome viewer, with a large support base</li>
     <li>This wrapper makes is very simple for non-technical users to take data uploaded into DISCVR-Seq and use the power of JBrowse to see your data.  While we find JBrowse easy to use, preparing samples for the browser often requires command-line tools or was otherwise beyond the level of a typical scientist</li>
     <li>For our usage, we find it important to be able to quickly view different combinations of files.  For example, after running an alignment or calling SNPs on a set of samples, you might want to view these specific samples in conjunction with a specific set of genome annotations (coding regions, regulatory regions, site of known variation, etc).  </li>
-    <li>The files createde by genome-scale projects can become large.  If we allow users to create many different JBrowse databases, we need to be careful about the amount of files this creates.  The JBrowse module does a fair amount of work behind the scenes to avoid duplicating files.  For example, if the same genome is used as the base for 100s of JBrowse sessions, there will only be one copy of the processed sequences files saved.  The same is true for all tracks that are loaded.</li>
+    <li>The files created by genome-scale projects can become large.  If we allow users to create many different JBrowse databases, we need to be careful about the amount of files this creates.  The JBrowse module does a fair amount of work behind the scenes to avoid duplicating files.  For example, if the same genome is used as the base for 100s of JBrowse sessions, there will only be one copy of the processed sequences files saved.  The same is true for all tracks that are loaded.</li>
 </ul>
\ No newline at end of file

From 6790e2d7646478da3884a93ffa4f3d748f6c389b Mon Sep 17 00:00:00 2001
From: bbimber <bbimber@gmail.com>
Date: Wed, 23 Jul 2025 12:37:30 -0700
Subject: [PATCH 2/2] Improve test logging

---
 jbrowse/src/org/labkey/jbrowse/JBrowseManager.java | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/jbrowse/src/org/labkey/jbrowse/JBrowseManager.java b/jbrowse/src/org/labkey/jbrowse/JBrowseManager.java
index f3e56434b..d727be3a8 100644
--- a/jbrowse/src/org/labkey/jbrowse/JBrowseManager.java
+++ b/jbrowse/src/org/labkey/jbrowse/JBrowseManager.java
@@ -238,7 +238,19 @@ public static class TestCase extends Assert
         public void testJBrowseCli() throws Exception
         {
             File exe = JBrowseManager.get().getJbrowseCli();
-            String output = new SimpleScriptWrapper(_log).executeWithOutput(Arrays.asList(exe.getPath(), "help"));
+            SimpleScriptWrapper wrapper = new SimpleScriptWrapper(_log);
+            wrapper.setThrowNonZeroExits(false);
+
+            String output = wrapper.executeWithOutput(Arrays.asList(exe.getPath(), "help"));
+            if (wrapper.getLastReturnCode() != 0)
+            {
+                _log.error("Non-zero exit from testJBrowseCli: " + wrapper.getLastReturnCode());
+                wrapper.getCommandsExecuted().forEach(_log::error);
+                _log.error("output: ");
+                _log.error(output);
+
+                throw new RuntimeException("Non-zero exit running testJBrowseCli: " + wrapper.getLastReturnCode());
+            }
 
             assertTrue("Malformed output", output.contains("Add an assembly to a JBrowse 2 configuration"));
         }