jest-image-snapshot 2.12.0 → 4.0.0

package/.github/workflows/stale.yml

@@ -0,0 +1,22 @@
+name: Mark stale issues and pull requests
+
+on:
+  schedule:
+  - cron: "0 0 * * *"
+
+jobs:
+  stale:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/stale@v1
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+        stale-issue-message: 'This issue is stale because it has been open 30 days with no activity.'
+        stale-pr-message: 'This pull request is stale because it has been open 30 days with no activity.'
+        stale-issue-label: 'stale-issue'
+        exempt-issue-label: 'stale-issue'
+        exempt-pr-label: 'stale-pr'
+        stale-pr-label: 'stale-pr'
+        days-before-stale: 30

package/.travis.yml

@@ -1,14 +1,22 @@
 language: node_js
 node_js:
-  - "6"
-  - "8"
-  - "9"
-  - "10"
-deploy:
-  on:
-    tags: true
-    branch: master
-  provider: npm
-  email: open-source-ops@aexp.com
-  api_key:
-    secure: lhhShzouUcVTerUWMJDjS4ChuRCNXlCKo4wLJINNjcg5MP1d7XrlU6zbv7JPfQKU4pYoMcEKeldzWNGIEtKePStr4bWaiCFzgjiskdxt80mtMLC+t0bWWUZPQPGQeoEVQb7v3RLEXhFZHqqjiDFmp4TPXfcyCyzSIn6YpNEGFydtMaf8TN0haPAI4k43ZFqA2JrMD4/lARR7WDFkP9otywxOtQVQGnrLVSNMuNtJV6MvImDJoeh6AIeZl9kuRn5EZMQw7cDuLp+yjRs4O76Lz8EkfT8gq1flUYwLv0DgIuEwgbb4h8LFv1gaS7lWkoADRfTufeToau7KSMgk70h/IvOQ4CB+R5wLZdDoin1zLZx+fEPU7PQQSaXHk0nn6XeFQG+j2lJnde3YVs5AeaaAYUhvHyHs+btWoGLvXREeYA61bEn7p46GAh/JjZ3ydyqHBTHU2dh11QV6B3gjyFlfD1ul6NJvMnwd64vpeWWlG0gci0P1RAwgc2gGvNZ00/Qv6XoNiZpPkp7GQkpGocN+Vr4/zgPpaApk5HjSQL/4X7vh4M2eAhslT91vxcKzjyhAvwjAbFd5ZUd2lAjwIzEfzSzsXe7YS2KhsSf/+qln2Sx4nxCmraUtebAYxPpaeYWd3jbXzHBt8K51mzO8OuOKtAG1Ck0zB3W5XENhxJpfU0I=
+  - 10
+  - 12
+before_install:
+  # Create a master branch for commitlint
+  # https://github.com/conventional-changelog/commitlint/issues/6
+  - git remote set-branches origin master && git fetch
+jobs:
+  include:
+    # Define the release stage that runs semantic-release
+    - stage: release
+      node_js:
+        - 10
+        - 12
+      deploy:
+        on:
+          branch: master
+        provider: script
+        skip_cleanup: true
+        script:
+          - npx semantic-release

package/CHANGELOG.md

