add tests for yank scrolling

Author: jacekkopecky

spec/operators-spec.coffee

@@ -745,8 +745,9 @@ describe "Operators", ->
 
   describe "the y keybinding", ->
     beforeEach ->
-      editor.getBuffer().setText("012 345\nabc\n")
+      editor.getBuffer().setText("012 345\nabc\ndefg\n")
       editor.setCursorScreenPosition([0, 4])
+      vimState.setRegister('"', text: '345')
 
     describe "when selected lines in visual linewise mode", ->
       beforeEach ->
@@ -857,6 +858,18 @@ describe "Operators", ->
       it "leaves the cursor at the starting position", ->
         expect(editor.getCursorScreenPosition()).toEqual [0, 4]
 
+    describe "with an up motion", ->
+      beforeEach ->
+        editor.setCursorScreenPosition([2, 2])
+        keydown 'y'
+        keydown 'k'
+
+      it "saves both full lines to the default register", ->
+        expect(vimState.getRegister('"').text).toBe "abc\ndefg\n"
+
+      it "puts the cursor on the first line and the original column", ->
+        expect(editor.getCursorScreenPosition()).toEqual [1, 2]
+
     describe "when followed by a G", ->
       beforeEach ->
         originalText = "12345\nabcde\nABCDE"
@@ -913,6 +926,48 @@ describe "Operators", ->
         expect(vimState.getRegister('"').text).toBe '123'
         expect(editor.getCursorBufferPositions()).toEqual [[0, 0], [1, 2]]
 
+    describe "in a long file", ->
+      beforeEach ->
+        editor.setHeight(400)
+        editor.setLineHeightInPixels(10)
+        editor.setDefaultCharWidth(10)
+        text = ""
+        for i in [1..200]
+          text += "#{i}\n"
+        editor.setText(text)
+
+      describe "yanking many lines forward", ->
+        it "does not scroll the window", ->
+          editor.setCursorBufferPosition [40, 1]
+          top40 = editor.getScrollTop()
+
+          # yank many lines
+          keydown('y')
+          keydown('1')
+          keydown('6')
+          keydown('0')
+          keydown('G', shift: true)
+
+          expect(editor.getScrollTop()).toEqual(top40)
+          expect(editor.getCursorBufferPosition()).toEqual [40, 1]
+          expect(vimState.getRegister('"').text.split('\n').length).toBe 121
+
+      describe "yanking many lines backwards", ->
+        it "scrolls the window", ->
+          editor.setCursorBufferPosition [140, 1]
+          top140 = editor.getScrollTop()
+
+          # yank many lines
+          keydown('y')
+          keydown('6')
+          keydown('0')
+          keydown('G', shift: true)
+
+          expect(editor.getScrollTop()).toNotEqual top140
+          expect(editor.getCursorBufferPosition()).toEqual [59, 1]
+          expect(vimState.getRegister('"').text.split('\n').length).toBe 83
+
+
   describe "the yy keybinding", ->
     describe "on a single line file", ->
       beforeEach ->

spec/scroll-spec.coffee

@@ -18,6 +18,39 @@ describe "Scrolling", ->
     options.element ?= editorElement
     helpers.keydown(key, options)
 
+  describe "normal movements (sanity check for scrolling)", ->
+    beforeEach ->
+      editor.setHeight(400)
+      editor.setLineHeightInPixels(10)
+      editor.setDefaultCharWidth(10)
+      text = ""
+      for i in [1..200]
+        text += "#{i}\n"
+      editor.setText(text)
+
+    describe "the G keybinding", ->
+      it "scrolls the window", ->
+        editor.setCursorBufferPosition [0, 0]
+        top0 = editor.getScrollTop()
+
+        editor.setCursorBufferPosition [100, 0]
+        top100 = editor.getScrollTop()
+        expect(top100).toBeGreaterThan(top0)
+
+        editor.setCursorBufferPosition [101, 0]
+        top101 = editor.getScrollTop()
+        expect(top101-top100).toEqual(10)
+
+        editor.setCursorBufferPosition [161, 0]
+        top161 = editor.getScrollTop()
+        expect(top161-top100).toEqual(610)
+
+        editor.setCursorBufferPosition [0, 0]
+        expect(editor.getScrollTop()).toNotBe top101
+        editor.setCursorBufferPosition [101, 0]
+        expect(editor.getScrollTop()).toBe top101
+
+
   describe "scrolling keybindings", ->
     beforeEach ->
       editor.setText("1\n2\n3\n4\n5\n6\n7\n8\n9\n10")