GridBounds: coordsIter output should match coords

locationtech · Jul 21, 2017 · 3d987f2 · 3d987f2
1 parent d93600e
commit 3d987f2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/raster-test/src/test/scala/geotrellis/raster/GridBoundsSpec.scala b/raster-test/src/test/scala/geotrellis/raster/GridBoundsSpec.scala
@@ -100,4 +100,13 @@ class GridBoundsSpec extends FunSpec with Matchers{
       GridBounds.distinct(gridBounds).map(_.size).sum should be ((101 * 101) - (25 * 25 * 2))
     }
   }
+
+  describe("GridBounds.coords") {
+    // TODO This test can be removed in 2.0
+    it("should match the output of coordsIter") {
+      val gbs = GridBounds(0, 0, 10, 10)
+
+      gbs.coordsIter.toSeq shouldBe gbs.coords.toSeq
+    }
+  }
 }
diff --git a/raster/src/main/scala/geotrellis/raster/GridBounds.scala b/raster/src/main/scala/geotrellis/raster/GridBounds.scala
@@ -180,7 +180,7 @@ case class GridBounds(colMin: Int, rowMin: Int, colMax: Int, rowMax: Int) {
   def coordsIter: Iterator[(Int, Int)] = for {
     row <- Iterator.range(0, height)
     col <- Iterator.range(0, width)
-  } yield (row, col)
+  } yield (col, row)
 
   /**
     * Return the intersection of the present [[GridBounds]] and the