python3Packages.mocket: disable test_httprettish_httpx_session on darwin

[devusb]

Resolves #400373

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[devusb]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 400644

---

aarch64-darwin

⏩ 10 packages marked as broken and skipped:

• froide
• froide-govplan
• froide-govplan.dist
• froide.dist
• python312Packages.froide
• python312Packages.froide.dist
• python312Packages.parsedmarc
• python312Packages.parsedmarc.dist
• python313Packages.parsedmarc
• python313Packages.parsedmarc.dist

✅ 54 packages built:

• ansible (ansible_2_18, python312Packages.ansible-core)
• ansible-later
• ansible-later.dist
• ansible-lint
• ansible-lint.dist
• ansible-navigator
• ansible-navigator.dist
• ansible.dist (ansible_2_18.dist, python312Packages.ansible-core.dist)
• ansible_2_16
• ansible_2_16.dist
• ansible_2_17
• ansible_2_17.dist
• molecule (python312Packages.molecule)
• molecule.dist (python312Packages.molecule.dist)
• napalm
• napalm.dist
• parsedmarc
• parsedmarc.dist
• python312Packages.ansible
• python312Packages.ansible-compat
• python312Packages.ansible-compat.dist
• python312Packages.ansible-kernel
• python312Packages.ansible-kernel.dist
• python312Packages.ansible-runner
• python312Packages.ansible-runner.dist
• python312Packages.ansible-vault-rw
• python312Packages.ansible-vault-rw.dist
• python312Packages.ansible.dist
• python312Packages.geoip2
• python312Packages.geoip2.dist
• python312Packages.mocket
• python312Packages.mocket.dist
• python312Packages.msticpy
• python312Packages.msticpy.dist
• python312Packages.napalm
• python312Packages.napalm-hp-procurve
• python312Packages.napalm-hp-procurve.dist
• python312Packages.napalm.dist
• python312Packages.pytest-ansible
• python312Packages.pytest-ansible.dist
• python312Packages.pytest-testinfra
• python312Packages.pytest-testinfra.dist
• python312Packages.sopel
• python312Packages.sopel.dist
• ttp (python312Packages.ttp)
• ttp.dist (python312Packages.ttp.dist)
• python313Packages.geoip2
• python313Packages.geoip2.dist
• python313Packages.mocket
• python313Packages.mocket.dist
• python313Packages.msticpy
• python313Packages.msticpy.dist
• python313Packages.sopel
• python313Packages.sopel.dist

[mayersj1]

Is there a known issue with this test on aarch64-darwin, or are you just commenting out the test because it's failing?

[devusb]

@mayersj1 I spent maybe 20 minutes looking for an upstream issue and trying to see if there was an obvious reason why it was failing, but didn't find anything conclusive. figured unbreaking all dependent packages was worth just disabling the test, but happy to do something different if you're able to find a way to fix the actual test -- would definitely be preferable.

[XYenon]

This test failed on x86_64-darwin even without nix.

[freddydumont]

is there anything blocking a merge for this PR?

[devusb]

not that I’m aware of.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/2355

[devusb]

actually, I take it back -- didn't think through all the rebuilds fixing __darwinAllowLocalNetworking syntax would cause.

I re-limited this PR to only skipping the test -- the other fix will likely need to target staging.

[booxter]

I confirmed the test passed in previous version. Requesting a revert for the bump while we explore what broke it in upstream: #402824

[booxter]

The revert will have to go through staging due to large number of linux rebuilds. This is a good stop-gap until we figure it out to unblock important packages for darwin like ansible.

[dezren39]

any chance we can merge this?

[booxter]

@NixOS/darwin-maintainers

[booxter]

ZHF: #403336

[booxter]

FYI this was fixed upstream with: mindflayer/python-mocket@937e625 so we can revert this PR on the next bump.