chore: Fix test commands (#460)

This PR improves the handling of test commands. The previous
implementation broke support for passing arguments to the test command
because it included a posttest script that automatically added a newline
to vitest.config. However, this is already handled by the lint-staged
step in the pre-push hook.

Author: sirtimid

.github/workflows/lint-build-test.yml

@@ -149,7 +149,7 @@ jobs:
           yarn build-release
           cd -
       - run: yarn build
-      - run: yarn run test
+      - run: yarn test:ci
       - name: Require clean working directory
         shell: bash
         run: |

package.json

@@ -31,13 +31,12 @@
     "postinstall": "simple-git-hooks",
     "prepack": "./scripts/prepack.sh",
     "pretest": "bash scripts/reset-coverage-thresholds.sh",
-    "posttest": "printf '\\n' >> vitest.config.ts",
-    "test": "yarn pretest && vitest run && yarn posttest",
-    "test:dev": "yarn pretest && vitest run --mode development && yarn posttest",
-    "test:clean": "yarn test --no-cache --coverage.clean",
+    "test": "yarn pretest && vitest run",
+    "test:ci": "vitest run",
+    "test:dev": "yarn test --mode development",
     "test:e2e": "yarn workspaces foreach --all run test:e2e",
     "test:e2e:ci": "yarn workspaces foreach --all run test:e2e:ci",
-    "test:verbose": "yarn pretest && yarn test --reporter verbose && yarn posttest",
+    "test:verbose": "yarn test --reporter verbose",
     "test:watch": "vitest",
     "why:batch": "./scripts/why-batch.sh"
   },