Norgate-AV / react-crestron-ch5-hooks

97%

develop: 97%

push

github

feat(hooks): seed initial state from getState on mount

Subscribe hooks now call getState() synchronously during the useState
lazy initialiser so that on re-mounts (page/popup transitions, conditional
rendering) the hook returns the current signal value immediately rather
than the hard-coded type default (0 / false / "").

Affects all six subscribe hooks:
- useCrestronSubscribeAnalog
- useCrestronSubscribeDigital
- useCrestronSubscribeSerial
- useCrestronSubscribeAnalogCollection
- useCrestronSubscribeDigitalCollection
- useCrestronSubscribeSerialCollection

Note: cold-start behaviour is unchanged — getState returns null until
Csig.State_Synchronization completes, at which point the subscribe
callback fires and updates state as before.

Tests: added getState spy to setupSubscribeTest helper and two new cases
per subscribe hook (verifies getState is called with the correct
type/name; verifies the hook initialises from the returned value when
prior state exists).

18 of 24 branches covered (75.0%)

Branch coverage included in aggregate %.

23 of 23 new or added lines in 8 files covered. (100.0%)

207 of 207 relevant lines covered (100.0%)

12.67 hits per line