Fix RangeSet for validated_ledger as single ledger

XRPLF · Aug 4, 2015 · 9f9e76f · 9f9e76f
1 parent 5b51db1
commit 9f9e76f
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 1 deletion.
diff --git a/src/core/rangeset.js b/src/core/rangeset.js
@@ -45,7 +45,7 @@ class RangeSet {
     const rangeStrings = rangesString.split(',');
     _.forEach(rangeStrings, rangeString => {
       const range = rangeString.split('-').map(Number);
-      this.addRange(range[0], range[1]);
+      this.addRange(range[0], range.length === 1 ? range[0] : range[1]);
     });
   }
 

diff --git a/test/rangeset-test.js b/test/rangeset-test.js
@@ -26,6 +26,31 @@ describe('RangeSet', function() {
     r.parseAndAddRanges('4-5,7-10,1-2,3-3');
     assert.deepEqual(r.serialize(), '1-5,7-10');
   });
+  it('parseAndAddRanges() -- single ledger', function() {
+    const r = new RangeSet();
+
+    r.parseAndAddRanges('3');
+    assert.strictEqual(r.serialize(), '3-3');
+    assert(r.containsValue(3));
+    assert(!r.containsValue(0));
+    assert(!r.containsValue(2));
+    assert(!r.containsValue(4));
+    assert(r.containsRange(3, 3));
+    assert(!r.containsRange(2, 3));
+    assert(!r.containsRange(3, 4));
+
+    r.parseAndAddRanges('1-5');
+    assert.strictEqual(r.serialize(), '1-5');
+    assert(r.containsValue(3));
+    assert(r.containsValue(1));
+    assert(r.containsValue(5));
+    assert(!r.containsValue(6));
+    assert(!r.containsValue(0));
+    assert(r.containsRange(1, 5));
+    assert(r.containsRange(2, 4));
+    assert(!r.containsRange(1, 6));
+    assert(!r.containsRange(0, 3));
+  });
 
   it('containsValue()', function() {
     const r = new RangeSet();