In rare circumstances, Simulation.run targets can go out of date. Targets are now stored by-name.

Author: japgolly

HISTORY.md

@@ -3,6 +3,7 @@
 * `ComponentOrNode` moved to test module. Renamed to `ReactOrDomNode`.
 * Changed overloaded `classSet` methods into `classSet{,1}{,M}`.
 * Added `simulateKeyDownUp` and `simulateKeyDownPressUp` to `KeyboardEventData` in the test module.
+* In rare circumstances, `Simulation.run` targets can go out of date. Targets are now stored by-name.
 
 # 0.6.0 ([commit log](https://github.com/japgolly/scalajs-react/compare/v0.5.4...v0.6.0))
 

test/src/main/scala/japgolly/scalajs/react/test/Simulation.scala

@@ -6,20 +6,26 @@ import ReactTestUtils.Simulate
 /**
  * Allows composition and abstraction of `ReactTestUtils.Simulate` procedures.
  */
-case class Simulation(run: ReactOrDomNode => Unit) {
+class Simulation(_run: (() => ReactOrDomNode) => Unit) {
+
+  def run(n: => ReactOrDomNode): Unit =
+    _run(() => n)
 
   def andThen(f: Simulation) =
-    Simulation(n => { run(n); f.run(n) })
+    new Simulation(n => { _run(n); f.run(n()) })
 
   @inline final def >>     (f: Simulation) = this andThen f
   @inline final def compose(f: Simulation) = f andThen this
 
   final def runN(cs: ReactOrDomNode*): Unit =
-    cs foreach run
+    cs foreach (run(_))
 }
 
 object Simulation {
 
+  def apply(run: (=> ReactOrDomNode) => Unit): Simulation =
+    new Simulation(n => run(n()))
+
   // Don't use default arguments - they force parentheses on to caller.
   // Eg. Simulation.blur >> Simulation.focus becomes Simulation.blur() >> Simulation.focus(). Yuk.
 

test/src/test/scala/japgolly/scalajs/react/test/TestTest.scala

@@ -75,6 +75,16 @@ object TestTest extends TestSuite {
         events mustEqual Vector("focus", "change", "blur")
         i.getDOMNode().value mustEqual "good"
       }
+      'targetByName {
+        val c = ReactTestUtils.renderIntoDocument(IC())
+        var count = 0
+        def tgt = {
+          count += 1
+          Sel("input").findIn(c)
+        }
+        Simulation.focusChangeBlur("-") run tgt
+        assert(count == 3)
+      }
     }
   }
 }