diff --git a/spec/app/renderer-spec.coffee b/spec/app/renderer-spec.coffee index a79fa8512..f35115aed 100644 --- a/spec/app/renderer-spec.coffee +++ b/spec/app/renderer-spec.coffee @@ -1,7 +1,7 @@ Renderer = require 'renderer' Buffer = require 'buffer' -fdescribe "Renderer", -> +describe "Renderer", -> [renderer, buffer, changeHandler, tabText] = [] beforeEach -> tabText = ' ' @@ -309,9 +309,6 @@ fdescribe "Renderer", -> expect(fold1.startRow).toBe 2 expect(fold1.endRow).toBe 5 - buffer.logLines(0, 10) - renderer.logLines(0, 10) - expect(renderer.lineForRow(1).text).toBe "1" expect(renderer.lineForRow(2).text).toBe "2" expect(renderer.lineForRow(2).fold).toBe fold1 @@ -326,29 +323,32 @@ fdescribe "Renderer", -> expect(event.newRange).toEqual [[2, 0], [2, 1]] describe "when the end of the new range exceeds the end of the fold", -> - ffit "expands the fold to contain all the inserted lines", -> + it "expands the fold to contain all the inserted lines", -> buffer.change([[3, 0], [4, 0]], 'a\nb\nc\nd\n') expect(fold1.startRow).toBe 2 expect(fold1.endRow).toBe 7 - # buffer.logLines(0, 10) - # renderer.logLines(0, 10) - - # expect(renderer.lineForRow(1).text).toBe "1" - # expect(renderer.lineForRow(2).text).toBe "2" - # expect(renderer.lineForRow(2).fold).toBe fold1 - # expect(renderer.lineForRow(2).bufferDelta).toEqual [4, 0] - # expect(renderer.lineForRow(3).text).toMatch "5" - # expect(renderer.lineForRow(4).fold).toBe fold2 - # expect(renderer.lineForRow(5).text).toMatch /^9-+/ - - # expect(changeHandler).toHaveBeenCalled() - # [[event]] = changeHandler.argsForCall - # expect(event.oldRange).toEqual [[2, 0], [2, 1]] - # expect(event.newRange).toEqual [[2, 0], [2, 1]] + expect(renderer.lineForRow(1).text).toBe "1" + expect(renderer.lineForRow(2).text).toBe "2" + expect(renderer.lineForRow(2).fold).toBe fold1 + expect(renderer.lineForRow(2).bufferDelta).toEqual [6, 0] + expect(renderer.lineForRow(3).text).toMatch "5" + expect(renderer.lineForRow(4).fold).toBe fold2 + expect(renderer.lineForRow(5).text).toMatch /^9-+/ + expect(changeHandler).toHaveBeenCalled() + [[event]] = changeHandler.argsForCall + expect(event.oldRange).toEqual [[2, 0], [2, 1]] + expect(event.newRange).toEqual [[2, 0], [2, 1]] + describe "when the old range straddles the end of the fold", -> + describe "when the end of the new range precedes the end of the fold", -> + it "shortens the fold so its end matches the end of the new range", -> + fold2.destroy() + buffer.change([[3, 0], [6, 1]], 'a') + expect(fold1.startRow).toBe 2 + expect(fold1.endRow).toBe 3 describe "when the old range surrounds a fold", -> it "removes the fold and replaces the fold placeholder with the new text", -> @@ -387,9 +387,6 @@ fdescribe "Renderer", -> expect(fold1.startRow).toBe 2 expect(fold1.endRow).toBe 6 - buffer.logLines(0, 10) - console.log "================================================" - renderer.logLines(0, 10) expect(renderer.lineForRow(1).text).toBe '1a' expect(renderer.lineForRow(2).text).toBe 'b' diff --git a/src/app/fold.coffee b/src/app/fold.coffee index 6d71004a2..244a36bd0 100644 --- a/src/app/fold.coffee +++ b/src/app/fold.coffee @@ -48,8 +48,6 @@ class Fold else delta = 0 - console.log "start row delta", delta - @startRow += delta updateEndRow: (event) -> @@ -58,11 +56,8 @@ class Fold if oldRange.end.row <= @endRow delta = newRange.end.row - oldRange.end.row else if newRange.end.row <= @endRow - console.log "newRange.end.row", newRange.end.row, " - @endRow", @endRow delta = newRange.end.row - @endRow else delta = 0 - console.log "end row delta", delta - @endRow += delta diff --git a/src/app/renderer.coffee b/src/app/renderer.coffee index ac1f7c1d1..d4861ee0e 100644 --- a/src/app/renderer.coffee +++ b/src/app/renderer.coffee @@ -119,7 +119,6 @@ class Renderer newScreenLines = @buildLinesForBufferRows(newRange.start.row, newRange.end.row) @lineMap.replaceScreenRows oldScreenRange.start.row, oldScreenRange.end.row, newScreenLines newScreenRange = @screenLineRangeForBufferRange(newRange) - console.log "New Screen Range", newScreenRange.inspect() @trigger 'change', { oldRange: oldScreenRange, newRange: newScreenRange, bufferChanged: true } @@ -179,7 +178,6 @@ class Renderer @foldsById[fold.id] = fold unregisterFold: (bufferRow, fold) -> - console.log "unregistering fold", fold.id folds = @activeFolds[bufferRow] _.remove(folds, fold) delete @foldsById[fold.id]