Open conorhastings opened 7 years ago
@conorhastings That would be awesome - I've been pondering the best approach myself. Right now the renderer doesn't use an intermediary, so it's straight to string concat. I think the simplest approach would be to add a { json:true }
option added that builds up objects instead of strings. It might end up being quite a few branches in the code though, not sure what to do about that. I guess another option would be to always render to Objects first, then if { json:true }
isn't passed do the xml/html stringification, but that'd make this two-pass, which would be quite a bit worse performance.
I have been doing just that in my current project using jest-serializer-html-string to serialize the html correctly
if that interests anyone I can put together a small boilerplate but should be fairly straightforward, just @developit example + jest snapshotSerializer config
that would be very useful @ruyadorno
@ruyadorno great job 👍 our project(https://github.com/ant-design/ant-design-mobile) use jest snapshot test, if preact can works with jest snapshot test, then we will be more confident to use preact replace react
I know you all have seen it in twitter but I'll leave the boilerplate link here for future reference:
https://github.com/ruyadorno/preact-jest-snapshot-test-boilerplate
@ruyadorno @developit seems like the render result is consistent, but html string is not serialized as the same way of enzyme-to-json https://github.com/ant-design/ant-design-mobile/pull/1306
@ruyadorno as https://github.com/ant-design/ant-design-mobile/pull/1306 ci result shownd, The problem is jest-serializer-html-string not place html attribute at new line, so the react snapshot not be consistent with preact snapshot, I will create a PR to fix this.
the difference is whether put each prop on a separate line. from https://github.com/adriantoine/enzyme-to-json#user-content-serializer:
This is inspired by jest-serializer-enzyme, I first added a note to jest-serializer-enzyme but I then realised that the output is different, so it is not retro compatible with enzyme-to-json because it's using Enzyme debug helper which doesn't put each prop on a separate line.
@paranoidjk ah - you probably want to use the JSX string renderer:
import preactRender from 'preact-render-to-string/jsx';
preactRender(<foo a="b" c="d">bar</foo>)
/*
<foo
a="b"
c="d"
>
bar
</foo>
*/
@paranoidjk yeah, even though it might be valid html, it's def not the standard way html is usually rendered, I'd rather stick with the default options from the html beautifier within jest-serializer-html-string as that would make more sense for most people (including me).
@ruyadorno jest-serializer-html-string doesn't seem to match jest's rendering behaviour. Jest is more like the jsx renderer, which has the advantage of clearer diffs on attribute changes. Because each attribute is a single line it's much easier to see which ones are removed and added when multiple attribute changes are made for a single component.
I love your package though, keep up the good work 👍
Trying to think of the slimmest way to produce JSON output from the string renderer. It's the same flow, just seems awkward to have to have that many branches.
@ruyadorno thanks for sharing your solution
Just came across this thread, my fork of preact-render-to-string renders to JSON: preact-render-to-json and is compatible with Jest and React testing frameworks.
@nathancahill Posting this here, as you don't have issues enabled on your repo: preact-render-to-json doesn't work with hooks, because of developit/preact#1373
Right now using preact-render-to-string directly you end up with double encoding in the snapshot output
see this twitter thread -- https://twitter.com/stillconor/status/805567906310451200 -- for more details.
cc @developit , I'd be interesting in taking this on if you could provide high level guidance on how this would would be worked on.