test: run the PCAP-NG JSON report test now that keys are escaped (#1795)

[JarryShaw]

Please follow the guide below

• You will be asked some questions, please read them carefully and answer honestly

• Put an x into all the boxes [ ] relevant to your pull request (like that [x])

• Use Preview tab to see how your pull request will actually look like

• Searched for similar pull requests

• Followed the coding style (make pylint, make mypy, make isort)

• make test passes, and a test case covers the change. Narrow runs only, listed below.

• Added a changelog entry under docs/source/changelog/ and regenerated CHANGELOG.md, if the change is user-visible: N/A — centralised in docs(changelog): shared 1.5.0 changelog — long-lived, merges last (#610, #616, #617, #618, #620) #657

What is the purpose of your pull request?

Tick the commit type your subject line carries.

• fix — corrects a defect
• feat — adds a feature
• perf — changes performance, not behaviour
• refactor — changes neither behaviour nor performance
• test — tests only
• docs — documentation only
• ci — workflows or build tooling
• release — bumps the version or rolls up a distribution
• chore — anything else

---

Summary

tests/integration/test_pcapng_end_to_end.py skipped PcapngUnescapedKeyTests.test_json_report_of_a_decryption_secrets_block_parses as "blocked on dictdumper". #1158 (for #1152) made pcapkit escape JSON mapping keys itself, so the block was no longer real.

• Skip removed. The class and test docstrings now say what the test guards: the JSON report parses because pcapkit escapes keys, although dictdumper still writes them raw. Line-number citations are replaced by symbol names.
• HAS_RUNTIME guard added, matching the sibling classes, so a checkout without the runtime dependencies skips rather than fails. tests/integration/_helpers.py now says every subclass carries the guard.
• Bite: with escape_key's JSON branch turned into a no-op, the test fails with JSONDecodeError. With HAS_RUNTIME false, it skips.
• Runs: the file has 7 passed; tests/project and tests/test_tier_guard.py pass.

Closes #1795

[JarryShaw]

Verdict: GOOD TO GO at 58da7d377. A Sonnet cross-review and re-check (author: Opus). The skip removal is right: #1158 escapes the keys. The new HAS_RUNTIME guard matches its siblings, and an AST scan finds no unguarded subclass left. The test fails with escaping disabled and skips without the runtime dependencies. tests/project and the tier guard pass.

[github-actions]

Coverage: 89.60% (unit tier, Python 3.15, 58da7d377, Unit Tests run success)

Package	Statements	Missed	Branches	Partial	Cover
pcapkit (top level)	104	3	20	3	95.16%
pcapkit/const	18862	954	2368	868	91.06%
pcapkit/corekit	2462	69	780	35	96.55%
pcapkit/dumpkit	276	3	100	3	98.40%
pcapkit/foundation	2915	164	1016	34	93.59%
pcapkit/interface	138	7	52	5	93.68%
pcapkit/protocols	17197	224	4580	189	98.09%
pcapkit/toolkit	872	177	298	8	77.18%
pcapkit/utilities	512	2	158	4	99.10%
pcapkit/vendor	4417	2342	1010	157	43.38%

Per-file detail: the coverage-html artifact of this run.