fix: replaceRange could now be implemented as an instance of the Range interface, rather than being an instance of the Range class

Author: mkslanc

src/snippets.js

@@ -483,7 +483,7 @@ var SnippetManager = function() {
         var processedSnippet = processSnippetText.call(this, editor, snippetText);
 
         var range = editor.getSelectionRange();
-        if (replaceRange && replaceRange.compareRange(range) === 0) {
+        if (replaceRange && range.compareRange(replaceRange) === 0) {
             range = replaceRange;
         }
         var end = editor.session.replace(range, processedSnippet.text);

src/snippets_test.js

@@ -328,6 +328,14 @@ module.exports = {
         this.editor.tabstopManager.tabNext();
         editor.execCommand("insertstring", ".");
         assert.equal(editor.getValue(), "qt qt qt.");
+    },
+    "test: should work as expected with object of Range interface": function () {
+        var content = "test";
+        this.editor.setValue("replace1");
+        snippetManager.insertSnippet(this.editor, content, {
+            start: {row: 0, column: 0}, end: {row: 0, column: 7}
+        });
+        assert.equal(this.editor.getValue(), "test1");
     }
 };