sea-execution: refresh test fakes for the merged-kernel binding surface

Rebased onto the fixed sea-auth (#379 review fixes). Updates the unit-test
fakes to the merged-kernel Connection/Statement surface so the spec
type-checks under ts-node:
- FakeNativeStatement: add statementId + the status accessors
  (numModifiedRows/displayMessage/diagnosticInfo/errorDetailsJson) and make
  schema() synchronous.
- FakeNativeConnection: add the sessionId getter; make executeStatement's
  options param optional so it stays assignable to the binding's
  executeStatement(sql) while still recording forwarded options.
- makeBinding: cast Connection/Statement through the binding's member types
  (typeof is illegal on the loader's type aliases).
- e2e: cast useSEA as ConnectionOptions & InternalConnectionOptions (the
  e2e test now lives under the wired tests/e2e/sea/ dir).

Note: the SeaOperationBackend status()/getResultMetadata() neutral-type
conformance and the per-statement-options -> session-level migration remain
pre-existing follow-ups for sea-results (tracked on the stack tip).

Co-authored-by: Isaac

Author: msrathore-db

tests/e2e/sea/execution-e2e.test.ts

@@ -14,6 +14,8 @@
 
 import { expect } from 'chai';
 import { DBSQLClient } from '../../../lib';
+import { ConnectionOptions } from '../../../lib/contracts/IDBSQLClient';
+import { InternalConnectionOptions } from '../../../lib/contracts/InternalConnectionOptions';
 
 /**
  * sea-execution end-to-end test.
@@ -66,7 +68,7 @@ describe('SEA execution end-to-end', function e2eSuite() {
       path: httpPath as string,
       token: token as string,
       useSEA: true,
-    });
+    } as ConnectionOptions & InternalConnectionOptions);
 
     const session = await client.openSession({
       initialCatalog: 'main',
@@ -99,7 +101,7 @@ describe('SEA execution end-to-end', function e2eSuite() {
       path: httpPath as string,
       token: token as string,
       useSEA: true,
-    });
+    } as ConnectionOptions & InternalConnectionOptions);
 
     // Sanity-check that supplying session-level Spark conf does not
     // break openSession. The SEA wire applies these as `parameters` on

tests/unit/sea/execution.test.ts

@@ -39,11 +39,15 @@ class FakeNativeStatement implements SeaNativeStatement {
 
   public cancelled = false;
 
+  // Mirrors the kernel `Statement.statementId` getter.
+  public readonly statementId = '01ef-fake-statement-id';
+
   public async fetchNextBatch() {
     return null;
   }
 
-  public async schema() {
+  // schema() is synchronous on the merged-kernel binding.
+  public schema() {
     return { ipcBytes: Buffer.alloc(0) };
   }
 
@@ -54,6 +58,23 @@ class FakeNativeStatement implements SeaNativeStatement {
   public async close() {
     this.closed = true;
   }
+
+  // Status accessors added by the kernel's status-fields surface.
+  public async numModifiedRows(): Promise<number | null> {
+    return null;
+  }
+
+  public async displayMessage(): Promise<string | null> {
+    return null;
+  }
+
+  public async diagnosticInfo(): Promise<string | null> {
+    return null;
+  }
+
+  public async errorDetailsJson(): Promise<string | null> {
+    return null;
+  }
 }
 
 class FakeNativeConnection implements SeaNativeConnection {
@@ -67,7 +88,14 @@ class FakeNativeConnection implements SeaNativeConnection {
 
   public statementToReturn: FakeNativeStatement = new FakeNativeStatement();
 
-  public async executeStatement(sql: string, options: SeaExecuteOptions): Promise<SeaNativeStatement> {
+  // Mirrors the kernel `Connection.sessionId` getter.
+  public readonly sessionId = '01ef-fake-session-id';
+
+  // `options` is optional so this stays structurally assignable to the
+  // merged binding's `executeStatement(sql)` while still recording any
+  // per-statement options the caller forwards (the kernel now applies
+  // those at session level — see the session-level options migration).
+  public async executeStatement(sql: string, options?: SeaExecuteOptions): Promise<SeaNativeStatement> {
     if (this.throwOnExecute) {
       throw this.throwOnExecute;
     }
@@ -88,8 +116,10 @@ function makeBinding(connection: SeaNativeConnection): SeaNativeBinding & {
   const binding: SeaNativeBinding = {
     version: () => 'test',
     openSession: openSessionStub,
-    Connection: function Connection() {},
-    Statement: function Statement() {},
+    // Index the binding type for the class constructor types; `typeof
+    // Connection` is illegal since they're exported as type aliases.
+    Connection: function Connection() {} as unknown as SeaNativeBinding['Connection'],
+    Statement: function Statement() {} as unknown as SeaNativeBinding['Statement'],
   };
   return Object.assign(binding, { openSessionStub });
 }