Merge pull request #2 from PowerInside/fix_morloderex_pr

added tests and docblocks

Author: morloderex

src/vue-numeric.vue

@@ -262,6 +262,18 @@ export default {
           this.$refs.readOnly.className = this.readOnlyClass
         })
       }
+    },
+
+    /**
+     * Immediately reflect separator changes
+     * @param {String} newValue
+     * @param {String} oldValue
+     */
+    separator (newValue, oldValue) {
+        if (newValue !== oldValue) {
+            this.process(this.valueNumber)
+            this.amount = this.format(this.valueNumber)
+        }
     }
   },
 

test/specs/vue-numeric.spec.js

@@ -206,4 +206,10 @@ describe('vue-numeric.vue', () => {
     const wrapper = mount(component)
     expect(wrapper.data().total).to.equal(0)
   })
+
+  it('apply new separator immediately if it is changed', () => {
+    const wrapper = mount(VueNumeric, { propsData: { value: 2000, separator: "," } })
+    wrapper.setProps({ separator: "." })
+    expect(wrapper.data().amount).to.equal("2.000")
+  })
 })