Fix the bug that compare to clipboard is not working

* Give a default value ([]) if line range is not given

Author: ryu1kn

lib/selection-info-registry.js

@@ -9,7 +9,7 @@ class SelectionInfoRegistry {
         this._data[key] = {
             text: textInfo.text,
             fileName: textInfo.fileName,
-            lineRanges: textInfo.lineRanges
+            lineRanges: textInfo.lineRanges || []
         };
     }
 

test/lib/selection-info-registry.test.js

@@ -3,19 +3,34 @@ const SelectionInfoRegistry = require('../../lib/selection-info-registry');
 
 suite('SelectionInfoRegistry', () => {
 
-    test('it saves a given text selection information with a given key', () => {
+    test('it saves selected text with a given key', () => {
         const selectionInfoRegistry = new SelectionInfoRegistry();
-        selectionInfoRegistry.set('KEY', {
-            text: 'TEXT',
-            fileName: 'FILE_NAME',
-            lineRanges: 'LINE_RANGES',
-            '..': '..'
-        });
-        expect(selectionInfoRegistry.get('KEY')).to.eql({
-            text: 'TEXT',
-            fileName: 'FILE_NAME',
-            lineRanges: 'LINE_RANGES'
-        });
+        selectionInfoRegistry.set('KEY', {text: 'TEXT'});
+        expect(selectionInfoRegistry.get('KEY').text).to.include('TEXT');
+    });
+
+    test('it saves the file name of the selected text with a given key', () => {
+        const selectionInfoRegistry = new SelectionInfoRegistry();
+        selectionInfoRegistry.set('KEY', {fileName: 'FILE_NAME'});
+        expect(selectionInfoRegistry.get('KEY').fileName).to.eql('FILE_NAME');
+    });
+
+    test('it saves the line ranges of the selected text with a given key', () => {
+        const selectionInfoRegistry = new SelectionInfoRegistry();
+        selectionInfoRegistry.set('KEY', {lineRanges: 'LINE_RANGES'});
+        expect(selectionInfoRegistry.get('KEY').lineRanges).to.eql('LINE_RANGES');
+    });
+
+    test('it ignores other attributes given in text info', () => {
+        const selectionInfoRegistry = new SelectionInfoRegistry();
+        selectionInfoRegistry.set('KEY', {SOMETHING: '..'});
+        expect(selectionInfoRegistry.get('KEY').SOMETHING).to.be.undefined;
+    });
+
+    test('it sets an empty list for line ranges if they are not given', () => {
+        const selectionInfoRegistry = new SelectionInfoRegistry();
+        selectionInfoRegistry.set('KEY', {});
+        expect(selectionInfoRegistry.get('KEY').lineRanges).to.eql([]);
     });
 
 });