interesting
/
go-opengl-pixel
mirror of https://github.com/faiface/pixel.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

#159 Name matrices in unit test to avoid confusion with test cases

This commit is contained in:
Jacek Olszak 2019-02-13 15:06:50 +01:00
parent 392fe90b11
commit 0a054f35fd
1 changed files with 16 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
geometry_test.go
View File

@ -121,16 +121,19 @@ func TestMatrix_Unproject(t *testing.T) {
assert.InDelta(t, unprojected.Y, 0, delta) assert.InDelta(t, unprojected.Y, 0, delta)
}) })
t.Run("for projected vertices using all kinds of matrices", func(t *testing.T) { t.Run("for projected vertices using all kinds of matrices", func(t *testing.T) {
matrices := [...]pixel.Matrix{ namedMatrices := []struct {
pixel.IM, name string
pixel.IM.Scaled(pixel.ZV, 0.5), matrix pixel.Matrix
pixel.IM.Scaled(pixel.ZV, 2), }{
pixel.IM.Rotated(pixel.ZV, math.Pi/4), {"IM", pixel.IM},
pixel.IM.Moved(pixel.V(0.5, 1)), {"Scaled", pixel.IM.Scaled(pixel.ZV, 0.5)},
pixel.IM.Moved(pixel.V(-1, -0.5)), {"Scaled x 2", pixel.IM.Scaled(pixel.ZV, 2)},
pixel.IM.Scaled(pixel.ZV, 0.5).Rotated(pixel.ZV, math.Pi/4), {"Rotated", pixel.IM.Rotated(pixel.ZV, math.Pi/4)},
pixel.IM.Scaled(pixel.ZV, 0.5).Rotated(pixel.ZV, math.Pi/4).Moved(pixel.V(1, 2)), {"Moved", pixel.IM.Moved(pixel.V(0.5, 1))},
pixel.IM.Rotated(pixel.ZV, math.Pi/4).Moved(pixel.V(1, 2)), {"Moved 2", pixel.IM.Moved(pixel.V(-1, -0.5))},
{"Scaled and Rotated", pixel.IM.Scaled(pixel.ZV, 0.5).Rotated(pixel.ZV, math.Pi/4)},
{"Scaled, Rotated and Moved", pixel.IM.Scaled(pixel.ZV, 0.5).Rotated(pixel.ZV, math.Pi/4).Moved(pixel.V(1, 2))},
{"Rotated and Moved", pixel.IM.Rotated(pixel.ZV, math.Pi/4).Moved(pixel.V(1, 2))},
} }
vertices := [...]pixel.Vec{ vertices := [...]pixel.Vec{
pixel.V(0, 0), pixel.V(0, 0),
@ -143,10 +146,11 @@ func TestMatrix_Unproject(t *testing.T) {
pixel.V(0, -10), pixel.V(0, -10),
pixel.V(5, -10), pixel.V(5, -10),
} }
for _, matrix := range matrices { for _, namedMatrix := range namedMatrices {
for _, vertex := range vertices { for _, vertex := range vertices {
testCase := fmt.Sprintf("for matrix %v and vertex %v", matrix, vertex) testCase := fmt.Sprintf("for matrix %s and vertex %v", namedMatrix.name, vertex)
t.Run(testCase, func(t *testing.T) { t.Run(testCase, func(t *testing.T) {
matrix := namedMatrix.matrix
projected := matrix.Project(vertex) projected := matrix.Project(vertex)
unprojected := matrix.Unproject(projected) unprojected := matrix.Unproject(projected)
assert.InDelta(t, vertex.X, unprojected.X, delta) assert.InDelta(t, vertex.X, unprojected.X, delta)