Fixes and debugging

Author: kriszyp

integrationTests/security/crl-verification.test.ts

@@ -25,7 +25,7 @@ import {
 
 const HTTPS_PORT = 9927;
 const FIXTURE_PATH = join(import.meta.dirname, 'fixture');
-const isBun = typeof globalThis.Bun !== 'undefined';
+const testsBun = process.env.HARPER_RUNTIME === 'bun';
 
 // The last test stops the CRL server to verify caching, so tests must run sequentially.
 // Tests CANNOT run concurrently because the caching test stops the server that earlier tests need.
@@ -35,7 +35,7 @@ suite(
 	(ctx: ContextWithHarper) => {
 		let crlServer: CrlServerContext | null = null;
 		let certsPath: string; // Track separately for cleanup even if server is stopped
-		if (isBun) return; // no Bun testing for now
+		if (testsBun) return; // no Bun testing for now
 		before(async () => {
 			// 1. Create temp directory for certificates
 			certsPath = await mkdtemp(join(tmpdir(), 'harper-crl-test-'));

integrationTests/security/ocsp-verification.test.ts

@@ -25,7 +25,7 @@ import { generateOcspCertificates } from '../utils/security/ocsp/generate-test-c
 
 const HTTPS_PORT = 9927;
 const FIXTURE_PATH = join(import.meta.dirname, 'fixture');
-const isBun = typeof globalThis.Bun !== 'undefined';
+const testsBun = process.env.HARPER_RUNTIME === 'bun';
 
 // The last test stops the OCSP responder to verify caching, so tests must run sequentially.
 // Tests CANNOT run concurrently because the caching test stops the responder that earlier tests need.
@@ -35,7 +35,7 @@ suite(
 	(ctx: ContextWithHarper) => {
 		let ocspResponder: OcspResponderContext | null = null;
 		let certsPath: string; // Track separately for cleanup even if responder is stopped
-		if (isBun) return; // no Bun testing for now
+		if (testsBun) return; // no Bun testing for now
 
 		before(async () => {
 			// 1. Create temp directory for certificates

integrationTests/upgrade/4.x-upgrade.test.ts

@@ -27,6 +27,7 @@ suite('Start 4.x server and test upgrade', (ctx: ContextWithHarper) => {
 			harperBinPath: join(legacyPath, 'bin', 'harperdb.js'),
 			harperRuntime: 'node', // legacy harperdb.js is node only
 		});
+		console.log('Started legacy harper on node', ctx.harper);
 		await sendOperation(ctx.harper, {
 			operation: 'create_table',
 			table: 'test',
@@ -58,7 +59,10 @@ suite('Start 4.x server and test upgrade', (ctx: ContextWithHarper) => {
 
 	test('upgrade and start', async () => {
 		await killHarper(ctx); // kill old 4.x harper
+		console.log('Killed legacy harper on node', ctx.harper);
+
 		await startHarper(ctx, { config: {}, env: {} }); // start on v5
+		console.log('Starting new harper on node', ctx.harper);
 		let response = await sendOperation(ctx.harper, {
 			operation: 'search_by_conditions',
 			table: 'test',

integrationTests/utils/harperLifecycle.ts

@@ -145,6 +145,7 @@ function runHarperCommand({
 		harperRuntime === 'bun'
 			? [harperScript, ...args]
 			: ['--trace-warnings', '--force-node-api-uncaught-exceptions-policy=true', harperScript, ...args];
+	console.log('running', harperRuntime, runtimeArgs);
 	const proc = spawn(harperRuntime, runtimeArgs, {
 		env: { ...process.env, ...env },
 	});