Skip to content

Commit 2695fb9

Browse files
Claudehotlong
andauthored
fix: add missing subpath aliases to vitest configs and fix plugin-auth test mocks
- Add @objectstack/spec/api and @objectstack/spec/kernel aliases to driver-memory, service-ai, and plugin-hono-server vitest configs - Add missing get() method to rawApp mocks in plugin-auth tests - Ensures all packages that depend on @objectstack/core have complete alias definitions for transitive dependencies - Fixes ENOTDIR error: "not a directory, open '.../spec/src/index.ts/api'" - Fixes TypeError: "rawApp.get is not a function" in plugin-auth tests Agent-Logs-Url: https://github.com/objectstack-ai/framework/sessions/dbd1f109-9696-4678-a767-f2f51a085159 Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
1 parent 5bad4e2 commit 2695fb9

4 files changed

Lines changed: 13 additions & 3 deletions

File tree

packages/plugins/driver-memory/vitest.config.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,10 @@ export default defineConfig({
1111
resolve: {
1212
alias: {
1313
'@objectstack/core': path.resolve(__dirname, '../../core/src/index.ts'),
14+
'@objectstack/spec/api': path.resolve(__dirname, '../../spec/src/api/index.ts'),
1415
'@objectstack/spec/contracts': path.resolve(__dirname, '../../spec/src/contracts/index.ts'),
1516
'@objectstack/spec/data': path.resolve(__dirname, '../../spec/src/data/index.ts'),
17+
'@objectstack/spec/kernel': path.resolve(__dirname, '../../spec/src/kernel/index.ts'),
1618
'@objectstack/spec/system': path.resolve(__dirname, '../../spec/src/system/index.ts'),
1719
'@objectstack/spec': path.resolve(__dirname, '../../spec/src/index.ts'),
1820
},

packages/plugins/plugin-auth/src/auth-plugin.test.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -137,6 +137,7 @@ describe('AuthPlugin', () => {
137137
it('should register routes with HTTP server on kernel:ready', async () => {
138138
const mockRawApp = {
139139
all: vi.fn(),
140+
get: vi.fn(),
140141
};
141142

142143
const mockHttpServer = {
@@ -173,6 +174,7 @@ describe('AuthPlugin', () => {
173174
it('should log via ctx.logger when better-auth returns a 500 response', async () => {
174175
const mockRawApp = {
175176
all: vi.fn(),
177+
get: vi.fn(),
176178
};
177179

178180
const mockHttpServer = {
@@ -270,7 +272,7 @@ describe('AuthPlugin', () => {
270272
});
271273

272274
it('should auto-detect baseUrl from http-server port when port differs', async () => {
273-
const mockRawApp = { all: vi.fn() };
275+
const mockRawApp = { all: vi.fn(), get: vi.fn() };
274276
const mockHttpServer = {
275277
post: vi.fn(), get: vi.fn(), put: vi.fn(), delete: vi.fn(),
276278
patch: vi.fn(), use: vi.fn(),
@@ -306,7 +308,7 @@ describe('AuthPlugin', () => {
306308
(mockContext.registerService as any).mockClear();
307309
await localPlugin.init(mockContext);
308310

309-
const mockRawApp = { all: vi.fn() };
311+
const mockRawApp = { all: vi.fn(), get: vi.fn() };
310312
const mockHttpServer = {
311313
post: vi.fn(), get: vi.fn(), put: vi.fn(), delete: vi.fn(),
312314
patch: vi.fn(), use: vi.fn(),
@@ -338,7 +340,7 @@ describe('AuthPlugin', () => {
338340
(mockContext.registerService as any).mockClear();
339341
await localPlugin.init(mockContext);
340342

341-
const mockRawApp = { all: vi.fn() };
343+
const mockRawApp = { all: vi.fn(), get: vi.fn() };
342344
const mockHttpServer = {
343345
post: vi.fn(), get: vi.fn(), put: vi.fn(), delete: vi.fn(),
344346
patch: vi.fn(), use: vi.fn(),
@@ -400,6 +402,7 @@ describe('AuthPlugin', () => {
400402

401403
const mockRawApp = {
402404
all: vi.fn(),
405+
get: vi.fn(),
403406
};
404407

405408
const mockHttpServer = {

packages/plugins/plugin-hono-server/vitest.config.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ export default defineConfig({
1313
'@objectstack/core': path.resolve(__dirname, '../../core/src/index.ts'),
1414
'@objectstack/spec/api': path.resolve(__dirname, '../../spec/src/api/index.ts'),
1515
'@objectstack/spec/contracts': path.resolve(__dirname, '../../spec/src/contracts/index.ts'),
16+
'@objectstack/spec/data': path.resolve(__dirname, '../../spec/src/data/index.ts'),
17+
'@objectstack/spec/kernel': path.resolve(__dirname, '../../spec/src/kernel/index.ts'),
1618
'@objectstack/spec/system': path.resolve(__dirname, '../../spec/src/system/index.ts'),
1719
'@objectstack/spec': path.resolve(__dirname, '../../spec/src/index.ts'),
1820
},

packages/services/service-ai/vitest.config.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,11 @@ export default defineConfig({
1212
alias: {
1313
'@objectstack/core': path.resolve(__dirname, '../../core/src/index.ts'),
1414
'@objectstack/spec/ai': path.resolve(__dirname, '../../spec/src/ai/index.ts'),
15+
'@objectstack/spec/api': path.resolve(__dirname, '../../spec/src/api/index.ts'),
1516
'@objectstack/spec/contracts': path.resolve(__dirname, '../../spec/src/contracts/index.ts'),
1617
'@objectstack/spec/data': path.resolve(__dirname, '../../spec/src/data/index.ts'),
18+
'@objectstack/spec/kernel': path.resolve(__dirname, '../../spec/src/kernel/index.ts'),
19+
'@objectstack/spec/system': path.resolve(__dirname, '../../spec/src/system/index.ts'),
1720
'@objectstack/spec': path.resolve(__dirname, '../../spec/src/index.ts'),
1821
},
1922
},

0 commit comments

Comments
 (0)