Merge pull request #790 from LiskHQ/789-passphrase-e2e-tests

Hot fix on passphrase utility - closes #789
LiskArchive · Sep 28, 2017 · 8f27050 · 8f27050
2 parents e5d7761 + 0b9df24
commit 8f27050
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/components/passphrase/passphraseVerifier.js b/src/components/passphrase/passphraseVerifier.js
@@ -19,11 +19,11 @@ class PassphraseConfirmator extends React.Component {
   }
 
   hideRandomWord(rand = Math.random()) {
-    const words = this.props.passphrase.trim().split(/\s+/);
+    const words = this.props.passphrase.trim().split(/\s+/).filter(item => item.length > 0);
     const index = Math.floor(rand * (words.length - 1));
 
     this.setState({
-      passphraseParts: this.props.passphrase.split(` ${words[index]} `),
+      passphraseParts: this.props.passphrase.split(words[index]),
       missing: words[index],
       answer: '',
     });

diff --git a/src/components/passphrase/passphraseVerifier.test.js b/src/components/passphrase/passphraseVerifier.test.js
@@ -41,8 +41,8 @@ describe('PassphraseVerifier', () => {
       const randomIndex = 0.6;
       const expectedValues = {
         passphraseParts: [
-          'survey stereo pool fortune oblige slight',
-          'goddess mistake sentence anchor pool',
+          'survey stereo pool fortune oblige slight ',
+          ' goddess mistake sentence anchor pool',
         ],
         missing: 'gravity',
         answer: '',

diff --git a/src/utils/passphrase.js b/src/utils/passphrase.js
@@ -30,8 +30,8 @@ const leftPadd = (str, pad, length) => {
  * Resets previous settings and creates a step with a random length between 1.6% to 3.2%
  */
 const init = (rand = Math.random()) => {
-  let step = Math.max((160 + Math.floor(rand * 160)));
-  step = step >= 0.01 ? step : step * 10;
+  let step = (160 + Math.floor(rand * 160)) / 100;
+  step = step >= 0.01 ? step : 0.1 + (step * 5);
   return {
     step,
     percentage: 0,