added vitest for testing

Author: dipaksarkar

.vscode/settings.json

@@ -0,0 +1,4 @@
+{
+    "editor.formatOnSave": true,
+    "editor.formatOnSaveMode": "modificationsIfAvailable"
+}

package.json

@@ -23,29 +23,37 @@
     "build": "tsc --emitDeclarationOnly --skipLibCheck --declaration --outDir .temp/types && rollup --c rollup.config.js",
     "docs:build": "vuepress build docs",
     "docs:dev": "vuepress dev docs",
-    "test": "jest tests/*",
+    "test": "vitest",
+    "coverage": "vitest run --coverage",
     "lint": "eslint 'src/**/*.{js,vue}'",
     "push": "clear && git config core.ignorecase false && branch=\"$(git symbolic-ref -q HEAD)\" || \"dev\" && branch=${branch##refs/heads/} && branch=${branch:-HEAD} && echo Pushing to Branch \"$branch\" && echo Please type your commit message && read msg && clear && git add . && git commit -m \"$msg\" && git push origin \"$branch\""
   },
   "devDependencies": {
+    "@rollup/plugin-node-resolve": "^15.0.1",
     "@rushstack/eslint-patch": "^1.2.0",
+    "@types/jest": "^29.4.0",
     "@typescript-eslint/eslint-plugin": "^5.38.1",
     "@typescript-eslint/parser": "^5.38.1",
-    "@rollup/plugin-node-resolve": "^15.0.1",
     "@vue/eslint-config-prettier": "^7.0.0",
     "@vue/eslint-config-typescript": "^11.0.2",
-    "prettier": "^2.7.1",
-    "rimraf": "^3.0.2",
+    "@vue/test-utils": "^1.3.0",
     "eslint": "^8.34.0",
     "eslint-config-prettier": "^8.5.0",
     "eslint-plugin-vue": "^9.9.0",
+    "jsdom": "latest",
     "lint-staged": "^13.1.2",
+    "prettier": "^2.7.1",
+    "rimraf": "^3.0.2",
     "rollup": "^2.6.1",
-    "rollup-plugin-dts": "^5.1.0",
+    "rollup-plugin-dts": "^4.2.2",
     "rollup-plugin-filesize": "^9.1.2",
     "rollup-plugin-typescript2": "^0.34.1",
     "typescript": "^4.9.5",
-    "vue": "^2.6.0"
+    "vite": "^4.1.1",
+    "vite-plugin-vue2": "^1.9.3",
+    "vitest": "^0.28.5",
+    "vue": "^2.6.0",
+    "vue-template-compiler": "^2.6.0"
   },
   "bugs": {
     "url": "https://github.com/coders-tm/vue-number-format/issues"

rollup.config.js

@@ -1,7 +1,7 @@
-import typescript from 'rollup-plugin-typescript2';
-import filesize from 'rollup-plugin-filesize';
-import resolve from '@rollup/plugin-node-resolve';
-import dts from 'rollup-plugin-dts';
+import typescript from 'rollup-plugin-typescript2'
+import filesize from 'rollup-plugin-filesize'
+import resolve from '@rollup/plugin-node-resolve'
+import dts from 'rollup-plugin-dts'
 import pkg from './package.json'
 
 const banner = `/**
@@ -10,7 +10,6 @@ const banner = `/**
  * @license ${pkg.license}
  */`
 
-
 export default [
   {
     input: 'src/index.ts',
@@ -46,4 +45,4 @@ export default [
     output: [{ file: 'dist/index.d.ts', format: 'es' }],
     plugins: [dts()]
   }
-];
+]

src/component.ts

@@ -1,5 +1,4 @@
-
-import Vue from 'vue';
+import Vue from 'vue'
 import directive from './directive'
 import { cloneDeep, CustomInputEvent, Input } from './core'
 import defaultOptions from './options'

src/core.ts

@@ -110,11 +110,8 @@ export function updateCursor(el: HTMLInputElement, position: number) {
  * @param {Boolean} options.force Forces the update even if the old value and the new value are the same
  */
 export function updateValue(el: CustomInputElement, vnode: VNode | null, { emit = true, force = false, clean = false } = {}) {
-  console.log(vnode);
   const { options, oldValue } = el
-
-  // TODO: let currentValue = vnode && vnode.data.model ? vnode.data.model.value : el.value
-  const currentValue = el.value
+  const currentValue = vnode && vnode.data && vnode.data.domProps ? vnode.data.domProps.value : el.value
 
   if (force || oldValue !== currentValue) {
     const number = new NumberFormat(options).clean(clean && !options.reverseFill)

src/number-format.ts

@@ -72,6 +72,9 @@ export default class NumberFormat {
   }
 
   sign() {
+    if (this.input === null || this.input === undefined) {
+      return ''
+    }
     const hasMinus = this.input.toString().indexOf('-') >= 0
     if (this.isClean) {
       return hasMinus && this.realNumber() > 0 ? '-' : ''
@@ -150,7 +153,7 @@ export default class NumberFormat {
     if (this.isNull() && !this.options.reverseFill) {
       return this.options.nullValue
     }
-    return [this.sign(), this.options.prefix, this.addSeparator(), this.options.suffix].join('')
+    return this.sign() + this.options.prefix + this.addSeparator() + this.options.suffix
   }
 
   /**

tests/number-format.spec.ts

@@ -0,0 +1,17 @@
+import NumberFormat from '../src/number-format'
+
+test('test number format', async () => {
+  const number = new NumberFormat({
+    prefix: '$',
+    suffix: '',
+    separator: ',',
+    decimal: '.',
+    precision: 2,
+    minimumFractionDigits: 1,
+    prefill: true,
+    reverseFill: false,
+    nullValue: ''
+  })
+
+  expect(number.format('458.869')).toBe('$458.87')
+})

tests/unit/directive.spec.js

@@ -0,0 +1,58 @@
+import NumberFormat from '../../src/number-format'
+
+test('when the value is invalid with custom config', () => {
+  const numberFormat = new NumberFormat({
+    prefix: '$',
+    separator: '.',
+    decimal: ',',
+    nullValue: '0'
+  })
+  test('should return as follows', () => {
+    expect(numberFormat.format('')).toEqual('')
+    expect(numberFormat.format('foo')).toEqual('')
+    expect(numberFormat.format('-foo')).toEqual('')
+    expect(numberFormat.format('-fo,o-')).toEqual('')
+    expect(numberFormat.format('-fo.o-')).toEqual('')
+    expect(numberFormat.format('!@#$%^&*()')).toEqual('')
+  })
+  test('should return as follows', () => {
+    expect(numberFormat.clean(true).unformat('')).toEqual('')
+    expect(numberFormat.clean(true).unformat('foo')).toEqual('')
+    expect(numberFormat.clean(true).unformat('-foo')).toEqual('')
+    expect(numberFormat.clean(true).unformat('-fo.o-')).toEqual('')
+    expect(numberFormat.clean(true).unformat('!@#$%^&*()')).toEqual('')
+  })
+})
+test('format when options are custom', () => {
+  const numberFormat = new NumberFormat({
+    prefix: '$',
+    separator: '.',
+    decimal: ',',
+    nullValue: ''
+  })
+  test('format string value', () => {
+    expect(numberFormat.format('')).toEqual('')
+    expect(numberFormat.format('0')).toEqual('$0')
+    expect(numberFormat.format('0,')).toEqual('$0')
+    expect(numberFormat.format('-0,0')).toEqual('$0')
+    expect(numberFormat.format('0,10')).toEqual('$0,1')
+    expect(numberFormat.format('0,0-')).toEqual('$0')
+    expect(numberFormat.format('0,10-')).toEqual('-$0,1')
+    expect(numberFormat.format('12.345,54921')).toEqual('$12.345,55')
+    expect(numberFormat.format('--12.345,12345')).toEqual('-$12.345,12')
+    expect(numberFormat.format('12.345.54321,12945')).toEqual('$1.234.554.321,13')
+    expect(numberFormat.format('-12.345,,54321-')).toEqual('-$12.345,54')
+  })
+  test('format numerical value', () => {
+    expect(numberFormat.format(0)).toEqual('$0')
+    expect(numberFormat.format(0)).toEqual('$0')
+    expect(numberFormat.format(0.0)).toEqual('$0')
+    expect(numberFormat.format(-0.1)).toEqual('-$0,1')
+    expect(numberFormat.format(-0.0)).toEqual('$0')
+    expect(numberFormat.format(0.1)).toEqual('$0,1')
+    expect(numberFormat.format(12345.54921)).toEqual('$12.345,55')
+    expect(numberFormat.format(12345.12345)).toEqual('$12.345,12')
+    expect(numberFormat.format(12345.54321)).toEqual('$12.345,54')
+    expect(numberFormat.format(12345.54321)).toEqual('$12.345,54')
+  })
+})