fix: propertly check input to public interface

We were depending on es6 default params to set expected values.  However default params only apply
to undefined values, and not for null.  Thus an erro was been thrown when null is been passed in as
the value or config object.

Author: RonaldJerez

src/core.js

@@ -1,8 +1,8 @@
 import masker from './masker'
 export const CONFIG_KEY = '__input-facade__'
 
-export function FacadeValue(val = '') {
-  this.masked = this.unmasked = val
+export function FacadeValue(val) {
+  this.masked = this.unmasked = val || ''
 }
 
 /**
@@ -22,12 +22,12 @@ export function FacadeInputEvent() {
  *
  * @param {object} config The mask config object
  */
-export function normalizeConfig(config = {}) {
+export function normalizeConfig(config) {
   if (Array.isArray(config) || typeof config === 'string') {
     config = { mask: config }
   }
 
-  return config
+  return config || {}
 }
 
 /**

src/masker.js

@@ -9,18 +9,19 @@ let tokenDefinitions = defaultTokens
  * @param {object} tokens the new token object
  */
 export function setTokens(tokens) {
+  if (!tokens) return
   tokenDefinitions = tokens
 }
 
 /**
  * Given an array of masks, determines which one is the appropriate one based on the value
  *
  * @param {String} inputValue the inputValue value to mask
- * @param {{masks: [String]}} config
+ * @param {object} config
  * @param {Array} config.masks the list of masks to choose from
  * @returns {FacadeValue} facade value object
  */
-export function dynamic(inputValue, config = {}) {
+export function dynamic(inputValue, config) {
   const masks = config.masks.slice().sort((a, b) => a.length - b.length)
   const withConfig = (overrides) => Object.assign({}, config, overrides)
 
@@ -53,19 +54,17 @@ export function dynamic(inputValue, config = {}) {
  *
  * @param {string} value the value to mask
  * @param {{mask: String, tokens: Object, prepend: Boolean}} config
+ * @param {object} config
  * @param {string} config.mask the masking string
  * @param {object} config.tokens the tokens to add/override to the global
  * @param {boolean} config.prepend whether or not to add masking characters to the input before the user types.
  */
-export function formatter(value = '', config = {}) {
+export function formatter(value, config) {
   let { mask = '', tokens, prepend = false } = config
 
   // append/override global tokens instead of complete override
   tokens = tokens ? Object.assign({}, tokenDefinitions, tokens) : tokenDefinitions
 
-  // ensure we have a string
-  value = value.toString()
-
   let output = new FacadeValue()
   let escaped = false
 
@@ -124,6 +123,8 @@ export function formatter(value = '', config = {}) {
  * @returns {FacadeValue} facade value object
  */
 export default function masker(value, config) {
+  // ensure we have proper input
+  value = (value || '').toString()
   config = normalizeConfig(config)
 
   // disable on empty mask

tests/formatter.test.js

@@ -1,9 +1,5 @@
 import { formatter } from '../src/masker'
 
-test('default vals', () => {
-  expect(formatter('123')).toMatchObject({ masked: '', unmasked: '' })
-})
-
 test('12 -> #.#', () => {
   expect(formatter('12', { mask: '#.#' })).toMatchObject({ masked: '1.2', unmasked: '12' })
 })

tests/masker.test.js

@@ -4,8 +4,17 @@ const tokens = {
   F: { pattern: /[a-f0-9]/i }
 }
 
-test('no mask given', () => {
+test('no errors with invalid input to public interface', () => {
+  expect(masker()).toMatchObject({ masked: '', unmasked: '' })
+  expect(masker(null)).toMatchObject({ masked: '', unmasked: '' })
+  expect(masker(null, null)).toMatchObject({ masked: '', unmasked: '' })
+  expect(masker(null, { mask: null })).toMatchObject({ masked: '', unmasked: '' })
+})
+
+test('no mask given, value should be returned as is', () => {
   expect(masker('123')).toMatchObject({ masked: '123', unmasked: '123' })
+  expect(masker('123', null)).toMatchObject({ masked: '123', unmasked: '123' })
+  expect(masker('123', { mask : null })).toMatchObject({ masked: '123', unmasked: '123' })
 })
 
 test('single mask given', () => {