Change when the new Alchemy instance is pushed to the list of instanc…

…es. … The previous push location was too late for code that depended on the length of Alchemy instances. See discussion: GraphAlchemist#543 Fixes GraphAlchemist#543
st-keller · Jun 23, 2015 · 8f107bf · 8f107bf
1 parent c71ddb9
commit 8f107bf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/app/scripts/alchemy/Alchemy.coffee.md b/app/scripts/alchemy/Alchemy.coffee.md
@@ -99,12 +99,12 @@ title: Anotated Source
         begin: (userConf) ->
             # overide configuration with user inputs
             conf = @setConf userConf
+            Alchemy::instances.push @
             switch typeof @conf.dataSource
                 when 'string' then d3.json @a.conf.dataSource, @a.startGraph
                 when 'object' then @a.startGraph @a.conf.dataSource
             
             @plugins.init()
-            Alchemy::instances.push @
 
             @
 

diff --git a/test/spec/uiTests.coffee b/test/spec/uiTests.coffee
@@ -8,5 +8,6 @@ do ->
             it "should present to the browser all info necessary to render data", ->
                 d3.select('g.node').text().should.equal "AlchemyJS"
                 d3.select('g.edge').text().should.equal "Maintains"
+                alchemy.vis.attr('alchInst').should.equal "0"
 
-    return
+    return