Add more tests, fix boundary bug

src/Buffer.test.ts

@@ -89,9 +89,19 @@ describe('Buffer', () => {
       });
       it('should return a range for the last row', () => {
         buffer.fillViewportRows();
-       (<any> buffer.lines.get(23)).isWrapped = true;
+        (<any> buffer.lines.get(23)).isWrapped = true;
         assert.deepEqual(buffer.getWrappedRangeForLine(buffer.lines.length - 1), { first: 22, last: 23 });
       });
+      it('should return a range for a row that wraps upward to first row', () => {
+        buffer.fillViewportRows();
+        (<any> buffer.lines.get(1)).isWrapped = true;
+        assert.deepEqual(buffer.getWrappedRangeForLine(1), { first: 0, last: 1 });
+      });
+      it('should return a range for a row that wraps downward to last row', () => {
+        buffer.fillViewportRows();
+        (<any> buffer.lines.get(buffer.lines.length - 1)).isWrapped = true;
+        assert.deepEqual(buffer.getWrappedRangeForLine(buffer.lines.length - 2), { first: 22, last: 23 });
+      });
     });
   });
 

src/Buffer.ts

@@ -267,7 +267,7 @@ export class Buffer implements IBuffer {
       first--;
     }
     // Scan downwards for wrapped lines
-    while (last + 1 < this.lines.length  - 1 && (<any>this.lines.get(last + 1)).isWrapped) {
+    while (last + 1 < this.lines.length && (<any>this.lines.get(last + 1)).isWrapped) {
       last++;
     }
     return { first, last };

src/SelectionManager.test.ts

@@ -298,6 +298,16 @@ describe('SelectionManager', () => {
       assert.deepEqual(selectionManager.model.finalSelectionStart, [0, 0]);
       assert.deepEqual(selectionManager.model.finalSelectionEnd, [terminal.cols, 0], 'The actual selection spans the entire column');
     });
+    it('should select the entire wrapped line', () => {
+      buffer.lines.set(0, stringToRow('foo'));
+      const line2 = stringToRow('bar');
+      (<any>line2).isWrapped = true;
+      buffer.lines.set(1, line2);
+      selectionManager.selectLineAt(0);
+      assert.equal(selectionManager.selectionText, 'foobar', 'The selected text is correct');
+      assert.deepEqual(selectionManager.model.finalSelectionStart, [0, 0]);
+      assert.deepEqual(selectionManager.model.finalSelectionEnd, [terminal.cols, 1], 'The actual selection spans the entire column');
+    });
   });
 
   describe('selectAll', () => {