@@ -0,0 +1,52 @@
+# [4.0.0](https://github.com/americanexpress/jest-image-snapshot/compare/v3.1.0...v4.0.0) (2020-05-14)
+
+
+### chore
+
+* upgrade to jest 26 + drop node 8 support ([#205](https://github.com/americanexpress/jest-image-snapshot/issues/205)) ([4834533](https://github.com/americanexpress/jest-image-snapshot/commit/4834533369dae1533c93ad883e3f66617d7d9c3f))
+
+
+### BREAKING CHANGES
+
+* drop node 8 support
+
+# [3.1.0](https://github.com/americanexpress/jest-image-snapshot/compare/v3.0.1...v3.1.0) (2020-04-17)
+
+
+### Features
+
+* **options:** add option to pass on size missmatch ([#174](https://github.com/americanexpress/jest-image-snapshot/issues/174)) ([cee46b1](https://github.com/americanexpress/jest-image-snapshot/commit/cee46b1fc94f962c34900a8b655d22665cea2854)), closes [#83](https://github.com/americanexpress/jest-image-snapshot/issues/83) [#85](https://github.com/americanexpress/jest-image-snapshot/issues/85)
+
+## [3.0.1](https://github.com/americanexpress/jest-image-snapshot/compare/v3.0.0...v3.0.1) (2020-03-25)
+
+
+### Performance Improvements
+
+* **diff-snapshot:** remove logic to bypass diff for identical images ([1be1b00](https://github.com/americanexpress/jest-image-snapshot/commit/1be1b006220b4144f98ad583c8cd6ff629aec7b3))
+
+# [3.0.0](https://github.com/americanexpress/jest-image-snapshot/compare/v2.12.0...v3.0.0) (2020-03-24)
+
+
+### Bug Fixes
+
+* **diff:** small default maxBuffer ([df713f6](https://github.com/americanexpress/jest-image-snapshot/commit/df713f6afb7ec7130ec07e94d6a137a3ea62c5de))
+* **diff-snapshot:** dumpDiffToConsole base64 string output ([#183](https://github.com/americanexpress/jest-image-snapshot/issues/183)) ([f73079f](https://github.com/americanexpress/jest-image-snapshot/commit/f73079f42f86696831ebe85d718e27d6f1d048c0))
+
+
+### chore
+
+* **packages:** updating jest to 25.1 for perf improvements ([#170](https://github.com/americanexpress/jest-image-snapshot/issues/170)) ([eb3dfa6](https://github.com/americanexpress/jest-image-snapshot/commit/eb3dfa605c0344ac4dc42cb7f9f76a5e4a732592))
+* **packages:** upgrade from pixelmatch 4.x to 5.x, and pngjs to 3.4 ([#186](https://github.com/americanexpress/jest-image-snapshot/issues/186)) ([1edc9a3](https://github.com/americanexpress/jest-image-snapshot/commit/1edc9a31db2130b1eafb45738ebc81fa544d380f))
+* **travis:** remove node 6 from travis config ([ce2b757](https://github.com/americanexpress/jest-image-snapshot/commit/ce2b757a6f337ac156e901a0f1e2851f94d0e7b2))
+
+
+### Features
+
+* **diff:** increase the maxBuffer to 10MB for the diff process ([#167](https://github.com/americanexpress/jest-image-snapshot/issues/167)) ([0927826](https://github.com/americanexpress/jest-image-snapshot/commit/0927826776e5fee04ea98cba5cd792aa5066e1fd))
+
+
+### BREAKING CHANGES
+
+* **packages:** pixelmatch is being major version bumped and so image diffs may be difference
+* **packages:** Node min version is now 8
+* **travis:** drop support for node 6

package/CONTRIBUTING.md

@@ -1,145 +1,115 @@
-# Contributing
+# Contributing to jest-image-snapshot
 
-The following guidelines must be followed by all contributors to this repository. Please review them carefully and do not hesitate to ask for help.
+✨ Thank you for taking the time to contribute to this project ✨
 
-### Code of Conduct
+## 📖 Table of Contents
 
-* Review and test your code before submitting a pull request.
-* Be kind and professional. Avoid assumptions; oversights happen.
-* Be clear and concise when documenting code; focus on value.
-* Don't commit commented code to the main repo (stash locally, if needed).
+* [Code of Conduct](#code-of-conduct)
+* [Developing](#developing)
+* [Submitting a new feature](#submitting-a-new-feature)
+* [Reporting bugs](#reporting-bugs)
+* [Contributing](#getting-in-contact)
+* [Coding conventions](#coding-conventions)
 
-### Git Commit Guidelines
-
-We follow precise rules for git commit message formatting. These rules make it easier to review commit logs and improve contextual understanding of code changes. This also allows us to auto-generate the CHANGELOG from commit messages.
-
-Each commit message consists of a **header**, **body** and **footer**.
+## Code of Conduct
 
-#### Header
+This project adheres to the American Express [Code of Conduct](./CODE_OF_CONDUCT.md). By contributing, you are expected to honor these guidelines.
 
-The header is required and must not exceed 70 characters to ensure it is well-formatted in common git tools. It has a special format that includes a *type*, *scope* and *subject*:
+## Developing
 
-Syntax:
+### Installation
 
-```bash
-<type>(<scope>): <subject>
-```
+1. Fork the repository `jest-image-snapshot` to your GitHub account.
+2. Afterwards run the following commands in your terminal
 
-#### Type
+    ```bash
+    $ git clone https://github.com/<your-github-username>/jest-image-snapshot
+    $ cd jest-image-snapshot
+    ```
 
-The *type* should always be lowercase as shown below.
+   > replace `your-github-username` with your github username
 
-##### Allowed `<type>` values:
+3. Install the dependencies by running
 
-* **feat** (new feature for the user)
-* **fix** (bug fix for the user, not a fix to build scripts)
-* **docs** (changes to documentation)
-* **style** (formatting, missing semi colons, etc; no functional code change)
-* **refactor** (refactoring production code, eg. renaming a variable)
-* **test** (adding missing tests, refactoring tests; no production code change)
-* **chore** (updating build/env/packages, etc; no production code change)
+    ```bash
+    $ npm install
+    ```
 
-#### Scope
+4. You can now run any of these scripts from the root folder.
 
-The *scope* describes the affected code. The descriptor may be a route, component, feature, utility, etc. It should be one word or camelCased, if needed:
+#### Running tests
 
-```bash
-feat(transactions): added column for quantity
-feat(BalanceModule): initial setup
-```
+- **`npm run lint`**
 
-The commit headers above work well if the commit affects many parts of a larger feature. If changes are more specific, it may be too broad. To better clarify specific scopes, you should use a `feature/scope` syntax:
+Verifies that your code matches the American Express code style defined in [`eslint-config-amex`](https://github.com/americanexpress/eslint-config-amex).
 
-```bash
-fix(transaction/details): missing quantity field
-```
+- **`npm test`**
 
-The above syntax helps reduce verbosity in the _subject_. In comparison, consider the following example:
+Runs unit tests **and** verifies the format of all commit messages on the current branch.
 
-```bash
-fix(transaction): missing quantity field in txn details
-```
+- **`npm posttest`**
 
-Another scenario for scope is using a `route/scope` (or `context/scope`) syntax. This would be useful when a commit only affects a particular instance of code that is used in multiple places.
+Runs linting on the current branch, checks that the commits follow [conventional commits](https://www.conventionalcommits.org/) and verifies that the `package-lock.json` file includes public NPM registry URLs.
 
-*Example*: Transactions may be shown in multiple routes/contexts, but a bug affecting transaction actions may only exist under the "home" route, possibly related to other code. In such cases, you could use the following format:
+## Submitting a new feature
 
-```bash
-fix(home/transactions): txn actions not working
-```
+When submitting a new feature request or enhancement of an existing feature please review the following:-
 
-This header makes it clear that the fix is limited in scope to transactions within the home route/context.
+### Is your feature request related to a problem
 
-#### Subject
+Please provide a clear and concise description of what you want and what your use case is.
 
-Short summary of the commit. Avoid redundancy and simplify wording in ways that do not compromise understanding.
+### Provide an example
 
-Good:
+Please include a snippets of the code of the new feature.
 
-```bash
-$ git commit -m "fix(nav/link): incorrect URL for Travel"
-```
+### Describe the suggested enhancement
 
-Bad:
+A clear and concise description of the enhancement to be added include a step-by-step guide if applicable.
+Add any other context or screenshots or animated GIFs about the feature request
 
-```bash
-$ git commit -m "fix(nav): incorrect URL for Travel nav item :P"
-```
+### Describe alternatives you've considered
 
-> Note that the _Bad_ example results in a longer commit header. This is partly attributed to the scope not being more specific and personal expression tacked on the end.
+A clear and concise description of any alternative solutions or features you've considered.
 
-**Note regarding subjects for bug fixes:**
+## Reporting bugs
 
-Summarize _what is fixed_, rather than stating that it _is_ fixed. The _type_ ("fix") already specifies the state of the issue.
+All issues are submitted within GitHub issues. Please check this before submitting a new issue.
 
-For example, don't do:
+### Describe the bug
 
-```bash
-$ git commit -m "fix(nav): corrected Travel URL"
-```
+A clear and concise description of what the bug is.
 
-Instead, do:
+### Provide step-by-step guide on how to reproduce the bug
 
-```bash
-$ git commit -m "fix(nav): broken URL for Travel"
-```
+Steps to reproduce the behavior, please provide code snippets or a link to repository
 
+### Expected behavior
 
-#### Body and Footer (optional)
+Please provide a description of the expected behavior
 
-The body and footer should wrap at 80 characters.
+### Screenshots
 
-The **body** describes the commit in more detail and should not be more than 1 paragraph (3-5 sentences). Details are important, but too much verbosity can inhibit understanding and productivity -- keep it clear and concise.
+If applicable, add screenshots or animated GIFs to help explain your problem.
 
-The **footer** should only reference Pull Requests or Issues associated with the commit.
+### System information
 
-For bug fixes that address open issues, the footer should be formatted like so:
+Provide the system information which is not limited to the below:
 
-```bash
-Closes #17, #26
-```
-and for Pull Requests, use the format:
+- Browser (if applies) [e.g. chrome, safari]
+- Version of jest-image-snapshot: [e.g. 5.0.0]
+- Node version:[e.g 10.15.1]
 
-```bash
-Related #37
-```
+### Security Bugs
 
-If a commit is associated with issues and pull requests, use the following format:
+Please review our [Security Policy](./SECURITY.md). Please follow the instructions outlined in the policy.
 
-```bash
-Closes #17, #26
-Related #37
-```
-> Issues should always be referenced before pull requests, as shown above.
+## Getting in contact
 
-#### Piecing It All Together
+- Join our [Slack channel](https://one-amex.slack.com) request an invite [here](https://join.slack.com/t/one-amex/shared_invite/enQtOTA0MzEzODExODEwLTlmYzI1Y2U2ZDEwNWJjOTAxYTlmZTYzMjUyNzQyZTdmMWIwZGJmZDM2MDZmYzVjMDk5OWU4OGIwNjJjZWRhMjY)
 
-Below is an example of a full commit message that includes a header, body and footer:
+## Coding conventions
 
-```bash
-refactor(nav/item): added prop (isActive)
-
-NavItem now supports an "isActive" property. This property is used to control the styling of active navigation links.
+### Git Commit Guidelines
 
-Closes #21
-```
+We follow [conventional commits](https://www.conventionalcommits.org/) for git commit message formatting. These rules make it easier to review commit logs and improve contextual understanding of code changes. This also allows us to auto-generate the CHANGELOG from commit messages.

package/README.md

@@ -18,12 +18,11 @@ Want to get paid for your contributions to `jest-image-snapshot`?
 * [Features](#-features)
 * [Usage](#-usage)
 * [API](#-api)
-* [Available Scripts](#-available-scripts)
 * [Contributing](#-contributing)
 
 ## ✨ Features
 
-* Take image snapshots of your React components
+* Take image snapshots of your application
 * Ability to compare snapshots from a baseline
 * Update snapshots when you're good with changes
 * Customize a difference threshold
@@ -76,7 +75,7 @@ Thanks `jest-image-snapshot`, that broken header would not have looked good in p
 npm i --save-dev jest-image-snapshot
 ```
 
-Please note that `Jest` `>=20 <=24` is a peerDependency. `jest-image-snapshot` will **not** work with anything below Jest 20.x.x
+Please note that `Jest` `>=20 <=25` is a peerDependency. `jest-image-snapshot` will **not** work with anything below Jest 20.x.x
 
 ### Invocation
 
@@ -112,9 +111,10 @@ See [the examples](./examples/README.md) for more detailed usage or read about a
 * `failureThreshold`: (default `0`) Sets the threshold that would trigger a test failure based on the `failureThresholdType` selected. This is different to the `customDiffConfig.threshold` above, that is the per pixel failure threshold, this is the failure threshold for the entire comparison.
 * `failureThresholdType`: (default `pixel`) (options `percent` or `pixel`) Sets the type of threshold that would trigger a failure.
 * `updatePassedSnapshot`: (default `false`) Updates a snapshot even if it passed the threshold against the existing one.
-* `blur`: (default `0`) Applies Gaussian Blur on compared images, accepts radius in pixels as value. Useful when you have noise after scaling images per different resolutions on your target website, usually setting it's value to 1-2 should be enough to solve that problem.
+* `blur`: (default `0`) Applies Gaussian Blur on compared images, accepts radius in pixels as value. Useful when you have noise after scaling images per different resolutions on your target website, usually setting its value to 1-2 should be enough to solve that problem.
 * `runInProcess`: (default `false`) Runs the diff in process without spawning a child process.
 * `dumpDiffToConsole`: (default `false`) Will output base64 string of a diff image to console in case of failed tests (in addition to creating a diff image). This string can be copy-pasted to a browser address string to preview the diff for a failed test.
+* `allowSizeMismatch`: (default `false`) If set to true, the build will not fail when the screenshots to compare have different sizes.
 
 ```javascript
 it('should demonstrate this matcher`s usage with a custom pixelmatch config', () => {
@@ -209,20 +209,6 @@ it('renders correctly', async () => {
 ...
 ```
 
-## 📜 Available Scripts
-
-**`npm run lint`**
-
-Verifies that your code matches the American Express code style defined in [`eslint-config-amex`](https://github.com/americanexpress/eslint-config-amex).
-
-**`npm test`**
-
-Runs unit tests **and** verifies the format of all commit messages on the current branch.
-
-**`npm posttest`**
-
-Runs linting on the current branch.
-
 ## 🏆 Contributing
 
 We welcome Your interest in the American Express Open Source Community on Github.
@@ -233,7 +219,7 @@ and to recipients of software distributed by American Express, You reserve all
 right, title, and interest, if any, in and to Your Contributions. Please [fill
 out the Agreement](https://cla-assistant.io/americanexpress/jest-image-snapshot).
 
-Please feel free to open pull requests and see [CONTRIBUTING.md](./CONTRIBUTING.md) for commit formatting details.
+Please feel free to open pull requests and see [CONTRIBUTING.md](./CONTRIBUTING.md) to learn how to get started contributing.
 
 ## 🗝️ License
 

package/commitlint.config.js

@@ -0,0 +1,20 @@
+/*
+ * Copyright 2020 American Express Travel Related Services Company, Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except
+ * in compliance with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software distributed under the License
+ * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,either express
+ * or implied. See the License for the specific language governing permissions and limitations
+ * under the License.
+ */
+
+module.exports = {
+  extends: ['@commitlint/config-conventional'],
+  rules: {
+    'scope-case': [2, 'always', ['pascal-case', 'camel-case', 'kebab-case']],
+  },
+};

package/package.json

@@ -1,15 +1,17 @@
 {
   "name": "jest-image-snapshot",
-  "version": "2.12.0",
+  "version": "4.0.0",
   "description": "Jest matcher for image comparisons. Most commonly used for visual regression testing.",
   "main": "src/index.js",
   "engines": {
-    "node": ">=6.0.0"
+    "node": ">= 10.14.2"
   },
   "scripts": {
     "lint": "eslint ./ --ignore-path .gitignore --ext .js",
     "test": "jest --ci=false",
-    "posttest": "npm run lint"
+    "test:lockfile": "lockfile-lint -p package-lock.json -t npm -a npm -o https: -c -i",
+    "test:git-history": "commitlint --from origin/master --to HEAD",
+    "posttest": "npm run lint && npm run test:git-history && npm run test:lockfile"
   },
   "keywords": [
     "test",
@@ -42,13 +44,20 @@
   "author": "Andres Escobar <andres.escobar@aexp.com> (https://github.com/anescobar1991)",
   "license": "Apache-2.0",
   "devDependencies": {
-    "amex-jest-preset": "^5.0.0",
-    "eslint": "^4.15.0",
+    "@commitlint/cli": "^8.3.5",
+    "@commitlint/config-conventional": "^8.3.4",
+    "@semantic-release/changelog": "^5.0.0",
+    "@semantic-release/git": "^9.0.0",
+    "amex-jest-preset": "^6.0.1",
+    "eslint": "^6.8.0",
     "eslint-config-amex": "^7.0.0",
-    "image-size": "^0.7.1",
-    "jest": "^24.0.0",
-    "mock-spawn": "^0.2.6",
-    "jest-snapshot": "^23.0.0"
+    "husky": "^4.2.1",
+    "image-size": "^0.8.3",
+    "jest": "^26.0.1",
+    "jest-snapshot": "^26.0.1",
+    "semantic-release": "^17.0.4",
+    "lockfile-lint": "^4.0.0",
+    "mock-spawn": "^0.2.6"
   },
   "dependencies": {
     "chalk": "^1.1.3",
@@ -56,11 +65,28 @@
     "glur": "^1.1.2",
     "lodash": "^4.17.4",
     "mkdirp": "^0.5.1",
-    "pixelmatch": "^4.0.2",
-    "pngjs": "^3.3.3",
+    "pixelmatch": "^5.1.0",
+    "pngjs": "^3.4.0",
     "rimraf": "^2.6.2"
   },
   "peerDependencies": {
-    "jest": ">=20 <=24"
+    "jest": ">=20 <=26"
+  },
+  "husky": {
+    "hooks": {
+      "pre-commit": "npm test",
+      "commit-msg": "commitlint -E HUSKY_GIT_PARAMS"
+    }
+  },
+  "release": {
+    "plugins": [
+      "@semantic-release/commit-analyzer",
+      "@semantic-release/release-notes-generator",
+      "@semantic-release/changelog",
+      "@semantic-release/npm",
+      "@semantic-release/git",
+      "@semantic-release/github"
+    ],
+    "branch": "master"
   }
 }

package/src/diff-snapshot.js

@@ -19,7 +19,6 @@ const mkdirp = require('mkdirp');
 const pixelmatch = require('pixelmatch');
 const { PNG } = require('pngjs');
 const rimraf = require('rimraf');
-const { createHash } = require('crypto');
 const glur = require('glur');
 const ImageComposer = require('./image-composer');
 
@@ -84,8 +83,39 @@ const shouldUpdate = ({ pass, updateSnapshot, updatePassedSnapshot }) => (
   (!pass && updateSnapshot) || (pass && updatePassedSnapshot)
 );
 
+const shouldFail = ({
+  totalPixels,
+  diffPixelCount,
+  hasSizeMismatch,
+  allowSizeMismatch,
+  failureThresholdType,
+  failureThreshold,
+}) => {
+  let pass = false;
+  let diffSize = false;
+  const diffRatio = diffPixelCount / totalPixels;
+  if (hasSizeMismatch) {
+    // do not fail if allowSizeMismatch is set
+    pass = allowSizeMismatch;
+    diffSize = true;
+  }
+  if (!diffSize || pass === true) {
+    if (failureThresholdType === 'pixel') {
+      pass = diffPixelCount <= failureThreshold;
+    } else if (failureThresholdType === 'percent') {
+      pass = diffRatio <= failureThreshold;
+    } else {
+      throw new Error(`Unknown failureThresholdType: ${failureThresholdType}. Valid options are "pixel" or "percent".`);
+    }
+  }
+  return {
+    pass,
+    diffSize,
+    diffRatio,
+  };
+};
+
 function diffImageToSnapshot(options) {
-  /* eslint complexity: ["error", 12] */
   const {
     receivedImageBuffer,
     snapshotIdentifier,
@@ -98,6 +128,7 @@ function diffImageToSnapshot(options) {
     failureThreshold,
     failureThresholdType,
     blur,
+    allowSizeMismatch = false,
   } = options;
 
   let result = {};
@@ -142,40 +173,31 @@ function diffImageToSnapshot(options) {
 
     const diffImage = new PNG({ width: imageWidth, height: imageHeight });
 
-    let pass = false;
-    let diffSize = false;
-    let diffRatio = 0;
     let diffPixelCount = 0;
 
-    const receivedImageDigest = createHash('sha1').update(receivedImage.data).digest('base64');
-    const baselineImageDigest = createHash('sha1').update(baselineImage.data).digest('base64');
-
-    pass = receivedImageDigest === baselineImageDigest;
+    diffPixelCount = pixelmatch(
+      receivedImage.data,
+      baselineImage.data,
+      diffImage.data,
+      imageWidth,
+      imageHeight,
+      diffConfig
+    );
 
-    if (!pass) {
-      diffPixelCount = pixelmatch(
-        receivedImage.data,
-        baselineImage.data,
-        diffImage.data,
-        imageWidth,
-        imageHeight,
-        diffConfig
-      );
+    const totalPixels = imageWidth * imageHeight;
 
-      const totalPixels = imageWidth * imageHeight;
-      diffRatio = diffPixelCount / totalPixels;
-      // Always fail test on image size mismatch
-      if (hasSizeMismatch) {
-        pass = false;
-        diffSize = true;
-      } else if (failureThresholdType === 'pixel') {
-        pass = diffPixelCount <= failureThreshold;
-      } else if (failureThresholdType === 'percent') {
-        pass = diffRatio <= failureThreshold;
-      } else {
-        throw new Error(`Unknown failureThresholdType: ${failureThresholdType}. Valid options are "pixel" or "percent".`);
-      }
-    }
+    const {
+      pass,
+      diffSize,
+      diffRatio,
+    } = shouldFail({
+      totalPixels,
+      diffPixelCount,
+      hasSizeMismatch,
+      allowSizeMismatch,
+      failureThresholdType,
+      failureThreshold,
+    });
 
     if (isFailure({ pass, updateSnapshot })) {
       mkdirp.sync(diffDir);
@@ -213,7 +235,7 @@ function diffImageToSnapshot(options) {
         diffOutputPath,
         diffRatio,
         diffPixelCount,
-        imgSrcString: `data:image/png;base64,${pngBuffer}`,
+        imgSrcString: `data:image/png;base64,${pngBuffer.toString('base64')}`,
       };
     } else if (shouldUpdate({ pass, updateSnapshot, updatePassedSnapshot })) {
       mkdirp.sync(snapshotsDir);
@@ -222,6 +244,7 @@ function diffImageToSnapshot(options) {
     } else {
       result = {
         pass,
+        diffSize,
         diffRatio,
         diffPixelCount,
         diffOutputPath,
@@ -240,7 +263,11 @@ function runDiffImageToSnapshot(options) {
 
   const writeDiffProcess = childProcess.spawnSync(
     process.execPath, [`${__dirname}/diff-process.js`],
-    { input: Buffer.from(serializedInput), stdio: ['pipe', 'inherit', 'inherit', 'pipe'] }
+    {
+      input: Buffer.from(serializedInput),
+      stdio: ['pipe', 'inherit', 'inherit', 'pipe'],
+      maxBuffer: 10 * 1024 * 1024, // 10 MB
+    }
   );
 
   if (writeDiffProcess.status === 0) {