From 5bdb2ff1aedf3dc4e1181414cbcdf82b3c1f63bd Mon Sep 17 00:00:00 2001 From: Jonathan Cammisuli <4332460+Cammisuli@users.noreply.github.com> Date: Thu, 14 Nov 2024 11:15:39 -0500 Subject: [PATCH] fix(core): add dts option for napi executor (#66) --- e2e/rust-e2e/src/napi.spec.ts | 13 ++++++++++++- project.json | 3 ++- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/e2e/rust-e2e/src/napi.spec.ts b/e2e/rust-e2e/src/napi.spec.ts index 2b7e3b7..0825972 100644 --- a/e2e/rust-e2e/src/napi.spec.ts +++ b/e2e/rust-e2e/src/napi.spec.ts @@ -1,7 +1,7 @@ import { execSync } from 'child_process'; import { createTestProject, runNxCommand } from './utils'; import { rmSync } from 'fs'; -import { listFiles } from '@nx/plugin/testing'; +import { listFiles, readFile, updateFile } from '@nx/plugin/testing'; describe('napi', () => { let projectDirectory: string; @@ -31,6 +31,15 @@ describe('napi', () => { projectDirectory ); + const projectConfigPath = `test-project-napi/napi_proj/project.json`; + const projectFile = JSON.parse(readFile(projectConfigPath)); + projectFile['targets']['build']['options'] = { + ...projectFile['targets']['build']['options'], + jsFile: 'native.js', + dts: 'native.d.ts', + }; + updateFile(projectConfigPath, JSON.stringify(projectFile, null, 2)); + expect(listFiles(`test-project-napi/napi_proj/npm`).length).toBeGreaterThan( 0 ); @@ -40,6 +49,8 @@ describe('napi', () => { ).not.toThrow(); const files = listFiles(`test-project-napi/napi_proj`); + expect(files.some((file) => file.endsWith('native.js'))).toBeTruthy(); + expect(files.some((file) => file.endsWith('native.d.ts'))).toBeTruthy(); expect(files.some((file) => file.endsWith('.node'))).toBeTruthy(); expect(() => diff --git a/project.json b/project.json index d47f979..ad8c30c 100644 --- a/project.json +++ b/project.json @@ -6,7 +6,8 @@ "executor": "@nx/js:verdaccio", "options": { "config": ".verdaccio/config.yml", - "port": 3889 + "port": 3889, + "clear": true } } }