test: add "describeMethods" scope

Author: aleclarson

packages/core/src/SpringValue.test.ts

@@ -15,41 +15,7 @@ describe('SpringValue', () => {
   })
 
   describeProps()
-
-  describe('"set" method', () => {
-    it('stops the active animation', async () => {
-      const spring = new SpringValue(0)
-      const promise = spring.start(1)
-
-      await advanceUntilValue(spring, 0.5)
-      spring.set(2)
-
-      expect(spring.idle).toBeTruthy()
-      expect(await promise).toMatchObject({
-        finished: false,
-        value: 2,
-      })
-    })
-
-    describe('when a new value is passed', () => {
-      it('calls the "onChange" prop', () => {
-        const onChange = jest.fn()
-        const spring = new SpringValue(0, { onChange })
-        spring.set(1)
-        expect(onChange).toBeCalledWith(1, spring)
-      })
-      it.todo('wraps the "onChange" call with "batchedUpdates"')
-    })
-
-    describe('when the current value is passed', () => {
-      it('skips the "onChange" call', () => {
-        const onChange = jest.fn()
-        const spring = new SpringValue(0, { onChange })
-        spring.set(0)
-        expect(onChange).not.toBeCalled()
-      })
-    })
-  })
+  describeMethods()
 
   describeTarget('another SpringValue', from => {
     const node = new SpringValue(from)
@@ -128,6 +94,43 @@ function describeConfigProp() {
   })
 }
 
+function describeMethods() {
+  describe('"set" method', () => {
+    it('stops the active animation', async () => {
+      const spring = new SpringValue(0)
+      const promise = spring.start(1)
+
+      await advanceUntilValue(spring, 0.5)
+      spring.set(2)
+
+      expect(spring.idle).toBeTruthy()
+      expect(await promise).toMatchObject({
+        finished: false,
+        value: 2,
+      })
+    })
+
+    describe('when a new value is passed', () => {
+      it('calls the "onChange" prop', () => {
+        const onChange = jest.fn()
+        const spring = new SpringValue(0, { onChange })
+        spring.set(1)
+        expect(onChange).toBeCalledWith(1, spring)
+      })
+      it.todo('wraps the "onChange" call with "batchedUpdates"')
+    })
+
+    describe('when the current value is passed', () => {
+      it('skips the "onChange" call', () => {
+        const onChange = jest.fn()
+        const spring = new SpringValue(0, { onChange })
+        spring.set(0)
+        expect(onChange).not.toBeCalled()
+      })
+    })
+  })
+}
+
 /** The minimum requirements for testing a dynamic target */
 type OpaqueTarget = {
   node: FrameValue