rackerlabs / snappit-visual-regression

An NPM module for visual regression testing

snappit-visual-regression

An NPM module for visual regression testing.

snappit-visual-regression's Issues

Add run scripts

SVR currently does not expose scripts for updating its selenium, it's browser drivers, or sauce-connect. It also does not provide a way to easily start selenium or the sauce-connect binary.

This issue requests that SVR expose commands for updating and running it's delegate binaries.

Version 10 of node.js has been released

Version 10 of Node.js (code name Dubnium) has been released! 🎊

To see what happens to your code in Node.js 10, Greenkeeper has created a branch with the following changes:

• Added the new Node.js version to your .travis.yml
• The new Node.js version is in-range for the engines in 1 of your package.json files, so that was left alone

If you’re interested in upgrading this repo to Node.js 10, you can open a PR with these changes. Please note that this issue is just intended as a friendly reminder and the PR as a possible starting point for getting your code running on Node.js 10.

More information on this issue

Greenkeeper has checked the engines key in any package.json file, the .nvmrc file, and the .travis.yml file, if present.

• engines was only updated if it defined a single version, not a range.
• .nvmrc was updated to Node.js 10
• .travis.yml was only changed if there was a root-level node_js that didn’t already include Node.js 10, such as node or lts/*. In this case, the new version was appended to the list. We didn’t touch job or matrix configurations because these tend to be quite specific and complex, and it’s difficult to infer what the intentions were.

For many simpler .travis.yml configurations, this PR should suffice as-is, but depending on what you’re doing it may require additional work or may not be applicable at all. We’re also aware that you may have good reasons to not update to Node.js 10, which is why this was sent as an issue and not a pull request. Feel free to delete it without comment, I’m a humble robot and won’t feel rejected 🤖

---

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.

---

Your Greenkeeper Bot 🌴

Seeing bitblt errors with some element screenshots.

There appears to be a bug with some screenshots that try to calculate element offsets.

Element Screenshots Incompatible with Web Components.

WebElement.takeScreenshot(); is not properly implemented in the current versions of Geckodriver/Chromedriver that we use. As a result, we're using a workaround that involves creating a flattened/frozen copy of the element in a canvas and screenshotting that instead.

The copy process doesn't handle the shadow dom properly, resulting in an incorrect screenshot.

Handling "full page" screenshots versus "full viewport" screenshots

Brought up here: #25 (comment)

What do we do when no element is passed in? Currently, the browser default is used, which varies. Should this be consistent, allowing for any browser to take a screenshot versus the visible viewport as a "full page" screenshot?

An in-range update of pngjs is breaking the build 🚨

The dependency pngjs was updated from 3.3.3 to 3.4.0.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

pngjs is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details

• ❌ continuous-integration/travis-ci/push: The Travis CI build failed (Details).

---

Commits

The new version differs by 8 commits.

• 9e499c4 v3.4.0
• 7e16096 Update eslint to the latest version 🚀 (#86)
• d6bf8d1 Update to node 10 in .travis.yml (#124)
• 0b5822a Emit an error if the image is truncated (#118)
• 728cc7e Added a browserified version (#98)
• 7acfa68 chore(package): update tap-dot to version 2.0.0 (#113)
• decc76f Return a correct alpha value in the metadata callback when a tRNS chunk is present (#119)
• 2d78949 Speed up mapImage8Bit and mapImageCustomBit (#123)

See the full diff

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.

---

Your Greenkeeper Bot 🌴

Dependency for selenium webdriver types

We are exporting selenium webdriver as a courtesy, but we need to also provide the types as an install dependency, not a dev-dependency.

Element screenshots do not support absolute positioned elements

Absolute positioned elements should not have their parent scrollable div at the viewport. Instead they should have the next scrollable positioned element.

For now, trivial support could be accomplished just by returning the viewport as docElement, since this will be the case for most absolute positioned elements.

Headless Chrome Viewport Cannot be Resized.

Chrome relies on an extension to resize the window/viewport, and the extension is currently not compatible with headless. More details: https://bugs.chromium.org/p/chromedriver/issues/detail?id=1625

An in-range update of @types/chai is breaking the build 🚨

The devDependency @types/chai was updated from 4.1.7 to 4.2.0.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

@types/chai is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Status Details

• ❌ continuous-integration/travis-ci/push: The Travis CI build failed (Details).

---

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.

---

Your Greenkeeper Bot 🌴

Allow Hiding of Elements While Taking Screenshots

It would be useful to be able to exclude certain elements when taking screenshots. Perhaps by providing a list of elements whose opacity would temporarily be set to zero.

Cannot Specify Initial Viewport Size.

Because of #22 , it would be useful to have a way to set the initial Viewport size in Chrome Headless.

Action required: Greenkeeper could not be activated 🚨

🚨 You need to enable Continuous Integration on all branches of this repository. 🚨

To enable Greenkeeper, you need to make sure that a commit status is reported on all branches. This is required by Greenkeeper because we are using your CI build statuses to figure out when to notify you about breaking changes.

Since we did not receive a CI status on the greenkeeper/initial branch, we assume that you still need to configure it.

If you have already set up a CI for this repository, you might need to check your configuration. Make sure it will run on all new branches. If you don’t want it to run on every branch, you can whitelist branches starting with greenkeeper/.

We recommend using Travis CI, but Greenkeeper will work with every other CI service as well.

Once you have installed CI on this repository, you’ll need to re-trigger Greenkeeper’s initial Pull Request. To do this, please delete the greenkeeper/initial branch in this repository, and then remove and re-add this repository to the Greenkeeper integration’s white list on Github. You'll find this list on your repo or organiszation’s settings page, under Installed GitHub Apps.

Passing in scrollable elements to snap calls

Brought up here: #25 (comment)

I should be able to pass in an element that is scrollable, and snap should know that I want to take a picture of the entire content within the element.

An in-range update of @types/selenium-webdriver is breaking the build 🚨

The dependency @types/selenium-webdriver was updated from 3.0.12 to 3.0.13.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

@types/selenium-webdriver is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details

• ❌ continuous-integration/travis-ci/push: The Travis CI build failed (Details).

---

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.

---

Your Greenkeeper Bot 🌴

An in-range update of inversify is breaking the build 🚨

The devDependency inversify was updated from 4.13.0 to 4.14.0.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

inversify is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Status Details

• ❌ continuous-integration/travis-ci/push: The Travis CI build failed (Details).

---

Commits

The new version differs by 123 commits.

• 5ba6374 added auditignore as workaround to upgrading to gulp4.0
• ef20ace update package.json
• 8b10354 Merge pull request #966 from inversify/greenkeeper/publish-please-4.0.1
• dc04b34 chore(package): update publish-please to version 4.0.1
• 2516da7 Merge pull request #960 from inversify/greenkeeper/@types/sinon-5.0.3
• 5479eee chore(package): update @types/sinon to version 5.0.3
• b41f5bf Merge pull request #959 from inversify/greenkeeper/publish-please-4.0.0
• a2decd6 Merge branch 'master' into greenkeeper/publish-please-4.0.0
• 52fd454 Update README.md
• 3317f37 chore(package): update publish-please to version 4.0.0
• b0a31fb Merge pull request #957 from inversify/greenkeeper/browserify-16.2.3
• f30a4d0 chore(package): update browserify to version 16.2.3
• ab6a575 Merge pull request #946 from inversify/greenkeeper/bluebird-3.5.2
• df72d9e chore(package): update bluebird to version 3.5.2
• 34bc411 Merge pull request #945 from inversify/greenkeeper/@types/sinon-5.0.2

There are 123 commits in total.

See the full diff

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.

---

Your Greenkeeper Bot 🌴