Skip to content

bluetooth_vsc_espressif: Initial VSC batch - #5172

Open
antoniovazquezblanco wants to merge 1 commit into
secdev:masterfrom
antoniovazquezblanco:vscesp
Open

bluetooth_vsc_espressif: Initial VSC batch#5172
antoniovazquezblanco wants to merge 1 commit into
secdev:masterfrom
antoniovazquezblanco:vscesp

Conversation

@antoniovazquezblanco

Copy link
Copy Markdown
Contributor

AI-Assisted: yes (Claude Opus 5)

Description

More vendors :D

@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.15%. Comparing base (251daec) to head (f165aea).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5172      +/-   ##
==========================================
+ Coverage   81.13%   81.15%   +0.02%     
==========================================
  Files         392      393       +1     
  Lines       97843    97932      +89     
==========================================
+ Hits        79384    79480      +96     
+ Misses      18459    18452       -7     
Files with missing lines Coverage Δ
scapy/contrib/bluetooth_vsc_espressif.py 100.00% <100.00%> (ø)

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@antoniovazquezblanco
antoniovazquezblanco force-pushed the vscesp branch 3 times, most recently from 8436431 to c7018e3 Compare September 12, 2026 09:30
@antoniovazquezblanco

Copy link
Copy Markdown
Contributor Author

I am looking into test failures. Those are due to different VSCs colliding between different vendors.

AI-Assisted: yes (Claude Opus 5)
@antoniovazquezblanco

antoniovazquezblanco commented Sep 12, 2026

Copy link
Copy Markdown
Contributor Author

Tests fixed on my side. This prompts for a future design question at #5173. For the time being, I consider this is already reviewable.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant