DEVX-138: Add IO_INTERNAL_CALLS to check_test_isolation #214

Merged
emil merged 1 commits from DEVX-138-io-internal-calls into master 2026-07-14 01:21:17 +00:00
Owner
No description provided.
emil reviewed 2026-07-14 01:19:46 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: WARNING — source files changed but no test files updated
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-14 01:20:03 +00:00
Dismissed
kireto left a comment
Member

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emil added 1 commit 2026-07-14 01:20:09 +00:00
feat: add IO_INTERNAL_CALLS to check_test_isolation
CI / validate (pull_request) Successful in 50s
CI / auto-merge (pull_request) Successful in 15s
8f6e5bc61c
Add a mapping of I/O function internal dependencies so the static
analyzer recognizes when a test patches an internal dependency (e.g.
get_tofu_output) instead of the I/O function itself (e.g.
get_customer_vm_ip). This eliminates false positives for tests that
properly isolate I/O by patching at the right level.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
emil force-pushed DEVX-138-io-internal-calls from a0e004f467 to 8f6e5bc61c 2026-07-14 01:20:09 +00:00 Compare
emil reviewed 2026-07-14 01:20:58 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: WARNING — source files changed but no test files updated
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-14 01:21:14 +00:00
kireto left a comment
Member

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emil merged commit 326eccfd2f into master 2026-07-14 01:21:17 +00:00
emil deleted branch DEVX-138-io-internal-calls 2026-07-14 01:21:17 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#214