[cairo] [PATCH v2 0/7] Add check-ref-dups
Bryce W. Harrington
b.harrington at samsung.com
Wed Jul 3 15:18:16 PDT 2013
This patchset adds a script to locate redundant reference images that
can safely be dropped from the tree without affecting test results.
I've hooked this into 'make check-ref-dups' - which already existed in
the makefile but had bitrotted and no longer worked. The original
code was not descriminating enough; it included far too many false
positives to be of much use. However, the first change of this patchset
fixes the bitrot so the original code can be run. The subsequent
patches remove it in favor of the new checker.
The new checker only suggests removing files for which make check will
fall back to an equivalent more generic image. Currently this flags 1485
image files (approx. 30 mb in an unpacked tarball).
Running make check on the codebase (with default configuration) with the
redundant images removed produces essentially the same test results:
# Before
#######
# Tests run: 13687
# Passed: 9216
# Failed: 3566
# Expected Failed: 312
# Error: 1
# Crashed: 17
# Untested: 575
# Total: 13687
#######
# After
#######
# Tests run: 13689
# Passed: 9216
# Failed: 3566
# Expected Failed: 312
# Error: 1
# Crashed: 19
# Untested: 575
# Total: 13689
#######
The two additional crashes in the latter run are just due to irregular
crashes in the pthread-same-source test cases, which alternate between
crashing and failing randomly.
Bryce Harrington (7):
test: Fix make check-ref-dups due to move of ref images to reference/
test: Add script to check for redundant reference images
test: Use cmp to catch byte-by-byte identical files
test: Make check-ref-dups utilize perceptualdiff for comparisons
pdiff: Quell warning about signed/unsigned comparisons
pdiff: Drop unused variable
test: Add special cases for create-from-png and fallback-resolution
test/Makefile.am | 10 +++---
test/check_refs.sh | 74 +++++++++++++++++++++++++++++++++++++++++++
test/create-from-png.c | 5 +++
test/fallback-resolution.c | 1 +
test/pdiff/perceptualdiff.c | 6 ++--
5 files changed, 89 insertions(+), 7 deletions(-)
create mode 100755 test/check_refs.sh
--
1.7.9.5
--
Bryce Harrington
Senior Open Source Developer - b.harrington at samsung.com
Open Source Group - Samsung Research America
More information about the cairo
mailing list