Added live camera tests

realsense2_camera/CMakeLists.txt

@@ -314,6 +314,41 @@ if(BUILD_TESTING)
     )
     endforeach()
   endforeach()
+
+  unset(_pytest_folders)
+
+  set(rs_query_cmd "rs-enumerate-devices -s")
+  execute_process(COMMAND bash -c ${rs_query_cmd}
+      WORKING_DIRECTORY ${PROJECT_SOURCE_DIR}
+      RESULT_VARIABLE rs_result
+      OUTPUT_VARIABLE RS_DEVICE_INFO)
+  message(STATUS "rs_device_info:")
+  message(STATUS "${RS_DEVICE_INFO}")
+  if(RS_DEVICE_INFO MATCHES "D455")
+    message(STATUS "D455 device found")
+    set(_pytest_live_folders
+      test/live_camera
+    )
+  elseif(RS_DEVICE_INFO MATCHES "D415")
+    message(STATUS "D415 device found")
+    set(_pytest_live_folders
+      test/live_camera
+    )
+  endif()
+
+  foreach(test_folder ${_pytest_live_folders})
+    file(GLOB files "${test_folder}/test_*.py")
+    foreach(file ${files})
+
+    get_filename_component(_test_name ${file} NAME_WE)
+    ament_add_pytest_test(${_test_name} ${file}
+      APPEND_ENV PYTHONPATH=${CMAKE_CURRENT_BINARY_DIR}:${CMAKE_SOURCE_DIR}/test/utils:${CMAKE_SOURCE_DIR}/launch:${CMAKE_SOURCE_DIR}/scripts
+      TIMEOUT 500
+      WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
+    )
+    endforeach()
+  endforeach()
+
 endif()
 
 # Ament exports

realsense2_camera/package.xml

@@ -38,7 +38,8 @@
   <test_depend>sensor_msgs_py</test_depend>
   <test_depend>python3-requests</test_depend>
   <test_depend>tf2_ros_py</test_depend>
-
+  <test_depend>ros2topic</test_depend>
+
   <exec_depend>launch_ros</exec_depend>
   <build_depend>ros_environment</build_depend>
 

realsense2_camera/test/README.md

@@ -146,7 +146,21 @@ The below command finds the test with the name test_static_tf_1 in realsense2_ca
 
 	pytest-3 -s -k test_static_tf_1 realsense2_camera/test/
 
-### Points to be noted while writing pytests
-The tests that are in one file are nromally run in parallel. So if there are multiple tests in one file, the system capacity can influence the test execution. It's recomended to have 3-4 tests in file, more than that can affect the test results due to delays.
-
+## Points to be noted while writing pytests
+The tests that are in one file are normally run in parallel. So if there are multiple tests in one file, the system capacity can influence the test execution. It's recomended to have 3-4 tests in file, more than that can affect the test results due to delays.
+### Passing/changing parameters
+The parameters passed while creating the node can be initialized individually for each test, please see the test_parameterized_template example for reference. The default values are taken from rs_launch.py and the passed parameters are used for overriding the default values. The parameters that can be dynamically modified can be changed using the param interface provided. However, the function create_param_ifs has to be called to create this interface. Please see the test_d455_basic_tests.py for reference. There are specific functions to change the string, integer and bool parameters, the utils can be extended if any more types are needed.
+### Difference in setting the bool parameters
+There is a difference between setting the default values of bool parameters and setting them dynamically.
+The bool test params are assinged withn quotes as below.
+	test_params_all_profiles_d455 = {
+		'camera_name': 'D455',
+		'device_type': 'D455',
+		'enable_accel':"True",
+		'enable_gyro':"True",
+		'unite_imu_method':1,
+		}
+
+However the function that implements the setting of bool parameter dynamically takes the python bool datatype. For example:
+	self.set_bool_param('enable_accel', False)
 

realsense2_camera/test/live_camera/test_camera_all_profile_tests.py

