[nhanasi]

No description provided.

[github-actions]

Code Coverage Summary

                               Total:|84.0%   6280|97.3%  1668|    -      0

[Copilot]

🟢 Approval recommended

No active review issues are reported for the L2 workflow’s switch to the test repository’s feature/l2docupdate branch.

No open issues 🎉

What changed in this PR

This PR changes which branch of the external test repository is used by the remote debugger’s L2 test workflow.

Changes:

• Switches the rdkcentral/docker-device-mgt-service-test checkout from develop to feature/l2docupdate.

File	Description
.github/​workflows/​L2-tests.yml	Points the test repository checkout to feature/l2docupdate.

_{Review effort: Balanced}

[github-actions]

Code Coverage Summary

                               Total:|84.0%   6280|97.3%  1668|    -      0

[Copilot]

🟡 Changes recommended

The new fail-fast invocation aborts on clean checkouts because the shared runner calls automake before autoreconf generates aclocal.m4, preventing both builds from reaching tests.

1 open issue

• Run autoreconf before automake in the test runner · New

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

Fail-fast execution stops both PR and baseline jobs because Automake runs before Autotools bootstrapping on fresh checkouts, preventing tests and coverage collection.

1 open issue

• Run autoreconf before automake in the test runner

2 previously missed issues

In code that hasn't changed since last review

Bootstrap Autotools in the base checkout

.github/​workflows/​L1-Test.yml:47

The fresh base checkout lacks generated Autotools files, but run_ut.sh invokes Automake before Autoreconf. With sh -e, the initial Automake failure stops baseline coverage collection; bootstrapping the PR checkout does not initialize this separate tree. Run autoreconf --install in the base checkout's src/unittest before invoking the runner.

Exclude test and mock code from coverage totals

.github/​workflows/​L1-Test.yml:81

These totals include the test runner and mocks: run_ut.sh filters out only /usr/*, while src/unittest/Makefile.am instruments the entire runner. Adding covered test lines can therefore make this check pass even when production line coverage decreases. Exclude unit-test and mock sources from both totals before comparing coverage.

_{Review effort: Balanced}