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

Add test to reproduce non-removed IPC socket file #293

Closed
wants to merge 2 commits into from

Conversation

Peque
Copy link
Member

@Peque Peque commented Apr 6, 2018

Fixes #287.

WIP: still only reproducing the error. No fix.

@Peque Peque added the bug label Apr 6, 2018
@Peque Peque added this to the 0.7.0 milestone Apr 6, 2018
@Peque Peque self-assigned this Apr 6, 2018
@Peque Peque added wip and removed bug labels Apr 6, 2018
@codecov
Copy link

codecov bot commented Apr 6, 2018

Codecov Report

Merging #293 into master will decrease coverage by 0.13%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #293      +/-   ##
==========================================
- Coverage    99.1%   98.96%   -0.14%     
==========================================
  Files          26       26              
  Lines        3556     3572      +16     
  Branches      258      258              
==========================================
+ Hits         3524     3535      +11     
- Misses         19       22       +3     
- Partials       13       15       +2
Impacted Files Coverage Δ
osbrain/agent.py 97.71% <100%> (ø) ⬆️
osbrain/tests/test_bugs.py 92.3% <100%> (+8.97%) ⬆️
osbrain/__init__.py 91.66% <0%> (-8.34%) ⬇️
osbrain/tests/test_agent_transport.py 93.97% <0%> (-2.41%) ⬇️

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 9fc27f2...cc235ad. Read the comment docs.

@Peque
Copy link
Member Author

Peque commented Apr 6, 2018

Closing in favor of #294.

@Peque Peque closed this Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Socket file removal tests randomly failing on Travis
1 participant