tests: bluetooth: custom LTK feature tests

[ardo-nordic]

Adding bsim tests for basic custom LTK feature functions, it's coexistence with SMP and some invalid scenarios.

[Copilot]

Pull request overview

This PR introduces bsim tests for the custom LTK (Long Term Key) feature in Bluetooth functionality. The tests verify basic custom LTK operations, its coexistence with the Security Manager Protocol (SMP), and various invalid scenarios.

Key Changes

• Added comprehensive test suite with three test scripts covering basic functionality, SMP coexistence, and invalid scenarios
• Implemented test cases for both central and peripheral roles
• Created test infrastructure including configuration files and build setup

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/bluetooth/bsim/custom_ltk/testcase.yaml	Test configuration defining build and platform requirements
tests/bluetooth/bsim/custom_ltk/test_scripts/custom_ltk_invalid.sh	Shell script for testing invalid LTK scenarios
tests/bluetooth/bsim/custom_ltk/test_scripts/custom_ltk_coex.sh	Shell script for testing custom LTK coexistence with SMP
tests/bluetooth/bsim/custom_ltk/test_scripts/custom_ltk.sh	Shell script for testing basic custom LTK functionality
tests/bluetooth/bsim/custom_ltk/src/main.c	Main test implementation with test vectors and callbacks
tests/bluetooth/bsim/custom_ltk/prj.conf	Project configuration enabling required Bluetooth features
tests/bluetooth/bsim/custom_ltk/CMakeLists.txt	CMake build configuration for the test suite

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Missing space between 'bondable' and 'failed' in error message.

Suggested change

	TEST_ASSERT(!err, "bt_conn_set_bondablefailed (%d).", err);
	TEST_ASSERT(!err, "bt_conn_set_bondable failed (%d).", err);

[Copilot]

Corrected spelling of 'paskey' to 'passkey'.

Suggested change

	LOG_INF("Enable paskey");
	LOG_INF("Enable passkey");

[NordicBuilder]

CI Information

^{To view the history of this post, click the 'edited' button above}
Build number: 5

Inputs:

Sources:

more details

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

• ❌ Toolchain
• ❌ Build twister
• ❌ Integration tests

Note: This message is automatically posted and updated by the CI

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Missing space before opening brace. Add a space between ')' and '{' to match the coding style used throughout the file.

Suggested change

	if (test->conn_cb){
	if (test->conn_cb) {

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Tab character used for indentation instead of space. This is inconsistent with the surrounding code which uses spaces.

Suggested change

	err = bt_unpair(BT_ID_DEFAULT, BT_ADDR_LE_ANY);
	err = bt_unpair(BT_ID_DEFAULT, BT_ADDR_LE_ANY);