Skip to content

fix(test): update the ForcedTradeStrategy fixture to the current engine listener API - #2

Merged
mrmx merged 1 commit into
mainfrom
fix/store-first-listener-fixture
Jul 26, 2026
Merged

fix(test): update the ForcedTradeStrategy fixture to the current engine listener API#2
mrmx merged 1 commit into
mainfrom
fix/store-first-listener-fixture

Conversation

@mrmx

@mrmx mrmx commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

The integration fixture no longer compiles against a current engine.

Two things changed in the engine's listener API:

  • onChange used to receive a StateStoreSupport and required an initStore(store) call before the store field could be used. It now receives the resolved StateStore directly — the store is event data, handed in alongside the values it belongs to, so there is no init ritual and no store field to hold.
  • The listener base class was renamed AbstractOnChangeListenerAbstractWindowListener, since it is window-specific and the old name collided with java.awt.event.WindowListener in auto-import-heavy editors.
-        public void onChange(StateStoreSupport store, double prev, double actual) {
-            initStore(store);
-            long count = this.store.inc("count");
+        public void onChange(StateStore store, double prev, double actual) {
+            long count = store.inc("count");

Fixture-only change; no library code touched.

…ne listener API

The fixture no longer compiles against a current engine: onChange took a
StateStoreSupport and called initStore(…) to bind a store field, both of which
the engine removed. A listener now receives the resolved StateStore directly, as
event data handed in with the values it belongs to, and the base class it extends
was renamed AbstractOnChangeListener -> AbstractWindowListener.
@mrmx
mrmx temporarily deployed to Preproduction July 26, 2026 19:26 — with GitHub Actions Inactive
@mrmx
mrmx merged commit 887dbaf into main Jul 26, 2026
9 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant