chore: resolved some lint error by unit test

Author: baiwusanyu-c

packages/core/hmr/__test__/hmr.spec.ts

@@ -16,9 +16,9 @@ mockModuleNode.add({ id: 'foo.vue' })
 const mockFileToModulesMap = new Map()
 mockFileToModulesMap.set('../D/test', mockModuleNode)
 
-let hmrModule = null
+let hmrModule: any = null
 const mockServer = {
-  reloadModule: (m) => {
+  reloadModule: (m: any) => {
     hmrModule = m
   },
   moduleGraph: {

packages/core/inject/__test__/inject-cssvars.spec.ts

@@ -51,7 +51,7 @@ describe('inject-cssvars', () => {
 
   describe('createCSSVarsObjCode', () => {
     test('should return empty string if vbindVariableList is empty', () => {
-      const vbindVariableList = []
+      const vbindVariableList: any[] = []
       const result = createCSSVarsObjCode(vbindVariableList, false)
       expect(result).toBe('')
     })

packages/core/inject/inject-cssvars.ts

@@ -65,13 +65,19 @@ export function injectUseCssVarsSetup(
   parserRes: IParseSFCRes,
 ) {
   let resMgcStr = mgcStr
-  if (!hasUseCssVars) {
-    const start = parserRes.setupBodyNode.start + 1
-    resMgcStr = resMgcStr.prependLeft(start, useCssVars)
-    resMgcStr = resMgcStr.prependLeft(0, importer)
-  } else {
-    const start = parserRes.useCSSVarsNode.start + 1
-    resMgcStr = resMgcStr.prependLeft(start, useCssVars)
+  if (parserRes) {
+    if (!hasUseCssVars
+      && parserRes.setupBodyNode
+      && parserRes.setupBodyNode.start) {
+      const start = parserRes.setupBodyNode.start + 1
+      resMgcStr = resMgcStr.prependLeft(start, useCssVars)
+      resMgcStr = resMgcStr.prependLeft(0, importer)
+    } else if (hasUseCssVars
+      && parserRes.useCSSVarsNode
+      && parserRes.useCSSVarsNode.start) {
+      const start = parserRes.useCSSVarsNode.start + 1
+      resMgcStr = resMgcStr.prependLeft(start, useCssVars)
+    }
   }
 
   return resMgcStr
@@ -96,10 +102,13 @@ export function injectUseCssVarsOption(
             const _sfc_main = __default__
             function _sfc_render`)
     resMgcStr = resMgcStr.prependLeft(0, importer)
-  } else {
+  } else if (hasUseCssVars
+      && parserRes.useCSSVarsNode
+      && parserRes.useCSSVarsNode.start) {
     const start = parserRes.useCSSVarsNode.start + 1
     resMgcStr = resMgcStr.prependLeft(start, useCssVars)
   }
+
   return resMgcStr
 }
 

packages/core/parser/__test__/parser-compiled-sfc.spec.ts

@@ -10,7 +10,7 @@ import {
   reSetVar,
   setVar,
 } from '../parser-compiled-sfc'
-import type { ImportSpecifier } from '@babel/types'
+import type { CallExpression, Identifier, ImportSpecifier, ObjectExpression } from '@babel/types'
 describe('parseSetupBody', () => {
   let node: any
 
@@ -82,7 +82,7 @@ describe('parseHasCSSVars', () => {
     const code = 'const a = 1'
     const ast = parse(code)
 
-    const node = ast.program.body[0].declarations[0].init
+    const node = (ast as any).program.body[0].declarations[0].init
 
     parseHasCSSVars(node, null!)
 
@@ -102,47 +102,47 @@ describe('parseHasCSSVars', () => {
 })
 
 describe('parseUseCSSVars function', () => {
-  let node: Node & { scopeIds?: Set<string> }
-  let parent: Node & { scopeIds?: Set<string> }
+  let node: Node & { scopeIds?: Set<string> } & Identifier & CallExpression & ObjectExpression
+  let parent: Node & { scopeIds?: Set<string> } & Identifier & CallExpression & ObjectExpression
 
   beforeEach(() => {
     reSetVar()
-    node = {} as Node & { scopeIds?: Set<string> }
-    parent = {} as Node & { scopeIds?: Set<string> }
+    node = {} as Node & { scopeIds?: Set<string> } & Identifier & CallExpression & ObjectExpression
+    parent = {} as Node & { scopeIds?: Set<string> } & Identifier & CallExpression & ObjectExpression
   })
 
   test('sets isUseCSSVarsEnter to true when called with _useCssVars identifier and a CallExpression parent', () => {
-    node.type = 'Identifier';
-    (node as any).name = '_useCssVars'
-    parent.type = 'CallExpression'
+    (node as Identifier).type = 'Identifier';
+    (node as Identifier).name = '_useCssVars';
+    (parent as CallExpression).type = 'CallExpression'
 
-    parseUseCSSVars(node, parent as any)
+    parseUseCSSVars(node as any, parent as any)
 
     expect(getVar().isUseCSSVarsEnter).toBe(true)
   })
 
   test('does not set isUseCSSVarsEnter to true when called with non-_useCssVars identifier and a CallExpression parent', () => {
-    node.type = 'Identifier';
-    (node as any).name = 'foo'
-    parent.type = 'CallExpression'
+    (node as Identifier).type = 'Identifier';
+    (node as Identifier).name = 'foo';
+    (parent as CallExpression).type = 'CallExpression'
 
     parseUseCSSVars(node, parent as any)
 
     expect(getVar().isUseCSSVarsEnter).toBe(false)
   })
 
   test('does not set isUseCSSVarsEnter to true when called with _useCssVars identifier and a non-CallExpression parent', () => {
-    node.type = 'Identifier';
-    (node as any).name = '_useCssVars'
-    parent.type = 'Identifier'
+    (node as Identifier).type = 'Identifier';
+    (node as Identifier).name = '_useCssVars';
+    (parent as Identifier).type = 'Identifier'
 
     parseUseCSSVars(node, parent as any)
 
     expect(getVar().isUseCSSVarsEnter).toBe(false)
   })
 
   test('does not set isUseCSSVarsEnter to true when called with _useCssVars identifier and no parent', () => {
-    node.type = 'Identifier';
+    (node as Identifier).type = 'Identifier';
     (node as any).name = '_useCssVars'
 
     parseUseCSSVars(node, undefined!)
@@ -151,26 +151,25 @@ describe('parseUseCSSVars function', () => {
   })
 
   test('sets useCSSVarsNode to the node when isUseCSSVarsEnter is true and node is an ObjectExpression', () => {
-    setVar('isUseCSSVarsEnter', true)
-    node.type = 'ObjectExpression'
-
+    setVar('isUseCSSVarsEnter', true);
+    (node as ObjectExpression).type = 'ObjectExpression'
     parseUseCSSVars(node, parent as any)
 
     expect(getVar().useCSSVarsNode).toBe(node)
   })
 
   test('does not set useCSSVarsNode when isUseCSSVarsEnter is true and node is not an ObjectExpression', () => {
-    setVar('isUseCSSVarsEnter', true)
-    node.type = 'Identifier'
+    setVar('isUseCSSVarsEnter', true);
+    (node as Identifier).type = 'Identifier'
 
     parseUseCSSVars(node, parent as any)
 
     expect(getVar().useCSSVarsNode).not.toBe(node)
   })
 
   test('does not set useCSSVarsNode when isUseCSSVarsEnter is false and node is an ObjectExpression', () => {
-    setVar('isUseCSSVarsEnter', false)
-    node.type = 'ObjectExpression'
+    setVar('isUseCSSVarsEnter', false);
+    (node as ObjectExpression).type = 'ObjectExpression'
 
     parseUseCSSVars(node, parent as any)
 

packages/core/parser/index.ts

@@ -2,3 +2,4 @@ export * from './parser-variable'
 export * from './parser-import'
 export * from './parser-vbindm'
 export { parserCompiledSfc } from './parser-compiled-sfc'
+export type { IParseSFCRes } from './parser-compiled-sfc'

packages/core/parser/parser-compiled-sfc.ts

@@ -2,7 +2,7 @@ import { parse as babelParse } from '@babel/parser'
 import { walk } from 'estree-walker-ts'
 import type {
   BlockStatement,
-  CallExpression, ExpressionStatement,
+  CallExpression,
   Identifier,
   ImportSpecifier,
   ObjectExpression,
@@ -35,7 +35,7 @@ export function parseHasCSSVars(
   if ((node as Identifier).type === 'Identifier'
     && (node as Identifier).name === 'useCssVars'
     && parent
-    && ((parent.type as ImportSpecifier) === 'ImportSpecifier'))
+    && (((parent as ImportSpecifier).type) === 'ImportSpecifier'))
     hasCSSVars = true
 }
 
@@ -47,7 +47,7 @@ export function parseUseCSSVars(
   if ((node as Identifier).type === 'Identifier'
     && (node as Identifier).name === '_useCssVars'
     && parent
-    && ((parent.type as CallExpression) === 'CallExpression'))
+    && (((parent as CallExpression).type) === 'CallExpression'))
     isUseCSSVarsEnter = true
 
   if (isUseCSSVarsEnter && (node as ObjectExpression).type === 'ObjectExpression') {
@@ -56,7 +56,6 @@ export function parseUseCSSVars(
   }
 }
 
-// TODO: unit test
 export function parserCompiledSfc(code: string) {
   reSetVar()
   const ast = babelParse(code, {
@@ -100,7 +99,7 @@ export function getVar() {
 }
 
 // test only
-export function setVar(k, v) {
+export function setVar(k: string, v: any) {
   if (k === 'isSetupEnter')
     isSetupEnter = v