fix: TestWallet.simulateViaEntrypoint now propagates publicDataOverrides

TestWallet overrides BaseWallet.simulateViaEntrypoint and was dropping
publicDataOverrides on its way to pxe.simulateTx, so e2e tests using
.simulate({ publicDataOverrides }) saw the underlying real storage rather
than the override. Pull publicDataOverrides out of opts and pass it through.

Author: dbanks12

yarn-project/end-to-end/src/test-wallet/test_wallet.ts

@@ -260,7 +260,15 @@ export class TestWallet extends BaseWallet {
     executionPayload: ExecutionPayload,
     opts: SimulateViaEntrypointOptions,
   ): Promise<TxSimulationResultWithAppOffset> {
-    const { from, feeOptions, additionalScopes, skipTxValidation, skipFeeEnforcement, sendMessagesAs } = opts;
+    const {
+      from,
+      feeOptions,
+      additionalScopes,
+      skipTxValidation,
+      skipFeeEnforcement,
+      sendMessagesAs,
+      publicDataOverrides,
+    } = opts;
     const scopes = this.scopesFrom(from, additionalScopes);
     const skipKernels = this.simulationMode !== 'full';
     const useOverride = this.simulationMode === 'kernelless-override';
@@ -311,6 +319,7 @@ export class TestWallet extends BaseWallet {
       overrides,
       scopes,
       senderForTags: this.senderForTagsFrom(from, sendMessagesAs),
+      publicDataOverrides,
     });
     const appCallOffset = await this.computeAppCallOffset(from, feeOptions);
     return TxSimulationResultWithAppOffset.fromResultAndOffset(result, appCallOffset);