From 744a6c65a152dd00f1b7b8588705fa9e00569dd0 Mon Sep 17 00:00:00 2001 From: Victor Baranov Date: Fri, 23 Nov 2018 16:03:01 +0300 Subject: [PATCH] e.trim is not a function fix --- .../add-token/add-token.component.js | 3 +- test/unit/ui/add-token.spec.js | 37 +++++++++++++++++++ 2 files changed, 39 insertions(+), 1 deletion(-) diff --git a/old-ui/app/components/add-token/add-token.component.js b/old-ui/app/components/add-token/add-token.component.js index fe9425bca..dc5f11f5f 100644 --- a/old-ui/app/components/add-token/add-token.component.js +++ b/old-ui/app/components/add-token/add-token.component.js @@ -501,7 +501,8 @@ class AddTokenScreen extends Component { } handleCustomDecimalsChange = (value) => { - const customDecimals = (value && value.trim()) + let customDecimals = Number(value && value.toString().trim()) + customDecimals = isNaN(customDecimals) ? '' : customDecimals const validDecimals = customDecimals !== null && customDecimals !== '' && customDecimals >= 0 && diff --git a/test/unit/ui/add-token.spec.js b/test/unit/ui/add-token.spec.js index 81ad7d8dd..0d756291c 100644 --- a/test/unit/ui/add-token.spec.js +++ b/test/unit/ui/add-token.spec.js @@ -40,4 +40,41 @@ describe('Add Token Screen', function () { }) }) + + describe('#HandleCustomDecimalsChange', () => { + it.only('sets correct decimals', () => { + addTokenComponent.instance().handleCustomDecimalsChange(18) + const state = addTokenComponent.state() + assert.equal(state.customDecimals, 18) + assert.equal(state.customDecimalsError, null) + }) + + it.only('sets correct decimals', () => { + addTokenComponent.instance().handleCustomDecimalsChange(0) + const state = addTokenComponent.state() + assert.equal(state.customDecimals, 0) + assert.equal(state.customDecimalsError, null) + }) + + it.only('sets customDecimalsError', () => { + addTokenComponent.instance().handleCustomDecimalsChange('test') + const state = addTokenComponent.state() + assert.equal(state.customDecimals, '') + assert.equal(state.customDecimalsError, 'Decimals must be at least 0, and not over 36.') + }) + + it.only('sets customDecimalsError', () => { + addTokenComponent.instance().handleCustomDecimalsChange({}) + const state = addTokenComponent.state() + assert.equal(state.customDecimals, '') + assert.equal(state.customDecimalsError, 'Decimals must be at least 0, and not over 36.') + }) + + it.only('sets customDecimalsError', () => { + addTokenComponent.instance().handleCustomDecimalsChange() + const state = addTokenComponent.state() + assert.equal(state.customDecimals, '') + assert.equal(state.customDecimalsError, 'Decimals must be at least 0, and not over 36.') + }) + }) })