@363045841yyt/klinechart-core 0.9.0-alpha.2 → 0.9.0-alpha.3

package/src/engine/data/chartDataManager.ts

@@ -6,12 +6,13 @@ import { TimeShareBuffer } from '../../data/timeShareBuffer'
 import type { TimeShareFetcherFn } from '../../data/types'
 import { createSignal, type ReadonlySignal, type Signal } from '../../foundation/reactivity/signal'
 import type { KLineData, TimeShareData } from '../../foundation/types/price'
-import type { ChartDom, Viewport } from '../chartTypes'
+import type { ChartDom } from '../chartTypes'
 import type { VisibleRange, UpdateLevel } from '../layout/pane'
 import { getPhysicalKLineConfig } from '../utils/klineConfig'
-import { getVisibleRange } from '../viewport/viewport'
 import type { DataStateModule } from '../state/dataState'
 import type { DataManagerStateModule } from '../state/dataManagerState'
+import type { ViewportStateModule } from '../state/viewportState'
+import type { ComparisonStateModule } from '../state/comparisonState'
 
 import { ComparisonManager } from './comparisonManager'
 import { FetchBatchScheduler } from './fetchBatchScheduler'
@@ -20,17 +21,11 @@ import { ScrollCompensator } from './scrollCompensator'
 
 export interface DataDependencies {
   getOption: () => { kWidth: number; kGap: number }
-  getEffectiveDpr: () => number
-  getLogicalScrollLeft: () => number
-  getCachedScrollLeft: () => number
-  setScrollLeft: (v: number) => void
   getDom: () => ChartDom
-  getObservedSize: () => { width: number; height: number }
-  getViewport: () => Viewport | null
-  getVisibleRange: () => VisibleRange | null
-  /** 几何 SSOT：由 kernel viewport.readonly 注入 */
-  getLeftLoadBufferWidth: () => number
-  getContentWidth: () => number
+  /** scroll / dpr / 可见区间 / 几何 SSOT */
+  viewport: ViewportStateModule
+  /** 对比叠加状态 SSOT */
+  comparison: ComparisonStateModule
   scheduleDraw: (level?: UpdateLevel) => void
   resetInteraction: () => void
   getIndicatorScheduler: () => {
@@ -40,11 +35,8 @@ export interface DataDependencies {
   isPointerDown: () => boolean
   onTimeShareDataReady: (dataLength: number) => void
   onDataProcessed?: (data: KLineData[], range: VisibleRange) => void
+  /** 写 symbols 选择（含 primary + comparison） */
   setSymbols: (symbols: ReadonlyArray<SymbolSpec>) => void
-  setComparisonLoading: (loading: boolean) => void
-  comparisonSpecs$: ReadonlySignal<ReadonlyArray<SymbolSpec>>
-  comparisonColors$: ReadonlySignal<ReadonlyMap<string, string>>
-  comparisonLoading$: ReadonlySignal<boolean>
 }
 
 const BUF_PRIMARY = 'main'
@@ -95,10 +87,10 @@ export class ChartDataManager {
       getKLineBuffer: (key) => this._klineBuffers.get(key),
       getKLineBufferKeys: () => [...this._klineBuffers.keys()],
       scheduleDraw: () => this.deps.scheduleDraw(),
-      getSpecs: () => this.deps.comparisonSpecs$.peek(),
-      setLoading: (loading) => this.deps.setComparisonLoading(loading),
+      getSpecs: () => this.deps.comparison.readonly.specs.peek(),
+      setLoading: (loading) => this.deps.comparison.actions.setLoading(loading),
     })
-    this._comparisonSpecsUnsub = this.deps.comparisonSpecs$.subscribe(() => {
+    this._comparisonSpecsUnsub = this.deps.comparison.readonly.specs.subscribe(() => {
       this.reconcileComparisonBuffers()
     })
     this.reconcileComparisonBuffers()
@@ -305,7 +297,7 @@ export class ChartDataManager {
       this._dmState.actions.setRangeInitialized(true)
     }
 
-    let currentRange = this.deps.getVisibleRange()
+    let currentRange = this.getVisibleRangeOrNull()
     if (!currentRange && this._dmState.readonly.rangeInitialized.peek() && bufferData.length > 0) {
       currentRange = { start: 0, end: bufferData.length }
     }
@@ -325,7 +317,7 @@ export class ChartDataManager {
   }
 
   private recordIncrementalLoad(prependedCount: number): void {
-    this._dmState.actions.recordIncrementalLoad(prependedCount, this.deps.getLeftLoadBufferWidth())
+    this._dmState.actions.recordIncrementalLoad(prependedCount, this.deps.viewport.readonly.leftLoadBufferWidth.peek())
   }
 
   private scheduleIncrementalLoadHintFlush(key: string): void {
@@ -370,7 +362,7 @@ export class ChartDataManager {
   // ── Internal helpers ──
 
   getLeftLoadBufferWidth(): number {
-    return this.deps.getLeftLoadBufferWidth()
+    return this.deps.viewport.readonly.leftLoadBufferWidth.peek()
   }
 
   private getActiveKLineLength(): number {
@@ -378,9 +370,21 @@ export class ChartDataManager {
     return buf ? buf.getRawData().length : 0
   }
 
+  /** 无 viewport / 无数据时返回 null；clamped 可索引区间（start>=0） */
+  private getVisibleRangeOrNull(): VisibleRange | null {
+    if (this.deps.viewport.readonly.viewWidth.peek() === 0) return null
+    return this.deps.viewport.readonly.visibleRange.peek()
+  }
+
+  /** raw 可见区间（含左右扩窗，start 可能为 -1）；供增量加载左缘检测 */
+  private getRawVisibleRangeOrNull(): VisibleRange | null {
+    if (this.deps.viewport.readonly.viewWidth.peek() === 0) return null
+    return this.deps.viewport.readonly.rawVisibleRange.peek()
+  }
+
   /** 当前可见范围（on-demand 实时计算，消除 stale 缓存） */
   getCurrentVisibleRange(): VisibleRange | null {
-    return this.deps.getVisibleRange()
+    return this.getVisibleRangeOrNull()
   }
 
   /** Unified data signal — always reflects the active buffer's data */
@@ -469,7 +473,7 @@ export class ChartDataManager {
   }
 
   getComparisonSpecs(): SymbolSpec[] {
-    return this.deps.comparisonSpecs$.peek().map((spec) => ({ ...spec }))
+    return this.deps.comparison.readonly.specs.peek().map((spec) => ({ ...spec }))
   }
 
   get dataBuffer(): KLineBuffer {
@@ -485,15 +489,15 @@ export class ChartDataManager {
   }
 
   get comparisonColors(): ReadonlySignal<ReadonlyMap<string, string>> {
-    return this.deps.comparisonColors$
+    return this.deps.comparison.readonly.colors
   }
 
   get comparisonLoading(): ReadonlySignal<boolean> {
-    return this.deps.comparisonLoading$
+    return this.deps.comparison.readonly.loading
   }
 
   getComparisonColors(): Map<string, string> {
-    return new Map(this.deps.comparisonColors$.peek())
+    return new Map(this.deps.comparison.readonly.colors.peek())
   }
 
   // ── Data updates (KLine) ──
@@ -554,20 +558,22 @@ export class ChartDataManager {
     if (data.length === 0) return
     const window = buf.loadedWindow
     if (!window) return
-    const range = this.deps.getVisibleRange()
-    if (!range) return
+    // 左缘扩窗检测必须用 raw（start 可为 -1）；数据下标用 clamped
+    const rawRange = this.getRawVisibleRangeOrNull()
+    const range = this.getVisibleRangeOrNull()
+    if (!rawRange || !range) return
 
     const MS_PER_DAY = 86_400_000
     const spec = buf.currentSpec
     const gapDays = getPeriodDays(spec?.period)
     let firstVisibleTs: number | undefined
 
-    if (range.start < 0 && this._dataFetcher) {
+    if (rawRange.start < 0 && this._dataFetcher) {
       const earlierThanEarliest = window.earliestTs - gapDays * MS_PER_DAY
       buf.ensureRange(earlierThanEarliest, window.earliestTs)
       firstVisibleTs = data[0]?.timestamp
     } else if (range.start < data.length) {
-      firstVisibleTs = data[Math.max(0, range.start)]?.timestamp
+      firstVisibleTs = data[range.start]?.timestamp
       if (firstVisibleTs !== undefined && firstVisibleTs < window.earliestTs) {
         buf.ensureRange(firstVisibleTs, window.earliestTs)
       }
@@ -587,18 +593,18 @@ export class ChartDataManager {
 
   addComparisonSymbol(spec: SymbolSpec): void {
     const primary = this._dataState.readonly.symbols.peek()[0]
-    if (!primary || this.deps.comparisonSpecs$.peek().some((item) => item.symbol === spec.symbol))
+    if (!primary || this.deps.comparison.readonly.specs.peek().some((item) => item.symbol === spec.symbol))
       return
-    this.deps.setSymbols([primary, ...this.deps.comparisonSpecs$.peek(), spec])
+    this.deps.setSymbols([primary, ...this.deps.comparison.readonly.specs.peek(), spec])
   }
 
   setComparisonData(symbol: string, data: KLineData[]): void {
     const primary = this._dataState.readonly.symbols.peek()[0]
     if (!primary) return
-    if (!this.deps.comparisonSpecs$.peek().some((spec) => spec.symbol === symbol)) {
+    if (!this.deps.comparison.readonly.specs.peek().some((spec) => spec.symbol === symbol)) {
       this.deps.setSymbols([
         primary,
-        ...this.deps.comparisonSpecs$.peek(),
+        ...this.deps.comparison.readonly.specs.peek(),
         { symbol, period: 'daily' },
       ])
     }
@@ -607,11 +613,11 @@ export class ChartDataManager {
 
   removeComparisonSymbol(symbol: string): void {
     const primary = this._dataState.readonly.symbols.peek()[0]
-    if (!primary || !this.deps.comparisonSpecs$.peek().some((spec) => spec.symbol === symbol))
+    if (!primary || !this.deps.comparison.readonly.specs.peek().some((spec) => spec.symbol === symbol))
       return
     this.deps.setSymbols([
       primary,
-      ...this.deps.comparisonSpecs$.peek().filter((spec) => spec.symbol !== symbol),
+      ...this.deps.comparison.readonly.specs.peek().filter((spec) => spec.symbol !== symbol),
     ])
     this.deps.scheduleDraw()
   }
@@ -636,19 +642,8 @@ export class ChartDataManager {
     const dataLen = kRaw?.length ?? 0
     let visibleStart = 0
     if (dataLen > 0) {
-      const vp = this.deps.getViewport()
-      if (vp) {
-        const opt = this.deps.getOption()
-        const vRange = getVisibleRange(
-          vp.scrollLeft,
-          vp.plotWidth,
-          opt.kWidth,
-          opt.kGap,
-          dataLen,
-          vp.dpr,
-        )
-        visibleStart = vRange ? Math.max(0, vRange.start) : 0
-      }
+      const vRange = this.getVisibleRangeOrNull()
+      visibleStart = vRange ? Math.max(0, vRange.start) : 0
     }
     // 双路径守卫：
     //   - switchToTimeShareForDate 路径：setTimeShareQueryDate 已在
@@ -693,7 +688,7 @@ export class ChartDataManager {
       return
     }
     const next = { ...current, period }
-    this.setSymbols([next, ...this.deps.comparisonSpecs$.peek()])
+    this.setSymbols([next, ...this.deps.comparison.readonly.specs.peek()])
   }
 
   /**
@@ -725,7 +720,7 @@ export class ChartDataManager {
       incremental: false,
       source: source.source ?? 'custom',
     }
-    this.setSymbols([spec, ...this.deps.comparisonSpecs$.peek()])
+    this.setSymbols([spec, ...this.deps.comparison.readonly.specs.peek()])
 
     const symbolCode = spec.symbol
     if (symbolCode) {
@@ -760,7 +755,7 @@ export class ChartDataManager {
     this._dataState.actions.setData([])
     this._dmState.actions.setRangeInitialized(false)
     this._dmState.actions.setSavedScrollTimestamp(null)
-    this.setSymbols([spec, ...this.deps.comparisonSpecs$.peek()])
+    this.setSymbols([spec, ...this.deps.comparison.readonly.specs.peek()])
   }
 
   getPreCustomSpec(): SymbolSpec | null {
@@ -872,12 +867,12 @@ export class ChartDataManager {
     )
     this._dmState.actions.setSavedScrollTimestamp(null)
     if (idx >= 0) {
-      const dpr = this.deps.getEffectiveDpr()
+      const dpr = this.deps.viewport.readonly.dpr.peek()
       const opt = this.deps.getOption()
       const { unitPx, startXPx } = getPhysicalKLineConfig(opt.kWidth, opt.kGap, dpr)
       const leftBuffer = this.getLeftLoadBufferWidth()
       const scrollLeft = ((idx + 1) * unitPx + startXPx) / dpr + leftBuffer
-      this.deps.setScrollLeft(scrollLeft)
+      this.deps.viewport.actions.scrollTo(scrollLeft)
       return true
     }
     return false
@@ -886,7 +881,7 @@ export class ChartDataManager {
   // ── Content width ──
 
   getContentWidth(): number {
-    return this.deps.getContentWidth()
+    return this.deps.viewport.readonly.contentWidth.peek()
   }
 
   scrollToRight(): void {
@@ -962,12 +957,12 @@ export class ChartDataManager {
   }
 
   getLogicalIndexAtX(mouseX: number): number | null {
-    const vp = this.deps.getViewport()
-    if (!vp) return null
+    if (this.deps.viewport.readonly.viewWidth.peek() === 0) return null
+    const vp = this.deps.viewport.readonly.viewport.peek()
     const buf = this.getActiveDataBuffer()
     const data = buf ? buf.getRawData() : []
     if (data.length === 0) return null
-    const dpr = this.deps.getEffectiveDpr()
+    const dpr = this.deps.viewport.readonly.dpr.peek()
     const opt = this.deps.getOption()
     const { startXPx, unitPx } = getPhysicalKLineConfig(opt.kWidth, opt.kGap, dpr)
     const worldX = Math.round((vp.scrollLeft + mouseX) * dpr)

package/src/engine/data/incrementalLoadHint.ts

@@ -1,14 +1,14 @@
 import type { ChartDom } from '../chartTypes'
 import { getPhysicalKLineConfig } from '../utils/klineConfig'
+import type { ViewportStateModule } from '../state/viewportState'
 
 /** 与 theme selectionFill 对齐：light #2D7FF933 / dark #4A9EFF33；fallback 为 light 默认 */
 const INCREMENTAL_LOAD_HINT_BG = 'var(--klc-color-selection-fill, #2D7FF933)'
 
 export interface HintDeps {
   getOption: () => { kWidth: number; kGap: number }
-  getEffectiveDpr: () => number
   getDom: () => ChartDom
-  getViewport: () => { viewHeight: number } | null
+  viewport: ViewportStateModule
 }
 
 interface ActiveHint {
@@ -86,17 +86,18 @@ export class IncrementalLoadHint {
   }
 
   private _updateGeometry(hint: HTMLDivElement, count: number, leftBufferWidth: number): void {
-    const dpr = this.deps.getEffectiveDpr()
+    const dpr = this.deps.viewport.readonly.dpr.peek()
     const opt = this.deps.getOption()
     const { unitPx, startXPx } = getPhysicalKLineConfig(opt.kWidth, opt.kGap, dpr)
 
     hint.style.left = `${leftBufferWidth}px`
     const width = (startXPx + count * unitPx) / dpr
     hint.style.width = `${Math.max(0, width)}px`
-    hint.style.height = `${Math.max(
-      0,
-      this.deps.getViewport()?.viewHeight ?? this.deps.getDom().container?.clientHeight ?? 0,
-    )}px`
+    const viewHeight =
+      this.deps.viewport.readonly.viewHeight.peek() ||
+      this.deps.getDom().container?.clientHeight ||
+      0
+    hint.style.height = `${Math.max(0, viewHeight)}px`
   }
 
   private _fadeOutAndRemove(entry: ActiveHint): void {

package/src/engine/data/scrollCompensator.ts

@@ -1,17 +1,10 @@
-import type { ChartDom, Viewport } from '../chartTypes'
 import { getPhysicalKLineConfig } from '../utils/klineConfig'
+import type { ViewportStateModule } from '../state/viewportState'
 
 export interface ScrollDeps {
   getOption: () => { kWidth: number; kGap: number }
-  getEffectiveDpr: () => number
-  getCachedScrollLeft: () => number
-  setScrollLeft: (v: number) => void
-  getDom: () => ChartDom
-  getObservedSize: () => { width: number; height: number }
-  getViewport: () => Viewport | null
-  /** 几何 SSOT：由 kernel viewport.readonly 注入 */
-  getLeftLoadBufferWidth: () => number
-  getContentWidth: () => number
+  /** scroll / dpr / 几何 SSOT */
+  viewport: ViewportStateModule
 }
 
 export const SCROLL_TRAILING_SLOTS = 30
@@ -20,53 +13,52 @@ export class ScrollCompensator {
   constructor(private deps: ScrollDeps) {}
 
   compensatePrepend(count: number): void {
-    const dpr = this.deps.getEffectiveDpr()
+    const dpr = this.deps.viewport.readonly.dpr.peek()
     const opt = this.deps.getOption()
     const { unitPx } = getPhysicalKLineConfig(opt.kWidth, opt.kGap, dpr)
     const compensation = (count * unitPx) / dpr
-    const nextScrollLeft = this.deps.getCachedScrollLeft() + compensation
-    this.deps.setScrollLeft(nextScrollLeft)
+    const nextScrollLeft = this.deps.viewport.readonly.scrollLeft.peek() + compensation
+    this.deps.viewport.actions.scrollTo(nextScrollLeft)
   }
 
   adjustScrollAfterDataChange(dataLength: number): void {
-    const scrollLeft = this.deps.getCachedScrollLeft()
+    const scrollLeft = this.deps.viewport.readonly.scrollLeft.peek()
     if (scrollLeft < 0) return
 
     if (scrollLeft <= 0) {
-      this.deps.setScrollLeft(this.deps.getLeftLoadBufferWidth())
+      this.deps.viewport.actions.scrollTo(this.deps.viewport.readonly.leftLoadBufferWidth.peek())
       return
     }
 
-    const dpr = this.deps.getEffectiveDpr()
+    const dpr = this.deps.viewport.readonly.dpr.peek()
     const opt = this.deps.getOption()
     const { unitPx, startXPx } = getPhysicalKLineConfig(opt.kWidth, opt.kGap, dpr)
     const totalDataWidth = (startXPx + dataLength * unitPx) / dpr
-    const leftBuffer = this.deps.getLeftLoadBufferWidth()
+    const leftBuffer = this.deps.viewport.readonly.leftLoadBufferWidth.peek()
     if (scrollLeft >= leftBuffer + totalDataWidth) {
-      this.deps.setScrollLeft(leftBuffer)
+      this.deps.viewport.actions.scrollTo(leftBuffer)
     }
   }
 
   scrollToRight(dataLength: number): void {
     if (dataLength === 0) return
-    const dpr = this.deps.getEffectiveDpr()
+    const dpr = this.deps.viewport.readonly.dpr.peek()
     const opt = this.deps.getOption()
     const { unitPx, startXPx } = getPhysicalKLineConfig(opt.kWidth, opt.kGap, dpr)
     const lastKLineEndPx = (startXPx + dataLength * unitPx) / dpr
-    const viewport = this.deps.getViewport()
-    const clientWidth = viewport?.viewWidth ?? 0
+    const clientWidth = this.deps.viewport.readonly.viewWidth.peek()
     if (clientWidth <= 0) return
 
-    const leftBuffer = this.deps.getLeftLoadBufferWidth()
+    const leftBuffer = this.deps.viewport.readonly.leftLoadBufferWidth.peek()
     let target: number
     if (lastKLineEndPx <= clientWidth) {
       target = leftBuffer - (clientWidth - lastKLineEndPx)
     } else {
       target = leftBuffer + (lastKLineEndPx - clientWidth)
     }
-    const contentWidth = this.deps.getContentWidth()
+    const contentWidth = this.deps.viewport.readonly.contentWidth.peek()
     const maxScroll = Math.max(0, contentWidth - clientWidth)
     const scrollLeft = Math.round(Math.max(0, Math.min(target, maxScroll)) * dpr) / dpr
-    this.deps.setScrollLeft(scrollLeft)
+    this.deps.viewport.actions.scrollTo(scrollLeft)
   }
 }

package/src/engine/indicators/__tests__/chartIndicatorManager.test.ts

@@ -52,7 +52,7 @@ function createMockDeps() {
     upsertPane: vi.fn(),
     removePaneDefinition: vi.fn(),
     getPaneSpecs: vi.fn(() => []),
-    getPaneRatiosSignal: () => paneRatiosSignal,
+    paneRatios$: paneRatiosSignal,
     paneSpecs$: paneSpecsSignal,
     getInternalPaneRatios: vi.fn(() => new Map()),
     setInternalPaneRatio: vi.fn(),
@@ -68,30 +68,22 @@ function createMockDeps() {
     removeLayer: vi.fn(() => false),
     getLayer: vi.fn(() => null),
     setLayerVisibility: vi.fn(),
-    mainIndicators$: indicatorState.readonly.mainIndicators,
-    upsertMainIndicator: (id, p) => indicatorState.actions.upsert(id, p),
-    removeMainIndicator: (id) => indicatorState.actions.remove(id),
-    setMainIndicatorParams: (id, p) => indicatorState.actions.setParams(id, p),
-    replaceMainIndicators: (entries) => indicatorState.actions.replaceAll(entries),
-    clearMainIndicators: () => indicatorState.actions.clear(),
-    subPanes$: subPaneState.readonly.entries,
-    createSubPaneState: vi.fn((paneId, indicatorId, params) =>
-      subPaneState.actions.upsert({ paneId, indicatorId, params }),
-    ),
-    removeSubPaneState: vi.fn((paneId) => subPaneState.actions.remove(paneId)),
-    replaceSubPaneState: vi.fn((paneId, indicatorId, params) =>
-      subPaneState.actions.replace({ paneId, indicatorId, params }),
-    ),
-    updateSubPaneStateParams: vi.fn((paneId, params) =>
-      subPaneState.actions.setParams(paneId, params),
-    ),
-    clearSubPaneState: vi.fn(() => subPaneState.actions.clear()),
+    indicator: indicatorState,
+    subPaneOps: {
+      entries: subPaneState.readonly.entries,
+      create: vi.fn((paneId, indicatorId, params) =>
+        subPaneState.actions.upsert({ paneId, indicatorId, params }),
+      ),
+      remove: vi.fn((paneId) => subPaneState.actions.remove(paneId)),
+      replace: vi.fn((paneId, indicatorId, params) =>
+        subPaneState.actions.replace({ paneId, indicatorId, params }),
+      ),
+      setParams: vi.fn((paneId, params) => subPaneState.actions.setParams(paneId, params)),
+      clear: vi.fn(() => subPaneState.actions.clear()),
+    },
     projectPaneLayout: vi.fn(),
     runRendererTransaction: (run) => run(),
     getIndicatorScheduler: vi.fn(),
-    getRightAxisWidth: () => 60,
-    getPriceLabelWidth: () => 60,
-    getYPaddingPx: () => 4,
   } as IndicatorDependencies & { rendererMap: Map<string, any> }
 }
 
@@ -169,9 +161,9 @@ describe('ChartIndicatorManager', () => {
     it('creates sub-pane business state before projecting runtime resources', () => {
       expect(manager.createSubPane('RSI_0', 'RSI', { period1: 6 })).toBe(true)
 
-      expect(deps.createSubPaneState).toHaveBeenCalledTimes(1)
+      expect(deps.subPaneOps.create).toHaveBeenCalledTimes(1)
       expect(deps.useRenderer).toHaveBeenCalled()
-      expect(deps.createSubPaneState.mock.invocationCallOrder[0]).toBeLessThan(
+      expect(deps.subPaneOps.create.mock.invocationCallOrder[0]).toBeLessThan(
         deps.useRenderer.mock.invocationCallOrder[0]!,
       )
       expect(manager.getSubPaneEntry('RSI_0')?.params).toEqual({ period1: 6 })
@@ -183,7 +175,7 @@ describe('ChartIndicatorManager', () => {
 
       expect(manager.createSubPane('RSI_0', 'MACD', { fast: 5 })).toBe(true)
 
-      expect(deps.replaceSubPaneState).not.toHaveBeenCalled()
+      expect(deps.subPaneOps.replace).not.toHaveBeenCalled()
       expect(manager.getSubPaneEntry('RSI_0')?.indicatorId).toBe('RSI')
       expect(manager.getSubPaneEntry('RSI_0')?.params).toEqual({ period1: 6 })
     })
@@ -213,7 +205,7 @@ describe('ChartIndicatorManager', () => {
       manager.destroy()
       vi.clearAllMocks()
 
-      deps.upsertMainIndicator('MA', { ma5: true })
+      deps.indicator.actions.upsert('MA', { ma5: true })
 
       expect(deps.useRenderer).not.toHaveBeenCalled()
       expect(deps.scheduleDraw).not.toHaveBeenCalled()