Merge pull request #235 from hans-lizihan/move-to-vitest

move to vitest

Author: hans-lizihan

.github/workflows/tests.yml

@@ -13,13 +13,13 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [14.x, 16.x, 18.x]
+        node-version: [18.x, 20.x]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
     - run: npm ci
-    - run: npm test
+    - run: npm test -- run --coverage

.gitignore

@@ -1,4 +1,5 @@
 node_modules
+coverage
 
 # ide files
 .idea

lib/people.js

@@ -191,11 +191,11 @@ class MixpanelPeople extends ProfileHelpers() {
     */
     track_charge(distinct_id, amount, properties, modifiers, callback) {
         if (typeof(properties) === 'function' || !properties) {
-            callback = properties || function() {};
+            callback = properties || undefined;
             properties = {};
         } else {
             if (typeof(modifiers) === 'function' || !modifiers) {
-                callback = modifiers || function() {};
+                callback = modifiers || undefined;
                 if (properties.$ignore_time || properties.hasOwnProperty("$ip")) {
                     modifiers = {};
                     Object.keys(properties).forEach(function(key) {