Skip to content

test: add coverage for graph_query render_text and render_dot#1587

Merged
jamesadevine merged 1 commit into
mainfrom
test/graph-query-render-coverage-d2b3dc1a8abbba29
Jul 21, 2026
Merged

test: add coverage for graph_query render_text and render_dot#1587
jamesadevine merged 1 commit into
mainfrom
test/graph-query-render-coverage-d2b3dc1a8abbba29

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Test Gap Fixes

Test suite snapshot: 2,813 tests (unit + integration), 7 tests in inspect_integration (was 5)

Added Coverage

Module Function/Path Why It Matters Test Added
inspect/graph_query.rs render_text — all text sections render_text is a public pure function with 0 existing tests; exercises Step locations, Job edges, Job step counts, and Outputs-needing-isOutput sections graph_text_emits_expected_sections_for_standalone_pipeline
inspect/graph_query.rs render_dotPipelineBodySummary::Stages branch The DOT subgraph-cluster path was unreachable from the existing graph_dot_emits_digraph_with_known_edges test (which only uses a standalone / flat-jobs fixture); the Stages arm was dead coverage graph_dot_emits_subgraph_clusters_for_stage_template

Validation

  • cargo test --test inspect_integration — 7/7 pass
  • cargo clippy --all-targets --all-features — clean

This PR was created by the automated test gap finder. Previous run: none (first run). Modules audited this cycle: sanitize, validate, fuzzy_schedule, secure, hash, ndjson, agent_stats, execute, compile/filter_ir, compile/pr_filters, compile/path_layout_check, compile/source_path_guard, compile/gitattributes, compile/codemods, inspect/graph_query, inspect/graph_deps, inspect/graph_outputs, inspect/lint, inspect/whatif.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • spsprodeus21.vssps.visualstudio.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "spsprodeus21.vssps.visualstudio.com"

See Network Configuration for more information.

Generated by Test Gap Finder · 120.7 AIC · ⌖ 12.3 AIC · ⊞ 7.2K ·

Add two integration tests to cover previously untested rendering paths
in src/inspect/graph_query.rs:

1. graph_text_emits_expected_sections_for_standalone_pipeline — verifies
   render_text emits all expected text sections (Step locations, Job edges,
   Job step counts, Outputs needing isOutput=true) for a standalone pipeline.

2. graph_dot_emits_subgraph_clusters_for_stage_template — verifies render_dot
   emits subgraph cluster blocks and fully-qualified stage.job node IDs for a
   stage-template fixture, covering the PipelineBodySummary::Stages branch
   that was previously exercised only indirectly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine
jamesadevine marked this pull request as ready for review July 21, 2026 13:44
@jamesadevine
jamesadevine merged commit 708f1bf into main Jul 21, 2026
2 of 4 checks passed
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