Skip to content

Fixtures #1765

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

Merged
merged 5 commits into from
Dec 29, 2016
Merged

Fixtures #1765

merged 5 commits into from
Dec 29, 2016

Conversation

djarecka
Copy link
Collaborator

  • created nipype/testing/fixtures.py for pytest fixtures that are used in multiple tests
  • moved various version of create_files_in_director fixture to nipype/testing/fixtures.py
  • changed to pytest tmpdir in the fixtures

@codecov-io
Copy link

Current coverage is 72.43% (diff: 76.61%)

Merging #1765 into master will decrease coverage by <.01%

@@             master      #1765   diff @@
==========================================
  Files          1055       1056     +1   
  Lines         52823      52713   -110   
  Methods           0          0          
  Messages          0          0          
  Branches       7672       7660    -12   
==========================================
- Hits          38266      38183    -83   
+ Misses        13345      13321    -24   
+ Partials       1212       1209     -3   

Powered by Codecov. Last update 2d4726f...e9dc5a7

@djarecka djarecka requested a review from satra December 29, 2016 04:42
@satra satra merged commit 29ba1b2 into nipy:master Dec 29, 2016
@djarecka djarecka deleted the fixtures branch December 29, 2016 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants