Skip to content

python-pytest-xdist: disable tests on darwin #25725

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

LnL7
Copy link
Member

@LnL7 LnL7 commented May 12, 2017

Motivation for this change

Not sure why the failure only happens on darwin (python3). http://hydra.nixos.org/build/52755528

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Sorry, something went wrong.

@LnL7 LnL7 added 6.topic: darwin Running or building packages on Darwin 6.topic: python labels May 12, 2017
@mention-bot
Copy link

@LnL7, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh to be a potential reviewer.


# Test fails for python3:
# dsession (unittest.loader._FailedTest) ImportError: cannot import name 'collect_one_node'
doCheck = !stdenv.isDarwin;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think what is needed is a

checkPhase = ''
  py.test
'';

but I'm not sure

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indeed, I did have to disable some of the tests

Verified

This commit was signed with the committer’s verified signature. The key has expired.
LnL7 Daiderd Jordan
@LnL7 LnL7 force-pushed the python-pytest-xdist branch from e5247ad to fac85d4 Compare May 12, 2017 17:47
'';

checkPhase = ''
py.test
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In pytest runner you can disable tests like this:

py.test -k "not test_foo and not test_bar"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's about 10 test cases in total, I didn't want to disable them all separately.

@FRidh
Copy link
Member

FRidh commented May 15, 2017

Pushed e5a4da2
Hopefully we now get a channel update again.

@FRidh FRidh closed this May 15, 2017
@FRidh
Copy link
Member

FRidh commented May 15, 2017

Hmm, I didn't check the build before pushing, but I'm getting failing tests on NixOS as well.
I've disabled tests on 2.x and pushed a version bump 37cb24a

@LnL7 LnL7 deleted the python-pytest-xdist branch May 16, 2017 17:22
@LnL7
Copy link
Member Author

LnL7 commented May 16, 2017

Strange, pretty sure I tested it on both platforms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: darwin Running or building packages on Darwin 6.topic: python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants