tests: Bluetooth: Add BT Tester GAP smoke test

Add a babblesim test of the BT Tester doing a GAP smoke test
connecting 2 BT testers using BTP.

The purpose of this is to further increase the test coverage
of the BT Tester in CI, as it is only being built, and runtime
errors are typically not caught.

Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>

tests/bluetooth/tester/CMakeLists.txt

@@ -7,6 +7,13 @@ LIST(APPEND QEMU_EXTRA_FLAGS -serial unix:/tmp/bt-stack-tester)
 find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
 project(tester)
 
+if(CONFIG_SOC_SERIES_BSIM_NRFXX)
+zephyr_include_directories(
+  ${BSIM_COMPONENTS_PATH}/libUtilv1/src/
+  ${BSIM_COMPONENTS_PATH}/libPhyComv1/src/
+  )
+endif() # CONFIG_SOC_SERIES_BSIM_NRFXX
+
 zephyr_library_include_directories(${ZEPHYR_BASE}/samples/bluetooth)
 zephyr_library_include_directories(${ZEPHYR_BASE}/subsys/bluetooth/mesh)
 zephyr_library_include_directories(${ZEPHYR_BASE}/subsys/bluetooth/host)

tests/bluetooth/tester/Kconfig.sysbuild

@@ -7,6 +7,7 @@ config NET_CORE_BOARD
 	string
 	default "nrf5340dk/nrf5340/cpunet" if "$(BOARD)" = "nrf5340dk"
 	default "nrf5340_audio_dk/nrf5340/cpunet" if "$(BOARD)" = "nrf5340_audio_dk"
+	default "nrf5340bsim/nrf5340/cpunet" if $(BOARD_TARGET_STRING) = "NRF5340BSIM_NRF5340_CPUAPP"
 
 config NET_CORE_IMAGE_HCI_IPC
 	bool "HCI IPC image on network core"

tests/bluetooth/tester/boards/nrf52_bsim.conf

@@ -0,0 +1,15 @@
+# CONFIG_TEST enforces minimal logging, which we don't want
+CONFIG_TEST=n
+
+CONFIG_ASSERT=y
+CONFIG_THREAD_NAME=y
+
+CONFIG_LOG=y
+CONFIG_LOG_BACKEND_SHOW_COLOR=y
+CONFIG_LOG_FUNC_NAME_PREFIX_ERR=y
+CONFIG_LOG_FUNC_NAME_PREFIX_WRN=y
+
+CONFIG_LOG_DEFAULT_LEVEL=3
+CONFIG_BTTESTER_LOG_LEVEL_DBG=y
+
+CONFIG_UART_PIPE=n

tests/bluetooth/tester/boards/nrf5340bsim_nrf5340_cpuapp.conf

@@ -0,0 +1,15 @@
+# CONFIG_TEST enforces minimal logging, which we don't want
+CONFIG_TEST=n
+
+CONFIG_ASSERT=y
+CONFIG_THREAD_NAME=y
+
+CONFIG_LOG=y
+CONFIG_LOG_BACKEND_SHOW_COLOR=y
+CONFIG_LOG_FUNC_NAME_PREFIX_ERR=y
+CONFIG_LOG_FUNC_NAME_PREFIX_WRN=y
+
+CONFIG_LOG_DEFAULT_LEVEL=3
+CONFIG_BTTESTER_LOG_LEVEL_DBG=y
+
+CONFIG_UART_PIPE=n

tests/bluetooth/tester/src/btp.c

@@ -185,6 +185,19 @@ static void uart_send(const uint8_t *data, size_t len)
 {
 	uart_pipe_send(data, len);
 }
+#elif defined(CONFIG_SOC_SERIES_BSIM_NRFXX)
+extern void btp_register_tester(uint8_t *buffer, size_t len, uart_pipe_recv_cb cb);
+extern void btp_send_to_bsim(const uint8_t *data, size_t len);
+
+static void uart_init(uint8_t *data)
+{
+	btp_register_tester(data, BTP_MTU, recv_cb);
+}
+
+static void uart_send(const uint8_t *data, size_t len)
+{
+	btp_send_to_bsim(data, len);
+}
 #else /* !CONFIG_UART_PIPE */
 static uint8_t *recv_buf;
 static size_t recv_off;

