Fix Multi-Change Event Errors (#381)

* Fix Multi-Change Event Errors

An on-change event with multiple operations would throw an error. Examples of these operations include: `Ctrl + Z`, `Text Drag`, `Multi-line comment`, etc.

* Update Monaco Adapter Unit Tests

Update Monaco Adapter unit tests to reflect code changes

Author: BrChung

lib/monaco-adapter.js

@@ -304,7 +304,7 @@ var MonacoAdapter = function () {
     /** Change Informations */
     var text = change.text;
     var rangeLength = change.rangeLength;
-    var rangeOffset = change.rangeOffset + offset;
+    var rangeOffset = change.rangeOffset;
 
     /** Additional SEEK distance */
     var restLength = content.length + offset - rangeOffset;
@@ -373,8 +373,8 @@ var MonacoAdapter = function () {
         this.trigger('change', op, op);
       }
 
-      /** Reverse iterate all changes */
-      event.changes.reverse().forEach(function (change) {
+      /** Iterate through all changes */
+      event.changes.forEach(function (change) {
         var pair = _this.operationFromMonacoChanges(change, content, offset);
         offset += pair[0].targetLength - pair[0].baseLength;
 

test/specs/monaco-ops.spec.js

@@ -20,22 +20,22 @@ describe('Monaco Operations Test', function () {
 
     /** Editor Changes */
     var operations = [
-        { rangeLength: 0, text: ' */', rangeOffset: 299, forceMoveMarkers: false },
-        { rangeLength: 0, text: '/* ', rangeOffset: 21, forceMoveMarkers: false }
+        { rangeLength: 0, text: '/* ', rangeOffset: 21, forceMoveMarkers: false },
+        { rangeLength: 0, text: ' */', rangeOffset: 299, forceMoveMarkers: false }
     ];
 
     /** Expected Text Operations */
     var textOperations = [
         new firepad.TextOperation().retain(21).insert('/* ').retain(281),
-        new firepad.TextOperation().retain(302).insert(' */').retain(3)
+        new firepad.TextOperation().retain(299).insert(' */').retain(6)
     ];
 
     it('should convert Monaco Editor changes to Text Operation', function () {
         var MonacoAdapter = firepad.MonacoAdapter;
         var operationFromMonacoChange = MonacoAdapter.prototype.operationFromMonacoChanges;
 
         let offset = 0;
-        operations.reverse().forEach((operation, index) => {
+        operations.forEach((operation, index) => {
             var pair = operationFromMonacoChange.call(null, operation, editorContent, offset);
 
             /** Base Length of First Operation must be Target Length of Second Operation */