Fix for some sessions that don't instantiate sessionAssemblies e.g. b…

…are bones tests
GMOD · Feb 19, 2021 · 1956a70 · 1956a70
1 parent f576ea4
commit 1956a70
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/packages/core/assemblyManager/assemblyManager.ts b/packages/core/assemblyManager/assemblyManager.ts
@@ -32,7 +32,7 @@ export default function assemblyManagerFactory(
       get assemblyList() {
         return [
           ...getParent(self).jbrowse.assemblies.slice(),
-          ...getParent(self).session.sessionAssemblies.slice(),
+          ...(getParent(self).session.sessionAssemblies?.slice() || []),
         ]
       },
 

diff --git a/products/jbrowse-web/src/__snapshots__/sessionModelFactory.test.js.snap b/products/jbrowse-web/src/__snapshots__/sessionModelFactory.test.js.snap
@@ -7,6 +7,7 @@ Object {
   "drawerWidth": 384,
   "margin": 0,
   "name": "testSession",
+  "sessionAssemblies": Array [],
   "sessionConnections": Array [],
   "sessionTracks": Array [],
   "views": Array [],

diff --git a/products/jbrowse-web/src/sessionModelFactory.ts b/products/jbrowse-web/src/sessionModelFactory.ts
@@ -42,7 +42,7 @@ declare interface ReferringNode {
 
 export default function sessionModelFactory(
   pluginManager: PluginManager,
-  assemblyConfigSchemasType: any,
+  assemblyConfigSchemasType = types.frozen(), // if not using sessionAssemblies
 ) {
   const minDrawerWidth = 128
   return types