tests/bluetooth/tester/src/btp/btp.h

@@ -87,6 +87,8 @@
 
 #define BTP_STATUS_VAL(err) (err) ? BTP_STATUS_FAILED : BTP_STATUS_SUCCESS
 
+#define BTP_EVENT_OPCODE 0x80
+
 /* TODO indicate delay response, should be removed when all commands are
  * converted to cmd+status+ev pattern
  */

tests/bluetooth/tester/src/btp_gap.c

@@ -725,6 +725,7 @@ static uint8_t start_advertising(const void *cmd, uint16_t cmd_len,
 	if ((cmd_len < sizeof(*cp)) ||
 	    (cmd_len != sizeof(*cp) + cp->adv_data_len + cp->scan_rsp_len +
 			    sizeof(duration) + sizeof(own_addr_type))) {
+		LOG_DBG("Invalid command length: %u", cmd_len);
 		return BTP_STATUS_FAILED;
 	}
 
@@ -759,6 +760,7 @@ static uint8_t start_advertising(const void *cmd, uint16_t cmd_len,
 
 	err = tester_gap_create_adv_instance(&param, own_addr_type, ad, adv_len, sd, sd_len, NULL);
 	if (err != 0) {
+		LOG_DBG("Failed to create adv instance: %d", err);
 		return BTP_STATUS_FAILED;
 	}
 
@@ -771,7 +773,7 @@ static uint8_t start_advertising(const void *cmd, uint16_t cmd_len,
 
 	/* BTP API don't allow to set empty scan response data. */
 	if (err < 0) {
-		LOG_ERR("Failed to start advertising");
+		LOG_ERR("Failed to start advertising: %d", err);
 
 		return BTP_STATUS_FAILED;
 	}

tests/bluetooth/tester/src/main.c

@@ -2,14 +2,21 @@
 
 /*
  * Copyright (c) 2015-2016 Intel Corporation
+ * Copyright (c) 2025 Nordic Semiconductor ASA
  *
  * SPDX-License-Identifier: Apache-2.0
  */
 
+#include <zephyr/autoconf.h>
+
 #include <zephyr/kernel.h>
 #include <zephyr/types.h>
 #include <zephyr/toolchain.h>
 
+#if defined(CONFIG_SOC_SERIES_BSIM_NRFXX)
+#include "bstests.h"
+#endif /* CONFIG_SOC_SERIES_BSIM_NRFXX */
+
 #include <zephyr/logging/log.h>
 #define LOG_MODULE_NAME bttester_main
 LOG_MODULE_REGISTER(LOG_MODULE_NAME, CONFIG_BTTESTER_LOG_LEVEL);
@@ -19,5 +26,10 @@ LOG_MODULE_REGISTER(LOG_MODULE_NAME, CONFIG_BTTESTER_LOG_LEVEL);
 int main(void)
 {
 	tester_init();
+
+#if defined(CONFIG_SOC_SERIES_BSIM_NRFXX)
+	bst_main();
+#endif /* CONFIG_SOC_SERIES_BSIM_NRFXX */
+
 	return 0;
 }

tests/bluetooth/tester/sysbuild.cmake

@@ -19,7 +19,7 @@ if(SB_CONFIG_NET_CORE_IMAGE_HCI_IPC)
 	)
 
 	set(${NET_APP}_EXTRA_CONF_FILE
-	 ${APP_DIR}/hci_ipc_cpunet.conf
+	 ${ZEPHYR_BASE}/tests/bluetooth/tester/hci_ipc_cpunet.conf
 	 CACHE INTERNAL ""
 	)
 

tests/bsim/bluetooth/compile.nrf5340bsim_nrf5340_cpuapp.sh

@@ -13,6 +13,7 @@ export BOARD="${BOARD:-nrf5340bsim/nrf5340/cpuapp}"
 source ${ZEPHYR_BASE}/tests/bsim/compile.source
 
 ${ZEPHYR_BASE}/tests/bsim/bluetooth/host/compile.sh
+${ZEPHYR_BASE}/tests/bsim/bluetooth/tester/compile.sh
 
 app=tests/bsim/bluetooth/ll/conn conf_file=prj_split_privacy.conf sysbuild=1  compile
 app=tests/bsim/bluetooth/ll/throughput sysbuild=1 compile

tests/bsim/bluetooth/compile.sh

@@ -24,5 +24,6 @@ ${ZEPHYR_BASE}/tests/bsim/bluetooth/samples/compile.sh
 if [ ${BOARD} == "nrf52_bsim/native" ]; then
 	${ZEPHYR_BASE}/tests/bsim/bluetooth/hci_uart/compile.sh
 fi
+${ZEPHYR_BASE}/tests/bsim/bluetooth/tester/compile.sh
 
 wait_for_background_jobs

tests/bsim/bluetooth/tester/CMakeLists.txt

@@ -0,0 +1,29 @@
+# SPDX-License-Identifier: Apache-2.0
+
+cmake_minimum_required(VERSION 3.20.0)
+
+find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
+project(tester_bsim)
+
+# Add the BSIM tester
+add_subdirectory(${ZEPHYR_BASE}/tests/bluetooth/tester ${CMAKE_BINARY_DIR}/tester_build)
+
+# This contains babblesim-specific helpers, e.g. device synchronization.
+add_subdirectory(${ZEPHYR_BASE}/tests/bsim/babblekit babblekit)
+target_link_libraries(app PRIVATE babblekit)
+
+# Add the testlib
+add_subdirectory(${ZEPHYR_BASE}/tests/bluetooth/common/testlib testlib)
+target_link_libraries(app PRIVATE testlib)
+
+zephyr_include_directories(
+  ${BSIM_COMPONENTS_PATH}/libUtilv1/src/
+  ${BSIM_COMPONENTS_PATH}/libPhyComv1/src/
+  )
+
+target_sources(app PRIVATE
+  src/btp.c
+  src/gap_central.c
+  src/gap_peripheral.c
+  src/test_main.c
+)

tests/bsim/bluetooth/tester/Kconfig

@@ -0,0 +1,4 @@
+# Copyright (c) 2024-2025 Nordic Semiconductor ASA
+# SPDX-License-Identifier: Apache-2.0
+
+source "${ZEPHYR_BASE}/tests/bluetooth/tester/Kconfig"

tests/bsim/bluetooth/tester/Kconfig.sysbuild

@@ -0,0 +1,10 @@
+# Copyright (c) 2023-2025 Nordic Semiconductor ASA
+# SPDX-License-Identifier: Apache-2.0
+
+source "${ZEPHYR_BASE}/tests/bluetooth/tester/Kconfig.sysbuild"
+
+config NATIVE_SIMULATOR_PRIMARY_MCU_INDEX
+	int
+	# Let's pass the test arguments to the application MCU test
+	# otherwise by default they would have gone to the net core.
+	default 0 if $(BOARD_TARGET_STRING) = "NRF5340BSIM_NRF5340_CPUAPP"

tests/bsim/bluetooth/tester/compile.sh

@@ -0,0 +1,12 @@
+#!/usr/bin/env bash
+# Copyright 2025 Nordic Semiconductor ASA
+# SPDX-License-Identifier: Apache-2.0
+
+# Compile all the applications needed by the bsim tests in these subfolders
+
+#set -x #uncomment this line for debugging
+set -ue
+
+: "${ZEPHYR_BASE:?ZEPHYR_BASE must be set to point to the zephyr root directory}"
+
+west twister -T ${ZEPHYR_BASE}/tests/bsim/bluetooth/tester/

tests/bsim/bluetooth/tester/prj.conf

@@ -0,0 +1,2 @@
+# Please build using the sample configuration file:
+# ${ZEPHYR_BASE}/tests/bluetooth/tester/prj.conf