test : add unit tests for _unshare_net_supported in parser_sandbox

Đang mở
  • Có gì không ổn?
#2,444 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
35/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
testing

Hướng nghiên cứu

Read backend/secuscan/parser_sandbox.py to inspect _unshare_net_supported and its cache globals, then run the pytest command given in the issue. Add testing/backend/unit/test_parser_sandbox_unshare_supported.py with the four mocked platform and probe cases, resetting the cache around each test. Done means all four paths pass in isolation; the issue says tmdeveloper007 is handling it, so confirm availability before starting.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Summary of What Needs to be Done:
Add unit tests for the _unshare_net_supported function in backend/secuscan/parser_sandbox.py. This function probes whether the unshare --user --net system call works on the host, and caches the result to avoid repeated probes.

The function has several distinct code paths that should each be tested:

  1. Non-Linux platform returns False immediately
  2. unshare binary not found returns False
  3. unshare binary found but probe fails (non-zero exit) returns False
  4. unshare binary found and probe succeeds (exit 0) returns True

Changes that Need to be Made:
Create testing/backend/unit/test_parser_sandbox_unshare_supported.py:

  • Test non-Linux path by patching platform.system() to return "Darwin"
  • Test unshare binary not found by patching shutil.which to return None
  • Test probe failure by patching subprocess.run to return a non-zero returncode
  • Test probe success by patching subprocess.run to return zero returncode
  • The function uses module-level caching (_unshare_capability_checked, _unshare_available): each test case needs to reset these globals before and after to ensure test isolation
  • Import the real production function directly: from backend.secuscan.parser_sandbox import _unshare_net_supported
  • Run with python3 -m pytest testing/backend/unit/test_parser_sandbox_unshare_supported.py -v --noconftest

Impact that it would Provide:

  • Tests a code path (capability detection) that is currently completely untested
  • Prevents regressions when refactoring the unshare support detection logic
  • Ensures the function behaves correctly on different platforms and when the unshare binary is unavailable
  • Uses standard unittest.mock patching, no heavy framework dependencies

Note: This task is being handled by tmdeveloper007 — please assign to that account when picking it up.

Ngôn ngữ chính
Python
Star
36
Fork
234
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của utksh1/SecuScan

Tất cả issue của utksh1/SecuScan

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.