Fork me on GitHub
#re-frame
<
2016-08-03
>
denisj04:08:20

@mattsfrey: Typically unit tests exercise one event handler and it's scope of change is limited, so if a unit test verifies schema, it might choose to just test the parts of app-db known to be effected. But unless you are particularly concerned about performance of checking the entire app-db then I would consider it good practice to do so, because this could catch any unintentional changes the handler has made using an incorrect path into app-db

shader21:08:31

why is it that so many of the assertions in re-frame end up breaking the app so that I have to reload the page?

shader21:08:41

is there anything I can do to fix that?

shader21:08:07

that is, figwheel doesn't work, I have to refresh the whole page

danielcompton21:08:32

@shader, that sounds unusual, re-frame tries to be very robust to errors. Can you give some examples?

rui.yang22:08:34

wonder if anyone using secretary for routing? i wonder how to make it work with figwheel reloading.

rui.yang22:08:29

figwheel will push changes to browser, but because app-db not changed, no rerending happen

rui.yang22:08:08

so basically when changing html wont have effect on browser unless refresh or navigate to other page and navigate back

shaun-mahood22:08:24

@rui.yang: Might be worth checking out the re-frame template, it uses figwheel and has an option to add secretary https://github.com/Day8/re-frame-template It also looks like https://github.com/dhruvp/angular-phonecat-re-frame uses both figwheel and secretary with re-frame as well

rui.yang22:08:05

@shaun-mahood thanks, i will check it out