[stonecrop] use Vitest workspaces

.github/workflows/tests.yml

@@ -55,9 +55,10 @@ jobs:
 
     - name: Coverage Report
       if: always() # Also generate the report if tests are failing
-      uses: davelosert/vitest-coverage-report-action@v1
+      uses: davelosert/vitest-coverage-report-action@v2
       with:
         working-directory: ./aform
+        file-coverage-mode: 'all'
         vite-config-path: './vite.config.ts'
 
   atable:
@@ -93,7 +94,8 @@ jobs:
 
     - name: Coverage Report
       if: always() # Also generate the report if tests are failing
-      uses: davelosert/vitest-coverage-report-action@v1
+      uses: davelosert/vitest-coverage-report-action@v2
       with:
         working-directory: ./atable
+        file-coverage-mode: 'all'
         vite-config-path: './vite.config.ts'

aform/.eslintrc.cjs

@@ -61,5 +61,5 @@ module.exports = {
 		'@typescript-eslint/no-unsafe-return': 'off',
 	},
 
-	ignorePatterns: ['.eslintrc.cjs', '*.config.ts', 'node_modules/', 'dist/'],
+	ignorePatterns: ['.eslintrc.cjs', '*.config.ts', '*.spec.ts', 'node_modules/', 'dist/'],
 }

aform/package.json

@@ -36,14 +36,14 @@
 		"vue": "^3.2.47"
 	},
 	"devDependencies": {
-		"@stonecrop/atable": "workspace:*",
 		"@histoire/plugin-vue": "^0.16.1",
+		"@stonecrop/atable": "workspace:*",
 		"@types/uuid": "^9.0.0",
 		"@typescript-eslint/eslint-plugin": "^5.59.5",
 		"@typescript-eslint/parser": "^5.59.5",
 		"@vitejs/plugin-vue": "^4.2.1",
-		"@vitest/coverage-c8": "^0.31.0",
-		"@vitest/ui": "^0.31.0",
+		"@vitest/coverage-v8": "^1.5.0",
+		"@vitest/ui": "^1.5.0",
 		"@vue/test-utils": "^2.3.2",
 		"cypress": "^12.11.0",
 		"eslint": "^8.40.0",
@@ -53,7 +53,7 @@
 		"jsdom": "^22.0.0",
 		"typescript": "^5.0.4",
 		"vite": "^4.3.5",
-		"vitest": "^0.31.0",
+		"vitest": "^1.5.0",
 		"vue-router": "^4"
 	},
 	"peerDependencies": {

aform/src/components/AForm.vue

@@ -55,6 +55,7 @@ const childModels = computed({
 				},
 				set: newValue => {
 					// Find the component in modelValue and update it
+					// eslint-disable-next-line vue/no-mutating-props
 					props.modelValue[i].value = newValue
 					emit('update:modelValue', props.modelValue)
 				},

aform/tests/aform.spec.ts

@@ -1,8 +1,8 @@
 import { describe, it, expect } from 'vitest'
 import { mount } from '@vue/test-utils'
 
-import AForm from '@/components/AForm.vue'
-import ATextInput from '@/components/form/ATextInput.vue'
+import AForm from '../src/components/AForm.vue'
+import ATextInput from '../src/components/form/ATextInput.vue'
 
 describe('AForm Component', () => {
 	const form_schema = [
@@ -16,6 +16,7 @@ describe('AForm Component', () => {
 	const wrapper = mount(AForm, {
 		props: {
 			modelValue: form_schema,
+			data: {},
 		},
 		components: {
 			ATextInput,

aform/vite.config.ts

@@ -39,10 +39,13 @@ export default defineConfig({
 		globals: true,
 		environment: 'jsdom',
 		coverage: {
-			lines: 50,
-			branches: 50,
-			functions: 50,
-			statements: 50,
+			provider: 'v8',
+			thresholds: {
+				lines: 70,
+				branches: 70,
+				functions: 70,
+				statements: 70,
+			},
 			// required for Github Actions CI
 			reporter: ['text', 'json-summary', 'json'],
 			reportsDirectory: './coverage',
@@ -63,7 +66,6 @@ export default defineConfig({
 				'**/{karma,rollup,webpack,vite,vitest,jest,ava,babel,nyc,cypress,tsup,build}.config.*',
 				'**/.{eslint,mocha,prettier}rc.{js,cjs,yml}',
 			],
-			all: true,
 			skipFull: true,
 		},
 	},

atable/.eslintrc.cjs

@@ -61,5 +61,5 @@ module.exports = {
 		'@typescript-eslint/no-unsafe-return': 'off',
 	},
 
-	ignorePatterns: ['.eslintrc.cjs', '*.config.ts', 'node_modules/', 'dist/'],
+	ignorePatterns: ['.eslintrc.cjs', '*.config.ts', '*.spec.ts', 'node_modules/', 'dist/'],
 }