From b2f127e33f92d778c9a4700ea5c97ba6fb20fe4a Mon Sep 17 00:00:00 2001
From: Nick White <git@njw.name>
Date: Mon, 13 May 2019 17:17:55 +0100
Subject: Add some basic tests for cleanup
---
cleanup/main.go | 1 -
cleanup/main_test.go | 105 +++++++++++++++++++++
cleanup/testdata/pg1.png | Bin 0 -> 30803 bytes
.../testdata/pg1_integralwipesides_t0.02_w5.png | Bin 0 -> 33595 bytes
.../testdata/pg1_integralwipesides_t0.05_w25.png | Bin 0 -> 33432 bytes
.../testdata/pg1_integralwipesides_t0.05_w5.png | Bin 0 -> 14546 bytes
6 files changed, 105 insertions(+), 1 deletion(-)
create mode 100644 cleanup/main_test.go
create mode 100644 cleanup/testdata/pg1.png
create mode 100644 cleanup/testdata/pg1_integralwipesides_t0.02_w5.png
create mode 100644 cleanup/testdata/pg1_integralwipesides_t0.05_w25.png
create mode 100644 cleanup/testdata/pg1_integralwipesides_t0.05_w5.png
diff --git a/cleanup/main.go b/cleanup/main.go
index e0b66ce..fdec6e9 100644
--- a/cleanup/main.go
+++ b/cleanup/main.go
@@ -1,7 +1,6 @@
package main
// TODO: add minimum size variable (default ~30%?)
-// TODO: add tests
// TODO: make into a small library
// TODO: have the integral image specific stuff done by interface functions
diff --git a/cleanup/main_test.go b/cleanup/main_test.go
new file mode 100644
index 0000000..3c3ce0c
--- /dev/null
+++ b/cleanup/main_test.go
@@ -0,0 +1,105 @@
+package main
+
+// TODO: add different pages as test cases
+// TODO: test non integral img version
+
+import (
+ "flag"
+ "fmt"
+ "image"
+ "image/draw"
+ "image/png"
+ "os"
+ "testing"
+
+ "rescribe.xyz/go.git/binarize"
+)
+
+var update = flag.Bool("update", false, "update golden files")
+
+func decode(s string) (*image.Gray, error) {
+ f, err := os.Open(s)
+ defer f.Close()
+ if err != nil {
+ return nil, err
+ }
+ img, err := png.Decode(f)
+ if err != nil {
+ return nil, err
+ }
+ b := img.Bounds()
+ gray := image.NewGray(image.Rect(0, 0, b.Dx(), b.Dy()))
+ draw.Draw(gray, b, img, b.Min, draw.Src)
+ return gray, nil
+}
+
+func imgsequal(img1 *image.Gray, img2 *image.Gray) bool {
+ b := img1.Bounds()
+ if !b.Eq(img2.Bounds()) {
+ return false
+ }
+ for y := b.Min.Y; y < b.Max.Y; y++ {
+ for x := b.Min.X; x < b.Max.X; x++ {
+ r0, g0, b0, a0 := img1.At(x, y).RGBA()
+ r1, g1, b1, a1 := img2.At(x, y).RGBA()
+ if r0 != r1 {
+ return false
+ }
+ if g0 != g1 {
+ return false
+ }
+ if b0 != b1 {
+ return false
+ }
+ if a0 != a1 {
+ return false
+ }
+ }
+ }
+ return true
+}
+
+func TestWipeSides(t *testing.T) {
+ cases := []struct {
+ name string
+ orig string
+ golden string
+ thresh float64
+ wsize int
+ }{
+ {"integralwipesides", "testdata/pg1.png", "testdata/pg1_integralwipesides_t0.02_w5.png", 0.02, 5},
+ {"integralwipesides", "testdata/pg1.png", "testdata/pg1_integralwipesides_t0.05_w5.png", 0.05, 5},
+ {"integralwipesides", "testdata/pg1.png", "testdata/pg1_integralwipesides_t0.05_w25.png", 0.05, 25},
+ }
+
+ for _, c := range cases {
+ t.Run(fmt.Sprintf("%s_%0.2f_%d", c.name, c.thresh, c.wsize), func(t *testing.T) {
+ var actual *image.Gray
+ orig, err := decode(c.orig)
+ if err != nil {
+ t.Fatalf("Could not open file %s: %v\n", c.orig, err)
+ }
+ integral := binarize.Integralimg(orig)
+ lowedge, highedge := findedges(integral, c.wsize, c.thresh)
+ actual = wipesides(orig, lowedge, highedge)
+ if *update {
+ f, err := os.Create(c.golden)
+ defer f.Close()
+ if err != nil {
+ t.Fatalf("Could not open file %s to update: %v\n", c.golden, err)
+ }
+ err = png.Encode(f, actual)
+ if err != nil {
+ t.Fatalf("Could not encode update of %s: %v\n", c.golden, err)
+ }
+ }
+ golden, err := decode(c.golden)
+ if err != nil {
+ t.Fatalf("Could not open file %s: %v\n", c.golden, err)
+ }
+ if !imgsequal(golden, actual) {
+ t.Errorf("Processed %s differs to %s\n", c.orig, c.golden)
+ }
+ })
+ }
+}
diff --git a/cleanup/testdata/pg1.png b/cleanup/testdata/pg1.png
new file mode 100644
index 0000000..c7c4249
Binary files /dev/null and b/cleanup/testdata/pg1.png differ
diff --git a/cleanup/testdata/pg1_integralwipesides_t0.02_w5.png b/cleanup/testdata/pg1_integralwipesides_t0.02_w5.png
new file mode 100644
index 0000000..6b4ccb2
Binary files /dev/null and b/cleanup/testdata/pg1_integralwipesides_t0.02_w5.png differ
diff --git a/cleanup/testdata/pg1_integralwipesides_t0.05_w25.png b/cleanup/testdata/pg1_integralwipesides_t0.05_w25.png
new file mode 100644
index 0000000..39dc88d
Binary files /dev/null and b/cleanup/testdata/pg1_integralwipesides_t0.05_w25.png differ
diff --git a/cleanup/testdata/pg1_integralwipesides_t0.05_w5.png b/cleanup/testdata/pg1_integralwipesides_t0.05_w5.png
new file mode 100644
index 0000000..50df855
Binary files /dev/null and b/cleanup/testdata/pg1_integralwipesides_t0.05_w5.png differ
--
cgit v1.2.1-24-ge1ad