From ad22d106a69b60d88f0034d8dd12f53974e2318b Mon Sep 17 00:00:00 2001 From: Julian Stirling Date: Wed, 20 May 2026 14:11:49 +0000 Subject: [PATCH] Apply suggestions from code review of branch webapp_unit_testing Co-authored-by: Beth Probert --- webapp/vite.config.mjs | 2 +- webapp/vitest.config.js | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/webapp/vite.config.mjs b/webapp/vite.config.mjs index 37a19b7a..c6f02b96 100644 --- a/webapp/vite.config.mjs +++ b/webapp/vite.config.mjs @@ -40,7 +40,7 @@ export default defineConfig(({ command, mode }) => { ? [ (node) => { // Check if the current AST node is an HTML element - if (node.type === 1 /* NodeTypes.ELEMENT */) { + if (node.type === 1) { // Filter out any property named "data-test-id" node.props = node.props.filter((prop) => prop.name !== "data-test-id"); } diff --git a/webapp/vitest.config.js b/webapp/vitest.config.js index a07d6ae7..3dcaa9a8 100644 --- a/webapp/vitest.config.js +++ b/webapp/vitest.config.js @@ -44,8 +44,8 @@ export default defineConfig({ // Files to include or exclude in coverage include: ["src/**/*.vue", "src/**/*.js"], exclude: [], - // These thresholds raise ERROR if the coverture it's under the provided percentages. - // As the coverture is low we set the limit low, as testing spec files increase, + // These thresholds raise ERROR if the coverage it's under the provided percentages. + // As the coverage is low we set the limit low, as testing spec files increase, // these thresholds need to increase accordingly. thresholds: { lines: 4,