Skip to content

Commit a5bdf68

Browse files
committed
Fix testing failure
1 parent bf763af commit a5bdf68

4 files changed

Lines changed: 4 additions & 4 deletions

File tree

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
"docs": "make -C tools/jsdoc",
4747
"docs:gh-pages": "node tools/jsdoc/regenerate-published-docs.js --branch origin/gh-pages --preserve-published",
4848
"docs:gh-pages:full": "node tools/jsdoc/regenerate-published-docs.js --branch origin/gh-pages --full-rebuild",
49-
"test": "nyc node --expose-gc ./scripts/run_test.cjs && tsd && npm install --no-save electron && node test/electron/run_test.js",
49+
"test": "nyc node --expose-gc ./scripts/run_test.cjs && tsd && npm install --no-save electron && node test/electron/run_test.cjs",
5050
"test-idl": "nyc node --expose-gc ./scripts/run_test.cjs --idl",
5151
"lint": "eslint && node ./scripts/cpplint.cjs",
5252
"test:asan": "bash scripts/run_asan_test.sh",
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ try {
3636
}
3737

3838
let command = electron;
39-
let args = [path.join(__dirname, 'test_usability.js'), '--no-sandbox'];
39+
let args = [path.join(__dirname, 'test_usability.cjs'), '--no-sandbox'];
4040

4141
// Handle headless Linux environments (like CI) by using xvfb-run
4242
if (process.platform === 'linux' && !process.env.DISPLAY) {
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
const rclnodejs = require('../../index.js');
3+
const rclnodejs = require('../../index.js').default;
44
const { app } = require('electron');
55

66
app.on('ready', () => {

test/test-web-ws.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import assert from 'assert';
1313
import rclnodejs from '../index.js';
1414
import { createRuntime, WebSocketTransport } from '../lib/runtime/index.js';
1515

16-
// `web/` is ESM; await import() lets us pull it in from this CJS file.
16+
// `web/` is ESM; dynamic import() defers loading the browser SDK until the test starts.
1717
let connect;
1818
before(async function () {
1919
({ connect } = await import('../web/index.js'));

0 commit comments

Comments
 (0)