Skip to content

test(dependency-sources): verify configured scan budget behavior - #1

Merged
Harbor404 merged 1 commit into
Harbor404:fix/dependency-source-analysis-budgetfrom
agentsope:test/696-dependency-budget-behavior
Oct 1, 2026
Merged

Harbor404 merged 1 commit into
Harbor404:fix/dependency-source-analysis-budgetfrom
agentsope:test/696-dependency-budget-behavior

Conversation

@agentsope

Copy link
Copy Markdown

Add 10 simulated-clock regression cases for the runtime behavior of the configurable dependency-source ceiling in NVIDIA/SkillSpector#705.

The tests verify that raising the ceiling allows the same scan to finish beyond the historical 5-second limit, while shorter caller deadlines, exhausted budgets, retained findings, and partial analyzer status remain respected. They also cover a slow parse that emits no findings. No production code changes are included.

This PR targets your fix/dependency-source-analysis-budget branch so the tests can be incorporated into NVIDIA#705. The single signed-off commit can also be cherry-picked: 0c1f3bcf85082e67f3f13f19b05ef1c4de5f7b97.

Validation:

  • 357 related tests passed, including the new behavior cases, your configuration tests, dependency-source regressions, and source detection tests.
  • Repository-wide Ruff lint and format checks passed.
  • In separate processes, restoring a fixed 5-second ceiling caused 3 selected tests to fail; ignoring the caller deadline caused 4 selected tests to fail. These mutations were simulated in memory and were not committed.

The tests use a private clock and synthetic .invalid registry destinations; they do not sleep or contact registries.

Supplement PR NVIDIA#705 with simulated-clock scan and node regressions for raised ceilings, caller deadlines, retained findings, and incomplete analysis.

Signed-off-by: Whj9283 <[email protected]>
@Harbor404
Harbor404 merged commit 0c1f3bc into Harbor404:fix/dependency-source-analysis-budget Oct 1, 2026
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.

2 participants