Move serializers setup to jest config to be possible to override them

[yurii-sorokin]

#125

[thymikee]

Shouldn't the order be different? Have you tested if that changes anything?

[yurii-sorokin]

Just tried both and seems there is no any difference. Will look how jest applies this config.

[yurii-sorokin]

I bet I've seen something about the order in jest docs, but can't find it now. Anyway, jest reverses the list:

config.snapshotSerializers
    .concat()
    .reverse()
    .forEach(path => {
      addSerializer(localRequire(path));
    });

So it's same as before.

[ahnpnl]

nice 👍

[thymikee]

published v5.1.0 with this change

[ghost]

Does this mean I can now change the config for 'pretty-format'?

…

On 22 Feb 2018 08:52, "Michał Pierzchała" ***@***.***> wrote: published v5.1.0 with this change — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#126 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AQN8ea9o-bjakntlO5pZlt9ujER0dotoks5tXSrGgaJpZM4SO4VT> .

[yurii-sorokin]

You can try this serializer that uses the default serializers (built-in plugins for pretty-format). Basically, it transforms component fixture to react test object (mostly, as it does not convert dom nodes needed when using [innerHTML]). Which jest-snapshot understand well.

Or you can write your own serializer, for example, that uses require('util').inspect to serialize fixture or anything you like.