Skip to content

fix(Android): USB-host serial stack overhaul (lifecycle FSM, async write pump, hot-plug)#14536

Open
HTRamsey wants to merge 1 commit into
mavlink:masterfrom
HTRamsey:fix/android-serial-overhaul
Open

fix(Android): USB-host serial stack overhaul (lifecycle FSM, async write pump, hot-plug)#14536
HTRamsey wants to merge 1 commit into
mavlink:masterfrom
HTRamsey:fix/android-serial-overhaul

Android serial: USB-host stack overhaul

949efdb
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jun 18, 2026 in 0s

30.55% (target 30.00%)

View this Pull Request on Codecov

30.55% (target 30.00%)

Details

Codecov Report

❌ Patch coverage is 38.41121% with 659 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.55%. Comparing base (f29efd3) to head (949efdb).
⚠️ Report is 104 commits behind head on master.

Files with missing lines Patch % Lines
src/Comms/Serial/SerialLink.cc 41.09% 61 Missing and 101 partials ⚠️
src/Comms/Serial/QGCSerialPortInfo.cc 31.97% 60 Missing and 74 partials ⚠️
src/Vehicle/VehicleSetup/Bootloader.cc 0.00% 118 Missing ⚠️
src/Comms/Serial/HostSerialPort.cc 37.60% 56 Missing and 17 partials ⚠️
src/Comms/Serial/NmeaSerialDevice.cc 46.98% 29 Missing and 15 partials ⚠️
src/Comms/LinkManager.cc 17.14% 23 Missing and 6 partials ⚠️
src/GPS/SerialGPSTransport.cc 0.00% 21 Missing ⚠️
src/Android/Serial/SerialPortInfoCodec.cc 48.64% 0 Missing and 19 partials ⚠️
src/Comms/Serial/QGCSerialPortInfo_host.cc 64.00% 2 Missing and 7 partials ⚠️
src/Comms/Serial/HostSerialPort.h 38.46% 8 Missing ⚠️
... and 10 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14536      +/-   ##
==========================================
+ Coverage   25.47%   30.55%   +5.08%     
==========================================
  Files         769      787      +18     
  Lines       65912    67541    +1629     
  Branches    30495    31291     +796     
==========================================
+ Hits        16788    20635    +3847     
+ Misses      37285    32927    -4358     
- Partials    11839    13979    +2140     
Files with missing lines Coverage Δ
src/Android/Serial/AndroidSerialPortRegistry.h 100.00% <100.00%> (ø)
src/Android/Serial/AndroidSerialRxQueue.h 100.00% <100.00%> (ø)
src/Comms/LinkManager.h 83.33% <ø> (+33.33%) ⬆️
src/Comms/Serial/NmeaSerialDevice.h 100.00% <100.00%> (ø)
src/Comms/Serial/SerialPlatform.cc 100.00% <100.00%> (ø)
src/Vehicle/VehicleSetup/Bootloader.h 0.00% <ø> (ø)
.../Vehicle/VehicleSetup/FirmwareUpgradeController.cc 9.64% <ø> (ø)
...c/Vehicle/VehicleSetup/FirmwareUpgradeController.h 25.00% <ø> (ø)
src/Comms/Serial/SerialPlatform.h 75.00% <75.00%> (ø)
src/AutoPilotPlugins/APM/APMAutoPilotPlugin.cc 35.86% <0.00%> (+13.48%) ⬆️
... and 18 more

... and 375 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 27065dc...949efdb. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.