test: fix some type checking errors and warnings

Author: freddy38510

test/advanced.spec.ts

@@ -8,7 +8,7 @@ test('support chaining with other loaders', async () => {
   const { componentModule } = await mockBundleAndRun({
     entry: 'basic.vue',
     modify: (config) => {
-      config!.module!.rules[0] = {
+      config.module!.rules![0] = {
         test: /\.vue$/,
         use: ['@freddy38510/vue-loader', require.resolve('./mock-loaders/js')],
       }
@@ -22,7 +22,7 @@ test.skip('inherit queries on files', async () => {
   const { componentModule } = await mockBundleAndRun({
     entry: 'basic.vue?change',
     modify: (config) => {
-      config!.module!.rules[0] = {
+      config.module!.rules![0] = {
         test: /\.vue$/,
         use: [
           '@freddy38510/vue-loader',

test/edgeCases.spec.ts

@@ -31,8 +31,8 @@ test('vue rule with include', async () => {
   const result = await mockBundleAndRun({
     entry: 'basic.vue',
     modify: (config: any) => {
-      const i = config.module.rules.findIndex((r) =>
-        r.test.toString().includes('vue')
+      const i = (config.module.rules as webpack.RuleSetRule[]).findIndex((r) =>
+        r.test?.toString().includes('vue')
       )
       config.module.rules[i] = {
         test: /\.vue$/,
@@ -74,10 +74,10 @@ test('normalize multiple use + options', async () => {
   await bundle({
     entry: 'basic.vue',
     modify: (config: any) => {
-      const i = config.module.rules.findIndex((r) =>
-        r.test.toString().includes('vue')
+      const i = (config.module.rules as webpack.RuleSetRule[]).findIndex((r) =>
+        r.test?.toString().includes('vue')
       )
-      config!.module!.rules[i] = {
+      config.module.rules[i] = {
         test: /\.vue$/,
         use: [
           {
@@ -91,11 +91,11 @@ test('normalize multiple use + options', async () => {
 })
 
 test('should not duplicate css modules value imports', async () => {
-  const { window, exports } = await mockBundleAndRun({
+  const { window, _exports } = await mockBundleAndRun({
     entry: './test/fixtures/duplicate-cssm.js',
     modify: (config: any) => {
-      const i = config.module.rules.findIndex((r) =>
-        r.test.toString().includes('css')
+      const i = (config.module.rules as webpack.RuleSetRule[]).findIndex((r) =>
+        r.test?.toString().includes('css')
       )
       config.module.rules[i] = {
         test: /\.css$/,
@@ -117,8 +117,8 @@ test('should not duplicate css modules value imports', async () => {
   const style = normalizeNewline(styles[1]!.textContent!)
   // value should be injected
   expect(style).toMatch('color: red;')
-  // exports is set as the locals imported from values.css
-  expect(exports.color).toBe('red')
+  // _exports is set as the locals imported from values.css
+  expect(_exports.color).toBe('red')
 })
 
 // #1213
@@ -146,8 +146,8 @@ test('usage with null-loader', async () => {
   await mockBundleAndRun({
     entry: 'basic.vue',
     modify: (config: any) => {
-      const i = config.module.rules.findIndex((r) =>
-        r.test.toString().includes('css')
+      const i = (config.module.rules as webpack.RuleSetRule[]).findIndex((r) =>
+        r.test?.toString().includes('css')
       )
       config.module.rules[i] = {
         test: /\.css$/,

test/fixtures/entry.js

@@ -1,11 +1,11 @@
 import { createApp } from 'vue'
 
 import Component from '~target'
-import * as exports from '~target'
+import * as _exports from '~target'
 
 if (typeof window !== 'undefined') {
   window.componentModule = Component
-  window.exports = exports
+  window.exports = _exports
 
   const app = createApp(Component)
   const container = window.document.createElement('div')

test/fixtures/i18n-entry.js

@@ -2,7 +2,7 @@ import { createApp } from 'vue'
 import { createI18n } from 'vue-i18n'
 
 import Component from './i18n.vue'
-import * as exports from './i18n.vue'
+import * as _exports from './i18n.vue'
 
 const i18n = createI18n({
   locale: 'de',
@@ -12,7 +12,7 @@ const i18n = createI18n({
 
 if (typeof window !== 'undefined') {
   window.componentModule = Component
-  window.exports = exports
+  window.exports = _exports
 
   const app = createApp(Component).use(i18n)
 

test/fixtures/postcss.vue

@@ -7,6 +7,3 @@ h1
   color: red
   font-size: 14px
 </style>
-<script>
-export default {}
-</script>

test/script.spec.ts

@@ -1,30 +1,30 @@
 import { mockBundleAndRun } from './utils'
 
 test('named exports', async () => {
-  const { exports } = await mockBundleAndRun({
+  const { _exports } = await mockBundleAndRun({
     entry: 'named-exports.vue',
   })
-  expect(exports.default.name).toBe('named-exports')
-  expect(exports.foo()).toBe(1)
+  expect(_exports.default.name).toBe('named-exports')
+  expect(_exports.foo()).toBe(1)
 })
 
 test('experimental <script setup>', async () => {
   await mockBundleAndRun({ entry: 'ScriptSetup.vue' })
 })
 
 test('should handle custom resource query', async () => {
-  const { exports } = await mockBundleAndRun({
+  const { _exports } = await mockBundleAndRun({
     entry: 'custom-query.vue',
   })
 
-  expect(exports.default.data().msg).toBe('Hello from Component A!')
+  expect(_exports.default.data().msg).toBe('Hello from Component A!')
 })
 
 test('should support importing external types', async () => {
-  const { exports, window } = await mockBundleAndRun({
+  const { _exports, window } = await mockBundleAndRun({
     entry: 'imported-types.vue',
   })
-  expect(exports.default).toMatchObject({
+  expect(_exports.default).toMatchObject({
     props: {
       msg: {
         type: window.String,

test/utils.ts

@@ -156,13 +156,13 @@ export async function mockBundleAndRun(
   }
 
   const { window } = dom
-  const { componentModule, exports, instance } = window
+  const { componentModule, exports: _exports, instance } = window
 
   return {
     window,
 
     componentModule,
-    exports,
+    _exports,
     instance,
 
     code,