test: enable integation tests for image sharp

[wardpeet]

Description

Adding some integration tests for our lazy builds setup. It's primarily going to be used to validate #10964. And probably more issues that we need to validate on a real run instead of just testing the functions separately.

Related Issues

#10964, #7348, #10815

[DSchau]

This is looking awesome! Wish there were some way we could move off of all the hard-coded paths (because this really won't be resilient to changes if we make any changes--but maybe that's OK?), but beyond that--the functionality seems solid!

One comment on the above: maybe we could readdir on public/static/image-hash and then ensure there are X images rather than checking each image directly?

[wardpeet]

One comment on the above: maybe we could readdir on public/static/image-hash and then ensure there are X images rather than checking each image directly?

this is probably more maintainable 👍

[DSchau]

@wardpeet heads up I fixed the CircleCI config errors.

Super helpful to use the circleci CLI -> https://circleci.com/docs/2.0/local-cli/#installation

brew install circleci is my preferred way!

[wardpeet]

@wardpeet heads up I fixed the CircleCI config errors.

Super helpful to use the circleci CLI -> circleci.com/docs/2.0/local-cli/#installation

brew install circleci is my preferred way!

Sweet thanks a bunch! I'm on a linux vm now so I'm using apt-get but I saw that homebrew is now also available on linux

[DSchau]

Left some comments; great work!

This is pretty much good to go!

[DSchau]

Merely an alternate approach--both work well here!

const roots = glob
  .sync(`*/`, {
    cwd: path.join(process.cwd(), `integration-tests`),
  })
  .map(testPath => `<rootDir>/integration-tests/${testPath}`)

[wardpeet]

you probably want to replace process.cwd() with __dirname?
I copied this from the base config 😛 i'll update in another pr and take the other jest.config with it