Don't require storyshots users to transform lodash-es

[tmeasday]

Issue details

As part of #2558 we added a dependency on lodash-es, an uncompiled JS dependency.

This means that Jest needs to be configured to transform the file with:

  transformIgnorePatterns: ['/node_modules/(?!lodash-es/.*)'],

within each users' storyshots-using project.

Steps to reproduce

Create a storyshots project with 3.3.

[tmeasday]

Can we programmatically add to the user's jest config? Feels a little hacky, but then storyshots is pretty hacky ;)

It's probably too late in the piece to do it though :/

[tmeasday]

cc @igor-dv @Hypnosphi

[tmeasday]

The only other option I can think of is to use something like babel-register

[tmeasday]

I don't think babel-register works with jest, so I think that is out.

[igor-dv]

Hmm.. Shit. I've Forgotten about people need their own jest config. Storyshots already use babel-core internally maybe we can somehow use it for that?

[tmeasday]

It also overrides require, so maybe.. let's try some stuff.

[tmeasday]

This line is the place the exception comes from (it isn't clear from the stack trace):

https://github.com/storybooks/storybook/blob/79c07c727a2f5eedf34502df4207d7a6527fedf0/addons/storyshots/src/index.js#L46

[Hypnosphi]

I don't think babel-register works with jest

Why?

[igor-dv]

DIdn't work for me,

[tmeasday]

I don't think babel-register works with jest
Why?

Jest overrides require itself I guess. It doesn't seem like they are compatible.

[tmeasday]

If we want to try and get it working with Jest we should probably follow these for insights into a non-hacky solution: jestjs/jest#4842 jestjs/jest#2205

[eddiemonge]

This is a breaking change and is causing test failures for us

[igor-dv]

Will revert it today

[tmeasday]

This was fixed and released in 3.3.3