Skip to content
Snippets Groups Projects

fix: test running on macos

Merged Yaroslav Dynnikov requested to merge pytest-port-range into master
All threads resolved!

Pytest supports running tests in parallel using the xdist plugin. In order to support it Picodata should avoid port collision. It uses dedicated IP address 127.7.n.1 where n = xdist_worker_number. Unfortunately, it doesn't work on MacOS, because it doesn't provide any loopback aliases but original 127.0.0.1.

This patch provides another address generation logics. The subnet parameter is superseeded with a base_port, that is 3300 + n * 100. In this way, every pytest (xdist) worker gets a dedicated port range [3301, 3399], [3401, 3499] and so on.

Close #65 (closed)

Merge request reports

Merge request pipeline #5004 passed

Merge request pipeline passed for 8752aef7

Approval is optional

Set by to be merged automatically when the pipeline succeeds

Ready to merge by members who can write to the target branch.

Merge details

  • 1 commit will be added to master.
  • Source branch will be deleted.
  • Auto-merge enabled

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • Sergey V
  • Developer

    Тесты проходят, падает только process management.

    pgrp = 50557
    
        @funcy.retry(tries=10, timeout=0.01, errors=StillAlive)
        def waitpg(pgrp):
            try:
    >           os.killpg(pgrp, 0)
    E           PermissionError: [Errno 1] Operation not permitted
    
    test/int/test_basics.py:141: PermissionError
  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • mentioned in issue #70 (closed)

  • added 5 commits

    Compare with previous version

  • Yaroslav Dynnikov resolved all threads

    resolved all threads

  • mentioned in issue #210 (closed)

  • Please register or sign in to reply
    Loading