Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add tests to gatsby-remark-images #1559

Merged
merged 3 commits into from Jul 19, 2017
Merged

Conversation

DSchau
Copy link
Contributor

@DSchau DSchau commented Jul 19, 2017

Added some basic unit tests (and snapshots) to gatsby-remark-images

Note: there's probably more we can do here, but it's a start! Let me know how it looks

One thing I may want to do is re-run prettier on the code base, so I can do that too

@gatsbybot
Copy link
Collaborator

gatsbybot commented Jul 19, 2017

Deploy preview ready!

Built with commit 6d65616

https://deploy-preview-1559--gatsbyjs.netlify.com

@gatsbybot
Copy link
Collaborator

gatsbybot commented Jul 19, 2017

Deploy preview ready!

Built with commit 6d65616

https://deploy-preview-1559--using-drupal.netlify.com

@KyleAMathews
Copy link
Contributor

KyleAMathews commented Jul 19, 2017

Deploy preview ready!

Built with commit 6d65616

https://deploy-preview-1559--gatsbygram.netlify.com

@KyleAMathews
Copy link
Contributor

Deploy preview failed.

Built with commit 3a3074d

https://app.netlify.com/sites/using-contentful/deploys/596f8b1ca700c44c45258ee6

@KyleAMathews
Copy link
Contributor

Tests! Nice! The tests look spot on. Snapshots are perfect for this sort of thing.

Yeah, running npm run format when committing is great. We need to enforce that.

@DSchau
Copy link
Contributor Author

DSchau commented Jul 19, 2017

Re-ran prettier! Let me know if you want me to squash those into one commit or something, otherwise I think this stuff is good to go

@KyleAMathews
Copy link
Contributor

Awesome! Merging

@KyleAMathews KyleAMathews merged commit 97da0ce into gatsbyjs:master Jul 19, 2017
@DSchau DSchau deleted the tests branch July 19, 2017 18:29
@KyleAMathews
Copy link
Contributor

I always squash so no worries there :-)

@KyleAMathews
Copy link
Contributor

Deploy preview failed.

Built with commit 6d65616

https://app.netlify.com/sites/image-processing/deploys/596fa49a7960b12ad84b44b0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants