nigrosimone / ng-simple-state Goto Github PK

I have a quite big state object and I don't want all my observables to re-fire every time some unrelated part of the state changes.
In the current selectState there is a distinctUntilChanged (docs), but it is sadly not as effective, since without any arguments it would check at object identity which changes in my case as I create copies to keep everything immutable.

My own workaround was to override the selectState:

  // eslint-disable-next-line no-unused-vars
  override selectState<K>(
    selectFn?: (state: Readonly<S>) => K,
    comparator?: (previous: K, current: K) => boolean,
  ): Observable<K> {
    return super.selectState(selectFn).pipe(
      distinctUntilChanged(comparator=comparator)
    )
  }

(I only needed comparator for the time being)

This is okay, but now there is an extra unnecessary distinctUntilChanged in the chain.

Would it be possible to add support for something like this?

How can I do a setup for unit test when a Store depends on HttpClient for example?