Remove non-reusable methods from StateSnapshot.withReuse

Author: japgolly

extra/src/main/scala/japgolly/scalajs/react/extra/StateSnapshot.scala

@@ -46,9 +46,6 @@ object StateSnapshot {
     def apply[S](value: S): FromValue[S] =
       new FromValue(value)
 
-    def of[I, S](i: I)(implicit t: StateAccessor.ReadImpureWritePure[I, S], r: Reusability[S]): StateSnapshot[S] =
-      apply(t.state(i)).setStateVia(i)(t, r)
-
     /** This is meant to be called once and reused so that the setState callback stays the same. */
     def prepare[S](f: S => Callback): FromSetStateFn[S] =
       new FromSetStateFn(Reusable.fn(f))
@@ -80,9 +77,6 @@ object StateSnapshot {
     final class FromValue[S](private val value: S) extends AnyVal {
       def apply(set: S ~=> Callback)(implicit r: Reusability[S]): StateSnapshot[S] =
         new StateSnapshot(value, set, r)
-
-      def setStateVia[I](i: I)(implicit t: StateAccessor.WritePure[I, S], r: Reusability[S]): StateSnapshot[S] =
-        apply(Reusable.fn(t setState i))(r)
     }
 
     final class FromSetStateFn[S](private val set: S ~=> Callback) extends AnyVal {

test/src/test/scala/japgolly/scalajs/react/extra/StateSnapshotTest.scala

@@ -59,25 +59,7 @@ object StateSnapshotTest extends TestSuite {
         import japgolly.scalajs.react.test.InferenceUtil._
         def SS = StateSnapshot.withReuse
         implicit def rs: Reusability[S] = ???
-        'of {
-                           test[Render        ](SS.of(_)).expect[StateSnapshot[S]]
-          compileError(""" test[StateAccessP  ](SS.of(_)) """) // lack safe read
-          compileError(""" test[Backend       ](SS.of(_)) """) // lack safe read
-          compileError(""" test[ScalaMountedCB](SS.of(_)) """) // lack safe read
-          compileError(""" test[JsMounted     ](SS.of(_)) """) // use (x.state).setStateVia(x.pure)
-          compileError(""" test[ScalaMountedId](SS.of(_)) """) // use (x.state).setStateVia(x.pure)
-          compileError(""" test[StateAccessI  ](SS.of(_)) """) // use (x.state).setStateVia(x.pure)
-        }
         'apply_apply - test[S ~=> Callback](SS(S)(_)).expect[StateSnapshot[S]]
-        'apply_setStateVia {
-                           test[Render        ](SS(S).setStateVia(_)).expect[StateSnapshot[S]]
-                           test[Backend       ](SS(S).setStateVia(_)).expect[StateSnapshot[S]]
-                           test[ScalaMountedCB](SS(S).setStateVia(_)).expect[StateSnapshot[S]]
-                           test[StateAccessP  ](SS(S).setStateVia(_)).expect[StateSnapshot[S]]
-          compileError(""" test[JsMounted     ](SS(S).setStateVia(_)) """) // use setStateVia(x.pure)
-          compileError(""" test[ScalaMountedId](SS(S).setStateVia(_)) """) // use setStateVia(x.pure)
-          compileError(""" test[StateAccessI  ](SS(S).setStateVia(_)) """) // use setStateVia(x.pure)
-        }
         'zoom {
           def rs = ??? // shadow
           implicit def rt: Reusability[T] = ???