Skip to content
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

Fix tests via adding next(cg) #28

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

brookeferber
Copy link
Contributor

No description provided.

.travis.yml Outdated
@@ -57,6 +57,7 @@ install:
- source activate testenv
- 'pip install https://github.com/NSLS-II/portable-mds/zipball/master#egg=portable_mds'
- 'pip install https://github.com/NSLS-II/portable-fs/zipball/master#egg=portable_fs'
'pip install https://github.com/NSLS-II/bluesky/zipball/master#egg=bluesky'
Copy link
Member

Choose a reason for hiding this comment

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

You also need to remove bluesky fro the conda install section too

@codecov-io
Copy link

codecov-io commented Jul 25, 2017

Codecov Report

Merging #28 into master will increase coverage by 4.32%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   93.54%   97.87%   +4.32%     
==========================================
  Files           6        6              
  Lines         155      141      -14     
==========================================
- Hits          145      138       -7     
+ Misses         10        3       -7
Impacted Files Coverage Δ
xpdsim/tests/conftest.py 100% <ø> (+7.14%) ⬆️
xpdsim/dets.py 95.08% <100%> (+6.19%) ⬆️
xpdsim/tests/test_dets.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update adfa6d7...299698a. Read the comment docs.

@CJ-Wright
Copy link
Member

Can you please add # pragma: no cover to the try: block in conftest.py?

@CJ-Wright
Copy link
Member

I take that back, can you please add except ImportError: to the .coveragerc under exclude?
Like this

@brookeferber
Copy link
Contributor Author

brookeferber commented Jul 25, 2017

@CJ-Wright should I add # pragma: no cover to the except clause in conftest.py ?

@CJ-Wright
Copy link
Member

Yes please
If you need to you may also need to add that to the pass line as well

@CJ-Wright CJ-Wright changed the title Robot with Mediator Fix tests via adding next(cg) Jul 25, 2017
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