Fix setsockopt signature to match standard socket API

[Copilot]

MocketSocket.setsockopt was using (family, type, proto) — socket creation parameters — instead of the standard socket.setsockopt(level, optname, value[, optlen]) signature, making it a non-conforming mock.

Changes

• Signature fix: setsockopt(family, type, proto) → setsockopt(level, optname, value, optlen=None)
• Body fix: Removed incorrect storage of family/type/proto as instance attributes; now forwards to _true_socket with the correct arguments, handling both 3- and 4-argument call variants
• Docstring: Updated to reflect correct parameter semantics
• Tests: Added test_setsockopt_without_optlen and test_setsockopt_with_optlen to tests/test_socket.py to cover both forwarding paths

# Before (wrong — socket creation params, not socket options)
def setsockopt(self, family: int, type: int, proto: int) -> None:
    self._family = family
    self._type = type
    self._proto = proto

# After (matches standard socket.setsockopt API)
def setsockopt(self, level: int, optname: int, value: int | bytes | None, optlen: int | None = None) -> None:
    if self._true_socket:
        if optlen is not None:
            self._true_socket.setsockopt(level, optname, value, optlen)
        else:
            self._true_socket.setsockopt(level, optname, value)

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• httpbin.local
  • Triggering command: /usr/bin/python python -m pytest tests/test_http.py -q -p no:cacheprovider --no-header (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.06%. Comparing base (b74910a) to head (bab96a1).
⚠️ Report is 1 commits behind head on typehints-and-docstrings.

Additional details and impacted files

@@                     Coverage Diff                      @@
##           typehints-and-docstrings     #319      +/-   ##
============================================================
- Coverage                     99.06%   99.06%   -0.01%     
============================================================
  Files                            22       22              
  Lines                          1074     1073       -1     
============================================================
- Hits                           1064     1063       -1     
  Misses                           10       10              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[mindflayer]

@copilot add a test that covers the new line for when optlen is not None

[Copilot]

@copilot add a test that covers the new line for when optlen is not None

Added in commit bab96a1:

• test_setsockopt_without_optlen — covers the 3-argument path
• test_setsockopt_with_optlen — covers the optlen is not None branch using a realistic SO_LINGER struct value

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud