Skip to content

[PW_SID:1181897] Test links completing after their connection was aborted - #2639

Open
BluezTestBot wants to merge 2 commits into
masterfrom
1181897
Open

BluezTestBot wants to merge 2 commits into
masterfrom
1181897

Conversation

@BluezTestBot

Copy link
Copy Markdown

Add a test that closes the socket while the emulator holds the
Synchronous Connection Complete event of its setup, so that the kernel
aborts the connection before the link comes up. No connection takes the
link then, so the test expects the kernel to disconnect it, as the
controller would otherwise keep it and reject further setups for the
device.

eSCO CVSD - Close Before Complete

Assisted-by: Claude:claude-opus-5-5

tools/sco-tester.c | 45 +++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 45 insertions(+)

Add a test that closes the socket while the emulator holds the
Synchronous Connection Complete event of its setup, so that the kernel
aborts the connection before the link comes up. No connection takes the
link then, so the test expects the kernel to disconnect it, as the
controller would otherwise keep it and reject further setups for the
device.

eSCO CVSD - Close Before Complete

Assisted-by: Claude:claude-opus-5-5
Add BR/EDR and LE client tests that close the socket while the emulator
holds the Connection Complete or LE Connection Complete event, so that
the kernel aborts the connection before the link comes up. No
connection takes the link then, so the tests expect the kernel to
disconnect it instead of leaving it up in the controller.

L2CAP BR/EDR Client - Close Before Complete
L2CAP LE Client - Close Before Complete

Assisted-by: Claude:claude-opus-5-5
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 27 complexity · 2 duplication

Metric Results
Complexity 27
Duplication 2

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@BluezTestBot

Copy link
Copy Markdown
Author

CheckPatch
Desc: Run checkpatch.pl script
Duration: 0.69 seconds
Result: PASS

@BluezTestBot

Copy link
Copy Markdown
Author

GitLint
Desc: Run gitlint
Duration: 0.43 seconds
Result: FAIL
Output:

[BlueZ,2/2] tools/l2cap-tester: Add tests closing the socket before the link is up

1: T1 Title exceeds max length (82>80): "[BlueZ,2/2] tools/l2cap-tester: Add tests closing the socket before the link is up"

@BluezTestBot

Copy link
Copy Markdown
Author

BuildEll
Desc: Build and Install ELL
Duration: 19.21 seconds
Result: PASS

@BluezTestBot

Copy link
Copy Markdown
Author

BluezMake
Desc: Build BlueZ
Duration: 394.48 seconds
Result: PASS

@BluezTestBot

Copy link
Copy Markdown
Author

CheckSmatch
Desc: Run smatch tool with source
Duration: 290.89 seconds
Result: WARNING
Output:

tools/sco-tester.c: note: in included file:./lib/bluetooth/bluetooth.h:232:15: warning: array of flexible structures./lib/bluetooth/bluetooth.h:237:31: warning: array of flexible structures

@BluezTestBot

Copy link
Copy Markdown
Author

bluezmakeextell
Desc: Build Bluez with External ELL
Duration: 92.90 seconds
Result: PASS

@BluezTestBot

Copy link
Copy Markdown
Author

TestFunctional
Desc: Run test-functional
Duration: 1153.65 seconds
Result: PASS

@BluezTestBot

Copy link
Copy Markdown
Author

IncrementalBuild
Desc: Incremental build with the patches in the series
Duration: 385.57 seconds
Result: PASS

@BluezTestBot

Copy link
Copy Markdown
Author

ScanBuild
Desc: Run Scan Build
Duration: 1121.13 seconds
Result: PASS

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants