Skip to content

Commit 77b6aab

Browse files
elrrrrrrrclaude
andcommitted
fix: update cluster test path and ci workspace syntax
- Add hoisted node_modules path to cluster options test - Use package name instead of dir path for ut workspace flag Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 8baa31d commit 77b6aab

2 files changed

Lines changed: 6 additions & 4 deletions

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -216,8 +216,8 @@ jobs:
216216

217217
- name: Run tests
218218
run: |
219-
ut build --workspace ./tools/egg-bin
220-
ut ci --workspace ./tools/egg-bin
219+
ut build -w @eggjs/bin
220+
ut ci -w @eggjs/bin
221221
env:
222222
# https://github.com/jamiebuilds/ci-parallel-vars
223223
CI_NODE_INDEX: ${{ matrix.shardIndex }}
@@ -262,7 +262,7 @@ jobs:
262262
- name: Run tests
263263
run: |
264264
ut build
265-
ut ci --workspace ./tools/scripts
265+
ut ci -w @eggjs/scripts
266266
267267
- name: Code Coverage
268268
if: ${{ matrix.os != 'windows-latest' }}

packages/cluster/test/options.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -239,10 +239,12 @@ describe('test/options.test.ts', () => {
239239
baseDir,
240240
});
241241
const expectPaths = [
242-
// run int workspace root
242+
// run in workspace root
243243
path.join(__dirname, '../../egg'),
244244
// run in project root
245245
path.join(__dirname, '../node_modules/egg'),
246+
// hoisted to monorepo root node_modules
247+
path.join(__dirname, '../../../node_modules/egg'),
246248
];
247249
assert(
248250
expectPaths.includes(options.framework),

0 commit comments

Comments
 (0)