From d7371992996f4c8aa6105fa990c35e82c8e7a38d Mon Sep 17 00:00:00 2001
From: Corey Johnson <cj@github.com>
Date: Wed, 25 Jul 2012 15:13:46 -0700
Subject: [PATCH] Deactivate the root whew instead of deactivating the tree
 view

---
 spec/extensions/tree-view-spec.coffee | 36 +++++++++++++--------------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/spec/extensions/tree-view-spec.coffee b/spec/extensions/tree-view-spec.coffee
index 312f43f8d..1d1b1bfab 100644
--- a/spec/extensions/tree-view-spec.coffee
+++ b/spec/extensions/tree-view-spec.coffee
@@ -22,8 +22,7 @@ describe "TreeView", ->
     expect(treeView.root.directory.subscriptionCount()).toBeGreaterThan 0
 
   afterEach ->
-    treeView.deactivate()
-    rootView.remove()
+    rootView.deactivate()
 
   describe ".initialize(project)", ->
     it "renders the root of the project and its contents alphabetically with subdirectories first in a collapsed state", ->
@@ -49,7 +48,7 @@ describe "TreeView", ->
 
     describe "when the project has no path", ->
       beforeEach ->
-        treeView.deactivate()
+        rootView.deactivate()
 
         rootView = new RootView
         rootView.activateExtension(TreeView)
@@ -73,10 +72,11 @@ describe "TreeView", ->
         expect(oldRoot.hasParent()).toBeFalsy()
 
   describe "serialization", ->
-    newTreeView = null
+    [newRootView, newTreeView] = []
 
     afterEach ->
-      newTreeView.deactivate()
+      newRootView.deactivate()
+
   describe "when the prototypes deactivate method is called", ->
     it "calls the deactivate on tree view instance", ->
       spyOn(treeView, "deactivate").andCallThrough()
@@ -87,6 +87,8 @@ describe "TreeView", ->
       treeView.find('.directory:contains(zed)').click()
       sampleJs.click()
       newRootView = RootView.deserialize(rootView.serialize())
+      rootView.deactivate() # Deactivates previous TreeView
+
       newRootView.activateExtension(TreeView)
 
       newTreeView = newRootView.find(".tree-view").view()
@@ -94,7 +96,6 @@ describe "TreeView", ->
       expect(newTreeView).toExist()
       expect(newTreeView.selectedEntry()).toMatchSelector(".file:contains(sample.js)")
       expect(newTreeView.find(".directory:contains(zed)")).toHaveClass("expanded")
-      newRootView.remove()
 
     it "restores the focus state of the tree view", ->
       rootView.attachToDom()
@@ -102,13 +103,13 @@ describe "TreeView", ->
       expect(treeView).toMatchSelector ':focus'
 
       newRootView = RootView.deserialize(rootView.serialize())
-      rootView.remove()
+      rootView.deactivate() # Deactivates previous TreeView
+
       newRootView.attachToDom()
       newRootView.activateExtension(TreeView)
 
       newTreeView = newRootView.find(".tree-view").view()
       expect(newTreeView).toMatchSelector ':focus'
-      newRootView.remove()
 
   describe "when tree-view:toggle is triggered on the root view", ->
     beforeEach ->
@@ -440,7 +441,7 @@ describe "TreeView", ->
     [dirView, fileView, rootDirPath, dirPath, filePath] = []
 
     beforeEach ->
-      treeView.deactivate()
+      rootView.deactivate()
 
       rootDirPath = "/tmp/atom-tests"
       fs.remove(rootDirPath) if fs.exists(rootDirPath)
@@ -453,8 +454,8 @@ describe "TreeView", ->
 
       rootView = new RootView(rootDirPath)
       project = rootView.project
-      treeView = new TreeView(rootView)
-      treeView.root = treeView.root
+      rootView.activateExtension(TreeView)
+      treeView = rootView.find(".tree-view").view()
       dirView = treeView.root.entries.find('.directory:contains(test-dir)').view()
       dirView.expand()
       fileView = treeView.find('.file:contains(test-file.txt)').view()
@@ -598,15 +599,14 @@ describe "TreeView", ->
         describe "when the path is changed and confirmed", ->
           describe "when all the directories along the new path exist", ->
             it "moves the file, updates the tree view, and closes the dialog", ->
-              runs ->
-                newPath = fs.join(rootDirPath, 'renamed-test-file.txt')
-                moveDialog.miniEditor.setText(newPath)
+              newPath = fs.join(rootDirPath, 'renamed-test-file.txt')
+              moveDialog.miniEditor.setText(newPath)
 
-                moveDialog.trigger 'tree-view:confirm'
+              moveDialog.trigger 'tree-view:confirm'
 
-                expect(fs.exists(newPath)).toBeTruthy()
-                expect(fs.exists(filePath)).toBeFalsy()
-                expect(moveDialog.parent()).not.toExist()
+              expect(fs.exists(newPath)).toBeTruthy()
+              expect(fs.exists(filePath)).toBeFalsy()
+              expect(moveDialog.parent()).not.toExist()
 
               waitsFor "tree view to update", ->
                 treeView.root.find('> .entries > .file:contains(renamed-test-file.txt)').length > 0