chore: setup testing and refine data layer types

- Configure Vitest and add unit tests for System core

- Remove Codecov integration

- Refine types in BillboardLayer, ChartsLayer, and FormulaData

- Update CI workflow to run tests

Author: kaolayu

.github/workflows/ci.yml

@@ -25,6 +25,9 @@ jobs:
       - name: Build
         run: npm run build
 
+      - name: Test
+        run: npm run test
+
       - name: Type check
         run: npx tsc --noEmit
 

.gitignore

@@ -3,4 +3,4 @@ node_modules
 dist
 /temp
 package-lock.json
-yarn.lock
+coverage

examples/yarn.lock

@@ -21,6 +21,7 @@
     "build": "tsup src/index.ts --format cjs,esm --dts",
     "pack": "npm pack",
     "test": "vitest run",
+    "test:coverage": "vitest run --coverage",
     "lint": "eslint \"src/**/*.ts\"",
     "dev": "tsup src/index.ts --format cjs,esm --dts --watch"
   },
@@ -54,6 +55,7 @@
   },
   "sideEffects": false,
   "devDependencies": {
+    "@vitest/coverage-v8": "^1.6.1",
     "tsup": "^8.0.0",
     "typescript": "^5.0.0",
     "vitest": "^1.0.0"

package.json

@@ -2,6 +2,9 @@ import { fx } from "../../core/system/System";
 import { NodeType } from "../../core/types/NodeType";
 import { EventManager as Eve } from "../../core/events/EventManager";
 import { getCatchValue } from "../../utils/index";
+import { OperationLayerData } from "./OperationLayerData";
+import { MetadataData } from "../metadata/MetadataData";
+import { VariableValue } from "../models/VariableValue";
 
 /**
  * 元数据列表项的类型定义
@@ -22,9 +25,9 @@ export class BillboardLayer {
     /** 图层名称 */
     name: string = "";
     /** 运算层数组 */
-    operationArray: any[] = [];
+    operationArray: OperationLayerData[] = [];
     /** 元数据数组 */
-    metadataArray: any[] = [];
+    metadataArray: MetadataData[] = [];
     /** 唯一标识符 */
     id: number = 0;
 
@@ -59,7 +62,7 @@ export class BillboardLayer {
     pushOperationLayer(): void {
         if (
             fx.clickBody != null &&
-            fx.clickBody.type == NodeType.CalculationLayer
+            fx.clickBody.type == NodeType.CalculationLayer 
         ) {
             const operationLayerData = new fx.OperationLayerData(fx.clickBody);
 

src/data/layers/BillboardLayer.ts

@@ -1,6 +1,8 @@
 
 import { EventManager as Eve } from "../../core/events/EventManager";
 import { fx } from "../../core/system/System";
+import { VariableValue } from "../models/VariableValue";
+import { OperationLayerData } from "./OperationLayerData";
 
 /**
  * 图表图层类，用于管理图表相关的运算层和元数据
@@ -11,12 +13,12 @@ export class ChartsLayer {
   name: string | undefined;
 
   /** 运算层对象数组 */
-  operationArray: any[] = [];
+  operationArray: OperationLayerData[] = [];
 
   /** 元数据对象 */
-  metadata: any | null = null;
+  metadata: VariableValue | null = null;
   /** 元数据数组 */
-  metadataArray: any[] = [];
+  metadataArray: VariableValue[] = [];
 
   /** 初始单位值 */
   minValue: number = 1;
@@ -51,7 +53,7 @@ export class ChartsLayer {
     if (fx.clickBody != null) {
       this.metadata = (fx.clickBody as any).copy();
       // this.metadataArray.push(this.metadata);
-      this.metadataArray[0] = this.metadata;
+      this.metadataArray[0] = this.metadata as VariableValue;
     }
   }
 

src/data/layers/ChartsLayer.ts

@@ -1,10 +1,12 @@
-/**
+import { VariableValue } from "../models/VariableValue";
+
+/**
  * 运算层数据类，用于存储运算层的主体数据和视图信息
  * @class OperationLayerData
  */
 export class OperationLayerData {
   /** 主体数据对象 */
-  body: any;
+  body: VariableValue;
 
   /** 视图对象 */
   view: any = null;
@@ -13,7 +15,7 @@ export class OperationLayerData {
    * 创建运算层数据实例
    * @param body - 主体数据对象
    */
-  constructor(body: any) {
+  constructor(body: VariableValue) {
     this.body = body.copy();
   }
 }

src/data/layers/OperationLayerData.ts

@@ -2,6 +2,7 @@
 import { EventManager as Eve } from "../../core/events/EventManager";
 import { getCatchValue } from "../../utils/index";
 import { fx } from "../../core/system/System";
+import { BasicBody } from "../models/BasicBody";
 
 /**
  * 公式数据类，用于存储和管理公式相关信息
@@ -20,7 +21,7 @@ export class FormulaData {
   y: number | undefined;
 
   /** 公式的主体数据对象 */
-  body: any | null = null;
+  body: BasicBody | null = null;
 
   /** 公式的类型标识，固定为 "Formula" */
   type: string = "Formula";

src/data/metadata/FormulaData.ts

@@ -1,4 +1,4 @@
-// import { fx } from "../aextends/System";
+import { VariableValue } from "../models/VariableValue";
 
 /**
  * 元数据类，用于存储和管理元数据信息
@@ -8,7 +8,7 @@
  */
 export class MetadataData {
   /** 元数据的主体对象 */
-  body: any;
+  body: VariableValue;
 
   /** 元数据关联的视图对象 */
   view: any = null;
@@ -29,16 +29,16 @@ export class MetadataData {
   intervalValue: number = 0;
 
   /** 元数据的列表数据 */
-  list: any[];
+  list: any[] = [];
 
   /**
    * 创建元数据实例
    * 
    * @constructor
-   * @param {any} body - 主体数据对象
+   * @param {VariableValue} body - 主体数据对象
    * @description 初始化元数据对象，复制主体数据作为副本
    */
-  constructor(body: any) {
+  constructor(body: VariableValue) {
     this.body = body.copy();
   }
 }

src/data/metadata/MetadataData.ts

@@ -0,0 +1,46 @@
+import { describe, it, expect } from 'vitest';
+import { fx } from '../../src/core/system/System';
+
+describe('fx (System)', () => {
+  describe('Math Utilities', () => {
+    it('distance should calculate Euclidean distance', () => {
+      expect(fx.distance(0, 0, 3, 4)).toBe(5);
+      expect(fx.distance(0, 0, 1, 1)).toBeCloseTo(1.41421356);
+    });
+
+    it('lerp should interpolate between values', () => {
+      expect(fx.lerp(0, 10, 0.5)).toBe(5);
+      expect(fx.lerp(0, 10, 0)).toBe(0);
+      expect(fx.lerp(0, 10, 1)).toBe(10);
+    });
+
+    it('clamp should restrict values to range', () => {
+      expect(fx.clamp(5, 0, 10)).toBe(5);
+      expect(fx.clamp(-5, 0, 10)).toBe(0);
+      expect(fx.clamp(15, 0, 10)).toBe(10);
+    });
+
+    it('isNumber should identify numbers correctly', () => {
+      expect(fx.isNumber(123)).toBe(true);
+      expect(fx.isNumber('123')).toBe(true);
+      expect(fx.isNumber('abc')).toBe(false);
+      // Current implementation treats null as 0, hence isNumber(null) is true
+      expect(fx.isNumber(null)).toBe(true); 
+      expect(fx.isNumber(undefined)).toBe(false);
+    });
+  });
+
+  describe('Expression Evaluation', () => {
+    it('evaluateExpression should handle basic arithmetic', () => {
+      expect(fx.evaluateExpression('1 + 1')).toBe(2);
+      expect(fx.evaluateExpression('10 * 2')).toBe(20);
+      expect(fx.evaluateExpression('(2 + 3) * 4')).toBe(20);
+    });
+
+     // Test a more complex case if possible, but keep it simple for now
+    it('evaluateExpression should respect operator precedence', () => {
+        expect(fx.evaluateExpression('2 + 3 * 4')).toBe(14);
+    });
+  });
+});
+