Fix testing CMake issue to resolve Rolling regression (#961)

* Fix unit test CMake bug and small typos

* Fix bug when building with ros when testing is off

* Revert commit and eliminate BUILD_TESTING check

Author: sarcasticnature

CMakeLists.txt

@@ -79,21 +79,21 @@ if ( ament_cmake_FOUND )
 
     add_definitions( -DUSING_ROS2 )
     message(STATUS "------------------------------------------")
-    message(STATUS "BehaviourTree is being built using AMENT.")
+    message(STATUS "BehaviorTree is being built using AMENT.")
     message(STATUS "------------------------------------------")
     include(cmake/ament_build.cmake)
 
 elseif( CATKIN_DEVEL_PREFIX OR CATKIN_BUILD_BINARY_PACKAGE)
 
     add_definitions( -DUSING_ROS )
     message(STATUS "------------------------------------------")
-    message(STATUS "BehaviourTree is being built using CATKIN.")
+    message(STATUS "BehaviorTree is being built using CATKIN.")
     message(STATUS "------------------------------------------")
     include(cmake/catkin_build.cmake)
     set(catkin_FOUND TRUE)
 else()
     message(STATUS "------------------------------------------")
-    message(STATUS "BehaviourTree is being built with conan.")
+    message(STATUS "BehaviorTree is being built with conan.")
     message(STATUS "------------------------------------------")
     include(cmake/conan_build.cmake)
 endif()

tests/CMakeLists.txt

@@ -38,7 +38,7 @@ set(TEST_DEPENDECIES
     foonathan::lexy
     bt_sample_nodes)
 
-if(ament_cmake_FOUND AND BUILD_TESTING)
+if(ament_cmake_FOUND)
 
     find_package(ament_cmake_gtest REQUIRED)