@@ -0,0 +1,264 @@
+# Copyright 2023 Intel Corporation. All Rights Reserved.
+#
+# 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
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# 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.
+
+
+import os
+import sys
+import itertools
+
+
+import pytest
+import rclpy
+
+from sensor_msgs.msg import Image as msg_Image
+from sensor_msgs.msg import Imu as msg_Imu
+from sensor_msgs.msg import PointCloud2 as msg_PointCloud2
+
+import numpy as np
+
+sys.path.append(os.path.abspath(os.path.dirname(__file__)+"/../utils"))
+import pytest_rs_utils
+from pytest_rs_utils import launch_descr_with_parameters
+from pytest_rs_utils import delayed_launch_descr_with_parameters
+from pytest_rs_utils import get_rosbag_file_path
+from pytest_rs_utils import get_node_heirarchy
+import pytest_live_camera_utils
+from rclpy.parameter import Parameter
+from rcl_interfaces.msg import ParameterValue
+from rcl_interfaces.srv import SetParameters, GetParameters, ListParameters
+from pytest_live_camera_utils import debug_print
+def check_if_skip_test(profile, format):
+    '''
+    if profile == 'Color':
+        if "BGRA8" == format:
+            return True
+        if "RGBA8" == format:
+            return True
+        if "Y8" == format:
+            return True
+    elif profile == 'Depth':
+        if "Z16" == format:
+            return True
+    el
+    if profile == 'Infrared':
+        if "Y8" == format:
+            return True
+        if "Y16" == format:
+            return True
+        if "BGRA8" == format:
+            return True
+        if "RGBA8" == format:
+            return True
+        if "Y10BPACK" == format:
+            return True
+        if "UYVY" == format:
+            return True
+        if "BGR8" == format:
+            return True
+        if "RGB8" == format:
+            return True
+        if "RAW10" == format:
+            return True
+    elif profile == 'Infrared1':
+        if "Y8" ==format:
+            return True
+        if "Y16" ==format:
+            return True
+        if "Y10BPACK"  == format:
+            return True
+        if "UYVY" ==format:
+            return True
+        if "BGR8" ==format:
+            return True
+        if "RGB8" ==format:
+            return True
+        if "RAW10" ==format:
+            return True
+    if profile == 'Infrared2':                    
+        if "Y8" == format:
+            return True
+        if "Y16" == format:
+            return True
+        if "Y10BPACK" == format:
+            return True
+        if "UYVY" == format:
+            return True
+        if "BGR8" == format:
+            return True
+        if "RGB8" == format:
+            return True
+        if "RAW10" == format:
+            return True
+    '''
+    if profile == 'Infrared':
+        if "Y8" == format:
+            return True
+        if "Y16" == format:
+            return True
+    if profile == 'Infrared1':
+        if "Y8" ==format:
+            return True
+        if "Y16" ==format:
+            return True
+    if profile == 'Infrared2':                    
+        if "Y8" == format:
+            return True
+        if "Y16" == format:
+            return True
+    return False
+
+
+test_params_all_profiles_d455 = {
+    'camera_name': 'D455',
+    'device_type': 'D455',
+    }
+test_params_all_profiles_d415 = {
+    'camera_name': 'D415',
+    'device_type': 'D415',
+    }
+test_params_all_profiles_d435 = {
+    'camera_name': 'D435',
+    'device_type': 'D435',
+    }
+
+
+'''
+This test was implemented as a template to set the parameters and run the test.
+This directory is not added to the CMakeLists.txt so as to avoid the colcon failure in the
+machines that don't have the D455 connected.
+1. Only a subset of parameter types are implemented in py_rs_utils, it has to be extended for others
+2. After setting the param, rclpy.spin_once may be needed.Test passes even without this though.
+'''
+@pytest.mark.parametrize("launch_descr_with_parameters", [
+    pytest.param(test_params_all_profiles_d455, marks=pytest.mark.d455),
+    pytest.param(test_params_all_profiles_d415, marks=pytest.mark.d415),
+    pytest.param(test_params_all_profiles_d435, marks=pytest.mark.d435),]
+    ,indirect=True)
+@pytest.mark.launch(fixture=launch_descr_with_parameters)
+class TestLiveCamera_Change_Resolution(pytest_rs_utils.RsTestBaseClass):
+    def test_LiveCamera_Change_Resolution(self,launch_descr_with_parameters):
+        skipped_tests = []
+        failed_tests = []
+        num_passed = 0
+        num_failed = 0
+        params = launch_descr_with_parameters[1]
+        themes = [{'topic':get_node_heirarchy(params)+'/color/image_raw', 'msg_type':msg_Image,'expected_data_chunks':1}]
+        config = pytest_live_camera_utils.get_profile_config(get_node_heirarchy(params))
+        try:
+            ''' 
+            initialize, run and check the data 
+            '''
+            self.init_test("RsTest"+params['camera_name'])
+            self.spin_for_time(wait_time=1.0)
+            cap = pytest_live_camera_utils.get_camera_capabilities(params['device_type'])
+            if cap == None:
+                debug_print("Device not found? : " + params['device_type'])
+                return
+            self.create_param_ifs(get_node_heirarchy(params))
+            config["Color"]["default_profile1"],config["Color"]["default_profile2"] = pytest_live_camera_utils.get_default_profiles(cap["color_profile"], "Color")
+            config["Depth"]["default_profile1"],config["Depth"]["default_profile2"] = pytest_live_camera_utils.get_default_profiles(cap["depth_profile"], "Depth")
+            config["Infrared"]["default_profile1"],config["Infrared"]["default_profile2"] = pytest_live_camera_utils.get_default_profiles(cap["depth_profile"], "Infrared")
+            config["Infrared1"]["default_profile1"],config["Infrared1"]["default_profile2"] = pytest_live_camera_utils.get_default_profiles(cap["depth_profile"], "Infrared1")
+            config["Infrared2"]["default_profile1"],config["Infrared2"]["default_profile2"] = pytest_live_camera_utils.get_default_profiles(cap["depth_profile"], "Infrared2")
+            for key in cap["color_profile"]:
+                profile_type = key[0]
+                profile = key[1]
+                format = key[2]
+                if check_if_skip_test(profile_type, format):
+                    skipped_tests.append(" ".join(key))
+                    continue
+                print("Testing " + " ".join(key))
+                themes[0]['topic'] = config[profile_type]['topic']
+                themes[0]['width'] = int(profile.split('x')[0])
+                themes[0]['height'] = int(profile.split('x')[1])
+                if themes[0]['width'] == int(config[profile_type]["default_profile2"].split('x')[0]):
+                    self.set_string_param(config[profile_type]["profile"], config[profile_type]["default_profile1"])
+                else:
+                    self.set_string_param(config[profile_type]["profile"], config[profile_type]["default_profile2"])
+                self.set_bool_param(config[profile_type]["param"], True)
+                self.disable_all_params()
+                self.spin_for_time(wait_time=0.2)
+                self.set_string_param(config[profile_type]["profile"], profile)
+                self.set_string_param(config[profile_type]["format"], format)
+                self.set_bool_param(config[profile_type]["param"], True)
+                try:
+                    ret = self.run_test(themes)
+                    assert ret[0], ret[1]
+                    assert self.process_data(themes), " ".join(key) + " failed"
+                    num_passed += 1
+                except Exception as e:
+                    exc_type, exc_obj, exc_tb = sys.exc_info()
+                    fname = os.path.split(exc_tb.tb_frame.f_code.co_filename)[1]
+                    print("Test failed")
+                    print(e)
+                    print(exc_type, fname, exc_tb.tb_lineno)
+                    num_failed += 1
+                    failed_tests.append(" ".join(key))
+            debug_print("Color tests completed")
+            for key in cap["depth_profile"]:
+                profile_type = key[0]
+                profile = key[1]
+                format = key[2]
+                if check_if_skip_test(profile_type, format):
+                    skipped_tests.append(" ".join(key))
+                    continue
+                print("Testing " + " ".join(key))
+
+                themes[0]['topic'] = config[profile_type]['topic']
+                themes[0]['width'] = int(profile.split('x')[0])
+                themes[0]['height'] = int(profile.split('x')[1])
+                if themes[0]['width'] == int(config[profile_type]["default_profile2"].split('x')[0]):
+                    self.set_string_param(config[profile_type]["profile"], config[profile_type]["default_profile1"])
+                else:
+                    self.set_string_param(config[profile_type]["profile"], config[profile_type]["default_profile2"])
+                self.set_bool_param(config[profile_type]["param"], True)
+
+
+                self.disable_all_params()
+                self.spin_for_time(wait_time=0.2)
+                self.set_string_param(config[profile_type]["profile"], profile)
+                self.set_string_param(config[profile_type]["format"], format)
+                self.set_bool_param(config[profile_type]["param"], True)
+                try:
+                    ret = self.run_test(themes)
+                    assert ret[0], ret[1]
+                    assert self.process_data(themes), " ".join(key) + " failed"
+                    num_passed += 1
+                except Exception as e:
+                    print("Test failed")
+                    print(e)
+                    num_failed += 1
+                    failed_tests.append(" ".join(key))
+            debug_print("Depth tests completed")
+        finally:
+            #this step is important because the test will fail next time
+            pytest_rs_utils.kill_realsense2_camera_node()
+            self.shutdown()
+            print("Tests passed " + str(num_passed))
+            print("Tests skipped " + str(len(skipped_tests)))
+            if len(skipped_tests):
+                debug_print("\nSkipped tests:" + params['device_type'])
+                debug_print("\n".join(skipped_tests))
+            print("Tests failed " + str(num_failed))
+            if len(failed_tests):
+                print("\nFailed tests:" + params['device_type'])
+                print("\n".join(failed_tests))
+
+    def disable_all_params(self):
+        self.set_bool_param('enable_color', False)
+        self.set_bool_param('enable_depth', False)
+        self.set_bool_param('enable_infra', False)
+        self.set_bool_param('enable_infra1', False)
+        self.set_bool_param('enable_infra2', False)
+