test: coverage up (#1889)

* test: sunburst hierarchy util

* test: scatter util

* test: sunburst hierarchy partition

__tests__/unit/plots/scatter/regression-line-spec.ts

@@ -100,7 +100,7 @@ describe('scatter', () => {
     });
 
     scatter.render();
-    await delay(500);
+    await delay(100);
     const geometry = scatter.chart.geometries[0];
     const annotation = scatter.chart.annotation();
     // @ts-ignore
@@ -150,7 +150,7 @@ describe('scatter', () => {
     });
 
     scatter.render();
-    await delay(500);
+    await delay(100);
     const { width } = scatter.chart;
     const pathGroup = scatter.chart
       .getComponents()
@@ -162,4 +162,29 @@ describe('scatter', () => {
 
     scatter.destroy();
   });
+
+  it('regressionLine: nonealgorithm', async () => {
+    const scatter = new Scatter(createDiv('regressionLine*algorithm'), {
+      data,
+      width: 400,
+      height: 300,
+      appendPadding: 10,
+      xField: 'x',
+      yField: 'y',
+      size: 5,
+      animation: false,
+      regressionLine: {},
+    });
+
+    scatter.render();
+    await delay(100);
+    const pathGroup = scatter.chart
+      .getComponents()
+      .find((item) => item.type === 'annotation')
+      .component.cfg.group.cfg.children[0].getChildren();
+    const { path } = pathGroup?.[0]?.cfg?.attrs;
+    expect(path.length).toBe(2); // linear
+
+    scatter.destroy();
+  });
 });

__tests__/unit/plots/sunburst/hierarchy/partition-spec.ts

@@ -0,0 +1,19 @@
+import { partition } from '../../../../../src/plots/sunburst/hierarchy/partition';
+
+describe('hierarchy/partition', () => {
+  it('partition', () => {
+    expect(() => {
+      // @ts-ignore
+      partition([], {
+        as: null,
+      });
+    }).toThrow(`Invalid as: it must be an array with 2 strings (e.g. [ "x", "y" ])!`);
+
+    expect(() => {
+      partition([], {
+        // @ts-ignore
+        as: ['x', 'y', 'z'],
+      });
+    }).toThrow(`Invalid as: it must be an array with 2 strings (e.g. [ "x", "y" ])!`);
+  });
+});

__tests__/unit/plots/sunburst/hierarchy/util-spec.ts

@@ -0,0 +1,83 @@
+import { getField, getAllNodes } from '../../../../../src/plots/sunburst/hierarchy/util';
+
+describe('hierarchy/util', () => {
+  it('getField', () => {
+    expect(
+      getField({
+        field: '',
+        fields: ['b', 'c'],
+      })
+    ).toBe('');
+
+    expect(
+      getField({
+        field: 'a',
+        fields: ['b', 'c'],
+      })
+    ).toBe('a');
+
+    expect(
+      getField({
+        field: ['a'],
+        fields: ['b', 'c'],
+      })
+    ).toBe('a');
+
+    expect(
+      getField({
+        field: [],
+        fields: ['b', 'c'],
+      })
+    ).toBe(undefined);
+
+    expect(
+      getField({
+        field: undefined,
+        fields: 'b',
+      })
+    ).toBe('b');
+
+    expect(
+      getField({
+        field: undefined,
+        fields: ['b', 'c'],
+      })
+    ).toBe('b');
+
+    expect(
+      getField(
+        {
+          field: null,
+          fields: [],
+        },
+        'c'
+      )
+    ).toBe('c');
+
+    expect(() => {
+      getField({
+        field: null,
+        fields: [],
+      });
+    }).toThrow();
+  });
+
+  it('getAllNodes', () => {
+    expect(getAllNodes(null)).toEqual([]);
+
+    expect(getAllNodes({ a: 1 })).toEqual([]);
+
+    const nodes = ['a', 'b', 'c'];
+    expect(
+      getAllNodes({
+        each: (loop) => nodes.forEach(loop),
+      })
+    ).toEqual(nodes);
+
+    expect(
+      getAllNodes({
+        eachNode: (loop) => nodes.forEach(loop),
+      })
+    ).toEqual(nodes);
+  });
+});

src/plots/scatter/util.ts

@@ -22,7 +22,7 @@ const REGRESSION_MAP = {
   quad: regressionQuad,
 };
 
-type renderOptions = {
+type RenderOptions = {
   view: View;
   options: ScatterOptions;
 };
@@ -133,7 +133,7 @@ export function getQuadrantDefaultConfig(xBaseline: number, yBaseline: number) {
   return defaultConfig;
 }
 
-const splinePath = (data: number[][], config: renderOptions) => {
+const splinePath = (data: number[][], config: RenderOptions) => {
   const {
     view,
     options: { xField, yField },
@@ -146,7 +146,7 @@ const splinePath = (data: number[][], config: renderOptions) => {
   return getSplinePath(pathData, false);
 };
 
-export const getPath = (config: renderOptions) => {
+export const getPath = (config: RenderOptions) => {
   const { options } = config;
   const { xField, yField, data, regressionLine } = options;
   const { type = 'linear', algorithm } = regressionLine;