revert(module-event): remove event mock entry point

The recording/interception mock added in d1a8b58 broke the framework's
configurator/provider lifecycle contract: unlike msal's configurator (a
disposable builder resolved once during configure), the event module's
onDispatch hook is installed directly into the long-lived
FrameworkEventDispatcher, so the "configurator" was secretly a permanent
runtime component exposed through a pre-init-only accessor (.event) -
architecturally inconsistent with every other FrameworkMockConfigurator
accessor.

Reworking it into a MockEventModuleProvider reached through
fusion.modules.event surfaced a simpler question: the real provider
already exposes event$ and addEventListener, which is sufficient for
"check that expected events are dispatched" with zero test doubles.
Recording/interception isn't a gap worth a mock module for - a test
subscribes to event$ or registers its own addEventListener the same way
application code would.

Reverts all files to their state at 340a72f, keeping nothing from the
mock feature.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: odinr

.changeset/module-event_mock-entry-point.md

@@ -1,4 +1,4 @@
-import { describe, expect, it, vi } from 'vitest';
+import { describe, expect, it } from 'vitest';
 
 import type { Module } from '@equinor/fusion-framework-module';
 import { enableMsalMock } from '@equinor/fusion-framework-module-msal/mock';
@@ -171,31 +171,6 @@ describe('FrameworkMockConfigurator', () => {
     expect(fusion.modules.http.createClient('my-api')).toBeDefined();
   });
 
-  it('records every event dispatched through the framework, including onModulesLoaded', async () => {
-    const configurator = new FrameworkMockConfigurator();
-
-    const fusion = await init(configurator);
-    await fusion.modules.event.dispatchEvent('myFeature', { detail: { id: 1 } });
-
-    expect(configurator.event.getEvents('onModulesLoaded')).toHaveLength(1);
-    expect(configurator.event.lastEvent('myFeature')?.detail).toEqual({ id: 1 });
-  });
-
-  it('lets a test cancel an event before its listeners run', async () => {
-    const configurator = new FrameworkMockConfigurator();
-    configurator.event.intercept('myFeature', (event) => event.preventDefault());
-
-    const fusion = await init(configurator);
-    const listener = vi.fn();
-    fusion.modules.event.addEventListener('myFeature', listener);
-    await fusion.modules.event.dispatchEvent('myFeature', {
-      detail: null,
-      cancelable: true,
-    });
-
-    expect(listener).not.toHaveBeenCalled();
-  });
-
   it('exposes the same services configurator the services module is built from', () => {
     const configurator = new FrameworkMockConfigurator();
 

packages/framework/src/__tests__/mock/mock-framework.test.ts

@@ -3,11 +3,6 @@ import type { AnyModule } from '@equinor/fusion-framework-module';
 import contextModule, {
   type ContextModuleConfigurator,
 } from '@equinor/fusion-framework-module-context';
-import { eventModuleKey } from '@equinor/fusion-framework-module-event';
-import {
-  createEventMockModule,
-  type EventMockConfigurator,
-} from '@equinor/fusion-framework-module-event/mock';
 import {
   httpMockModule,
   type HttpMockConfigurator,
@@ -48,13 +43,10 @@ import { FrameworkConfigurator } from '../FrameworkConfigurator.js';
  * are reachable the same way `.msal` is, since their `configure` factories
  * take no `ref` and so lose nothing by being pinned early.
  *
- * `event` is pinned through {@link createEventMockModule} rather than
- * {@link _pin}: its `configure` factory reads `ref` to wire bubbling to a
- * parent event provider when this configurator is hoisted inside a host
- * framework, a decision `_pin` would freeze before any `ref` could be known.
- * `.event` still records every dispatched event and lets a test intercept or
- * cancel one, so a test never subscribes to `event$` or races
- * `addEventListener` against dispatch order just to observe what fired.
+ * `event` is deliberately not pinned: its `configure` factory reads `ref` to
+ * wire bubbling to a parent event provider when this configurator is hoisted
+ * inside a host framework, and pinning would freeze that decision before a
+ * `ref` could ever be known.
  *
  * Because this *is* a `FrameworkConfigurator`, every `enableX` helper an
  * application already uses accepts it unchanged — including the ones an
@@ -98,12 +90,6 @@ export class FrameworkMockConfigurator<
     this._pin(servicesModule);
     this._pin(contextModule);
     this._pin(telemetryModule);
-
-    // `event`'s configure factory needs `ref` at configure time to wire
-    // bubbling, so it is pinned by hand instead of through `_pin`.
-    const { configurator: eventConfigurator, module: eventMockModule } = createEventMockModule();
-    this.#configurators.set(eventMockModule.name, eventConfigurator);
-    this.addConfig({ module: eventMockModule });
   }
 
   /**
@@ -189,20 +175,6 @@ export class FrameworkMockConfigurator<
     return this._getConfig<MsalMockConfigurator>(msalMockModule.name);
   }
 
-  /**
-   * Records and intercepts every event dispatched through the framework.
-   *
-   * @remarks
-   * The same {@link EventMockConfigurator} the event module is configured
-   * from, so a change made here — such as canceling an event through
-   * {@link EventMockConfigurator.intercept} — is what the module sees.
-   *
-   * @returns The event mock configurator.
-   */
-  public get event(): EventMockConfigurator {
-    return this._getConfig<EventMockConfigurator>(eventModuleKey);
-  }
-
   /**
    * Configures the registry services are resolved from.
    *

packages/framework/src/mock/FrameworkMockConfigurator.ts

@@ -193,10 +193,4 @@ await modules.event.dispatchEvent('myEvent', {
   detail: data,
   canBubble: false,
 });
-```
-
-## Testing
-
-Import from `@equinor/fusion-framework-module-event/mock` to record every event dispatched through the framework and intercept or cancel one before its listeners run, instead of subscribing to `event$` or racing `addEventListener` against dispatch order.
-
-See [Testing](docs/testing.md) for `enableEventMock`, `EventMockConfigurator`, and `FrameworkMockConfigurator.event`.
+```

packages/modules/event/README.md

@@ -9,19 +9,12 @@
     ".": {
       "import": "./dist/esm/index.js",
       "types": "./dist/types/index.d.ts"
-    },
-    "./mock": {
-      "import": "./dist/esm/mock/index.js",
-      "types": "./dist/types/mock/index.d.ts"
     }
   },
   "typesVersions": {
     "*": {
       ".": [
         "dist/types/index.d.ts"
-      ],
-      "mock": [
-        "dist/types/mock/index.d.ts"
       ]
     }
   },