Skip to content

L2 Document update#207

Merged
nhanasi merged 1 commit into
developfrom
feature/l2docs
May 20, 2026
Merged

L2 Document update#207
nhanasi merged 1 commit into
developfrom
feature/l2docs

Conversation

@nhanasi

@nhanasi nhanasi commented May 20, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@nhanasi
nhanasi requested a review from a team as a code owner May 20, 2026 15:32
Copilot AI review requested due to automatic review settings May 20, 2026 15:32
@github-actions

Copy link
Copy Markdown

Code Coverage Summary

                               Total:|84.1%   6224|97.3%  1660|    -      0

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds repository documentation to (1) summarize current L2 functional-test coverage for the remotedebugger component and (2) provide a “BDD feature generator” skill guide for producing/maintaining Gherkin feature specifications and gap analysis.

Changes:

  • Introduces an L2 test coverage report describing feature↔test mapping, behavior coverage, and prioritized gaps.
  • Adds a new .github/skills/bdd-feature-generator skill document detailing how to generate BDD feature files from source analysis and how to compare them to L2 tests.

Reviewed changes

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

File Description
test/functional-tests/L2_Test_Coverage.md New L2 coverage report with feature/test inventory, mapping, and gap analysis.
.github/skills/bdd-feature-generator/SKILL.md New skill documentation for generating BDD features and performing coverage gap analysis.
Comments suppressed due to low confidence (3)

test/functional-tests/L2_Test_Coverage.md:53

  • The totals row reports 97 scenarios, but adding the per-feature scenario counts in this table yields 99 (1+2+1+5+5+4+4+5+5+5+4+4+4+5+5+4+5+2+2+6+21). Please correct the totals so they match the table data.
| 19 | `rrd_deepsleep_static_report.feature` | 2 | `test_rrd_deepsleep_static_report.py` | 5 | PASS |
| 20 | `rrd_debug_report_upload.feature` | 6 | `test_rrd_debug_report_upload.py` | 5 | PASS |
| 21 | `rrd_c_api_upload.feature` | 21 | `test_rrd_c_api_upload.py` | 5 | GAP |
| | **Totals** | **97** | | **112** | |

test/functional-tests/L2_Test_Coverage.md:280

  • The gap summary reports RFC enable/disable coverage as 100% with 2 feature scenarios and only 1 test function. Given test_rrd_start_control.py doesn't force the RFC value and may only exercise one branch per run, this coverage value is overstated. Please adjust the coverage calculation/labeling here to reflect the actual behavior exercised.
|---|:---:|:---:|:---:|---|
| Daemon startup/subscribe | 1 | 4 | 100% | — |
| RFC enable/disable | 2 | 1 | 100% | — |
| Single instance | 1 | 3 | 100% | — |

test/functional-tests/L2_Test_Coverage.md:372

  • The appendix total for feature scenarios is listed as 97, but the scenario counts in the table add up to 99. Please correct the total to match the listed per-file scenario counts.
| 20 | `rrd_debug_report_upload.feature` | 6 |
| 21 | `rrd_c_api_upload.feature` | 21 |
| | **Total** | **97** |

Comment thread test/functional-tests/L2_Test_Coverage.md
Comment thread test/functional-tests/L2_Test_Coverage.md
Comment thread .github/skills/bdd-feature-generator/SKILL.md
Comment thread .github/skills/bdd-feature-generator/SKILL.md
@nhanasi
nhanasi merged commit 8c222af into develop May 20, 2026
16 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators May 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants