The Cockpit integration tests can now contain “pixel tests”. Such a test will take a screenshot with the browser and compare it with a reference. The idea is that we can catch visual regressions much easier this way than if we would hunt for them in a purely manual fashion.
Preparing a repository for pixel tests
A pixel test will take a screenshot of part of the Cockpit UI and compare it with a reference. Thus, these reference images are important and play the biggest role.
A large part of dealing with pixel tests will consequently consist of maintaining the reference images. At the same time, we don’t want to clog up our main source repository with them. While the number and size of the reference images at any one point in time should not pose a problem, we will over time accumulate a history of them that we are afraid would dominate the source repository.
Thus, the reference images are not stored in the source repository. Instead, we store them in an external repository that is linked into the source repository as a submodule. That external repository doesn’t keep any history and can be aggressively pruned.
Developers are mostly isolated from this via the new
test/common/pixel-tests tool. But if you are familiar with git
submodules, there should be no suprises for you here.
A source repository needs to be prepared before it can store reference images in a external storage repository. You can let the tool do it by running
$ ./test/common/pixel-tests init
and then committing the changes it has done to the source repository. (Those changes will be a new or modified .gitmodules file, and a new gitlink at test/reference.)
Adding a pixel test
To add a pixel test to a test program, call the new “assert_pixels” function of the Browser class:
def testSomeDialog(self): ... self.browser.assert_pixels('#dialog button.apply', "button") ...
The first argument is a CSS selector that identifies the part of the UI that you want to compare. The screenshot will only include that element. The second argument is an arbitrary but unique key for this test point. It is used to name the files that go with it.
For each such call, there needs to be a reference image in the
As mentioned above, the
test/reference/ directory is very special
indeed, and needs to be carefully managed with the
test/common/pixel-tests tool (or even more carefully with
submodule et al).
First, make sure
test/reference is up-to-date:
$ ./test/common/pixel-tests pull
Then you can add new reference images to it.
The easiest way to get a new reference image is to just run the test once (locally or in the CI machinery). It will fail, but produce a reference image for the current state of the UI.
When you run the test locally, the new reference image will appear in the current directory. Just move it into test/reference/. The next run of the test should then be green.
When you run the tests in the CI machinery, you need to download the new reference images from the test results directory. They show up as regular screenshots.
If there are parts of the reference image that you want to ignore, you can pass a suitable “ignore” argument to assert_pixels. It contains a list of CSS selectors, and any pixel that is within their bounding rectangles is ignored. For example, this
def testSomeDialog(self): ... self.browser.assert_pixels('#dialog', "dialog", ignore=["#memory-available"]) ...
would compare a full dialog to a reference, but would exclude the DOM element that shows a number that is dependent on the environment that the test runs in.
You can also change the transparency (alpha channel) of parts of the reference image itself (with the GIMP, say). Any pixel that is not fully opaque will be ignored during comparison.
When you are done adding images to
test/reference, push them into
the storage repository like so:
$ ./test/common/pixel-tests push
push command will record a change to
test/reference in the
main repository, and you need to commit this change. This is how the
main repository specifies which reference images to use exactly for
each of its commits.
If you want to see what changes would be pushed without actually pushing them, you can run this:
$ ./test/common/pixel-tests status
Here is a PR that adds two pixel test points to the starter-kit, complete with reference images:
Debugging a failed pixel test
When making changes that change how the UI looks, some pixel tests will fail. The test results will contain the new pixels, and you can compare them with the referene image right in the browser when looking at the test logs.
Here is a PR that makes the two pixel tests fail that had been added in #436:
As you can see, it has “changed pixels” links in the same place as the well known “screenshot” links. Clicking on it gets you to a page where you can directly compare the previous and current UI.
As the author of the pull request, you can decide from there whether these changes are intended or not.
For a intended change, see the next section. For unintended changes, you need to fix your code and try again, of course.
Updating a pixel test
If you make a change that intentionally changes how Cockpit looks, you
need to install new reference images in
This is very similar to adding a new pixel test point. Take the
TestFoo-testBasic-pixels.png that was written by the failed test run,
move it into
test/reference, push it to the storage repository with
./test/common/pixel-tests push, and commit
test/reference in the
A local test run has dropped the new reference image into the current directory, for a remote run it will be in the test results directory and the pixel comparison view has a link to it.
When a test writes a new TestFoo-testBasic-pixels.png file for a failed test, it will have the alpha channel of the old reference copied into it. That makes it easy to keep ignoring the same parts.
Reviewing a changed pixel test
Here is a second version of the starter-kit pull request from the previous section:
It has the same code changes, but now the reference images have been updated as well, since the change in color was of course intended.
Now this PR needs to be reviewed, and the changed visuals need to be approved. But since the reference images are not stored in the main repository, Github will not include them in the PR diff view.
Instead, the robots will automatically add a comment to a pull request with a link to a page that allows reviewing the changed reference images.