auto merge of #2349 : zwarich/servo/reftest-print-diff, r=larsbergstrom

Having to match the numbered results in /tmp with the failing tests was bothering me, and I figured it would be better to print the path with the failure.

1 files changed, 3 insertions, 2 deletions

diff --git a/src/test/harness/reftest/reftest.rs b/src/test/harness/reftest/reftest.rs
index a6b0ed62298..22d7a90faee 100644
--- a/src/test/harness/reftest/reftest.rs
+++ b/src/test/harness/reftest/reftest.rs
@@ -162,7 +162,8 @@ fn check_reftest(reftest: Reftest) {
         }).collect();
 
     if pixels.iter().any(|&a| a < 255) {
-        let output = from_str::<Path>(format!("/tmp/servo-reftest-{:06u}-diff.png", reftest.id)).unwrap();
+        let output_str = format!("/tmp/servo-reftest-{:06u}-diff.png", reftest.id);
+        let output = from_str::<Path>(output_str).unwrap();
 
         let img = png::Image {
             width: left.width,
@@ -173,7 +174,7 @@ fn check_reftest(reftest: Reftest) {
         let res = png::store_png(&img, &output);
         assert!(res.is_ok());
 
-        assert!(reftest.kind == Different);
+        assert!(reftest.kind == Different, "rendering difference: {}", output_str);
     } else {
         assert!(reftest.kind == Same);
     }