Add spacing every 11 bits to the checksum

diff --git a/src/js/index.js b/src/js/index.js
index 0d573baa88b1fe83d2cda149df21014f928afb82..b8a76a970c98e8443a1c43ed32358fa5af8f30e8 100644 (file)
--- a/src/js/index.js
+++ b/src/js/index.js
@@ -1228,7 +1228,7 @@
         var entropyTypeStr = getEntropyTypeStr(entropy);
         var wordCount = Math.floor(numberOfBits / 32) * 3;
         var bitsPerEvent = entropy.bitsPerEvent.toFixed(2);
-        var spacedBinaryStr = entropy.binaryStr.match(/.{1,11}/g).join(" " );
+        var spacedBinaryStr = addSpacesEveryElevenBits(entropy.binaryStr);
         DOM.entropyFiltered.html(entropy.cleanHtml);
         DOM.entropyType.text(entropyTypeStr);
         DOM.entropyCrackTime.text(timeToCrack);
@@ -1479,6 +1479,11 @@
                 var start = binaryStr.length - checksumBitlength;
                 var end = binaryStr.length;
                 checksum = binaryStr.substring(start, end);
+                // add spaces so the last group is 11 bits, not the first
+                checksum = checksum.split("").reverse().join("")
+                checksum = addSpacesEveryElevenBits(checksum);
+                checksum = checksum.split("").reverse().join("")
+                break;
             }
         }
         DOM.entropyChecksum.text(checksum);
@@ -1504,6 +1509,10 @@
         DOM.csv.val(tableCsv);
     }
 
+    function addSpacesEveryElevenBits(binaryStr) {
+        return binaryStr.match(/.{1,11}/g).join(" ");
+    }
+
     var networks = [
         {
             name: "AXE - Axe",

diff --git a/tests/spec/tests.js b/tests/spec/tests.js
index f56b18d0aea1192785b540c21ee31641a38e4613..c7a988944741acb70edec4422063e5febce71323 100644 (file)
--- a/tests/spec/tests.js
+++ b/tests/spec/tests.js
@@ -2937,4 +2937,21 @@ it('Shows the checksum for the entropy', function(done) {
     });
 });
 
+it('Shows the checksum for the entropy with the correct groupings', function(done) {
+    driver.findElement(By.css('.use-entropy'))
+        .click();
+    // create a checksum of 20 bits, which spans multiple words
+    driver.findElement(By.css('.entropy'))
+        .sendKeys("F000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000");
+    driver.sleep(generateDelay).then(function() {
+        driver.findElement(By.css('.checksum'))
+            .getText()
+            .then(function(text) {
+                // first group is 9 bits, second group is 11
+                expect(text).toBe("011010111 01110000110");
+                done();
+            });
+    });
+});
+
 });