Skip to content

test: pin verify_cpp_itfs_rebuilt freshness branches (PRPUNDIT-7) - #1322

Open
jiagaoxiang wants to merge 2 commits into
AMD-AGI:mainfrom
jiagaoxiang:testgap/PRPUNDIT-7
Open

test: pin verify_cpp_itfs_rebuilt freshness branches (PRPUNDIT-7)#1322
jiagaoxiang wants to merge 2 commits into
AMD-AGI:mainfrom
jiagaoxiang:testgap/PRPUNDIT-7

Conversation

@jiagaoxiang

Copy link
Copy Markdown

Summary

  • Adds unit coverage for verify_cpp_itfs_rebuilt KEEP/REVERT freshness branches: non-cpp_itfs skip, absent/empty build_dir, stale vs fresh lib.so, 1s mtime slack, and empty module_names glob fallback.
  • Tightens the empty build_dir guard so Path('') cannot glob the process CWD (reviewer yihanwan found this live hole).

Closes test gap PRPUNDIT-7.

Test plan

  • PYTHONPATH=src pytest src/hyperloom/agents/kernel/tests/test_apply_kernel_patch_isolated_aiter.py -k verify_cpp_itfs_rebuilt

Cover the KEEP/REVERT cpp_itfs rebuild gate, including empty build_dir, mtime slack, and module glob fallback.
Reject Path('') so verification cannot glob the process CWD.
@jiagaoxiang
jiagaoxiang requested a review from a team as a code owner August 28, 2026 07:13
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