chore: eliminate test run.ts#8414
Conversation
…s to point to root mocharc
…i test runner memory limits, and remove all duplicate package-level configs
… script configurations
There was a problem hiding this comment.
Code Review
This pull request centralizes the Mocha test configuration by removing package-specific .mocharc.js files and updating package.json test scripts across the monorepo to reference a shared root configuration. It also increases the Node memory limit in the CI script. The review feedback highlights critical issues with the shared .mocharc.js changes: switching to ES module export syntax (export default) breaks CommonJS compatibility, and replacing the recursive option with parallel prevents nested test discovery. Additionally, several package test scripts, such as those in google-cloud-dialogflow and google-cloud-discoveryengine, are missing the shared configuration argument in their secondary or fallback Mocha invocations.
…able map test to support unified mocharc.cjs
…er ci/, and lower CI memory limit to 6GB
…d apply specific package fixes
…command to parallel mocha
…disable parallel execution
Eliminate the handwritten test runners (
test/run.ts) from major packages likegoogle-cloud-compute,google-cloud-dialogflow-cx, andgoogle-cloud-aiplatformto remove the maintenance of the handwritten test files. It standardizespackage.jsontest configurations monorepo-wide to use the root unified.mocharc.cjsconfiguration, and integrates the new shared batch runner forgoogle-cloud-computeto reduce memory load during execution.For #librarian/4583
For #librarian/5960