Merge pull request #5880 from hjmjohnson/convert-itkbresenhamlinetest-to-itkbresenhamlinegtest

ENH: Convert itkBresenhamLineTest to itkBresenhamLineGTest

Author: hjmjohnson

Modules/Core/Common/test/CMakeLists.txt

@@ -72,7 +72,6 @@ set(
   itkPointSetTest.cxx
   itkPointSetToImageFilterTest1.cxx
   itkPointSetToImageFilterTest2.cxx
-  itkBresenhamLineTest.cxx
   itkSparseFieldLayerTest.cxx
   itkDataObjectTest.cxx
   itkAtanRegularizedHeavisideStepFunctionTest1.cxx
@@ -769,12 +768,6 @@ itk_add_test(
     DATA{${ITK_DATA_ROOT}/Input/VascularTreePointSet.txt}
     ${ITK_TEST_OUTPUT_DIR}/itkPointSetToImageFilterTest2.mha
 )
-itk_add_test(
-  NAME itkBresenhamLineTest
-  COMMAND
-    ITKCommon1TestDriver
-    itkBresenhamLineTest
-)
 itk_add_test(
   NAME itkSparseFieldLayerTest
   COMMAND
@@ -1754,6 +1747,7 @@ set(
   itkStdStreamStateSaveGTest.cxx
   itkDecoratorGTest.cxx
   itkAbortProcessObjectGTest.cxx
+  itkBresenhamLineGTest.cxx
 )
 creategoogletestdriver(ITKCommon "${ITKCommon-Test_LIBRARIES}" "${ITKCommonGTests}")
 # If `-static` was passed to CMAKE_EXE_LINKER_FLAGS, compilation fails. No need to

Modules/Core/Common/test/itkBresenhamLineGTest.cxx

@@ -0,0 +1,66 @@
+/*=========================================================================
+ *
+ *  Copyright NumFOCUS
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *         https://www.apache.org/licenses/LICENSE-2.0.txt
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ *
+ *=========================================================================*/
+
+#include "itkBresenhamLine.h"
+#include "itkGTest.h"
+
+#include <iostream>
+
+TEST(BresenhamLine, BuildLineFromVector)
+{
+  // Test BuildLine(Vector, distance)
+  itk::Vector<float, 2> v;
+  v[0] = 1;
+  v[1] = 1;
+
+  itk::BresenhamLine<2>       line;
+  std::vector<itk::Offset<2>> offsets = line.BuildLine(v, 4);
+
+  EXPECT_EQ(offsets.size(), 4u);
+
+  for (int i = 0; i < 4; ++i)
+  {
+    const itk::Offset<2> expectedOffset{ { i, i } };
+    ITK_EXPECT_VECTOR_NEAR(offsets[i], expectedOffset, 0);
+  }
+}
+
+TEST(BresenhamLine, BuildLineFromIndices)
+{
+  // Test BuildLine(Index, Index)
+  itk::Index<2> p0;
+  p0[0] = 0;
+  p0[1] = 0;
+
+  itk::Index<2> p1;
+  p1[0] = 39;
+  p1[1] = 39;
+
+  itk::BresenhamLine<2>      line;
+  std::vector<itk::Index<2>> indices = line.BuildLine(p0, p1);
+
+  EXPECT_EQ(indices.size(), 40u);
+
+  for (int i = 0; i < 40; ++i)
+  {
+    const itk::Index<2> expectedIndex{ { i, i } };
+    ITK_EXPECT_VECTOR_NEAR(indices[i], expectedIndex, 0);
+  }
+
+  std::cout << "Test Passed !" << std::endl;
+}