Skip to content

Make close an async def for FakePairing testing#32

Merged
Jc2k merged 1 commit intoJc2k:mainfrom
bdraco:fix_close_testing
May 23, 2021
Merged

Make close an async def for FakePairing testing#32
Jc2k merged 1 commit intoJc2k:mainfrom
bdraco:fix_close_testing

Conversation

@bdraco
Copy link
Copy Markdown
Collaborator

@bdraco bdraco commented May 23, 2021

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 23, 2021

Codecov Report

Merging #32 (5bb4e35) into main (82a5d9d) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   85.37%   85.28%   -0.10%     
==========================================
  Files          45       45              
  Lines        3303     3303              
==========================================
- Hits         2820     2817       -3     
- Misses        483      486       +3     
Flag Coverage Δ
unittests 85.28% <100.00%> (-0.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
aiohomekit/testing.py 86.48% <100.00%> (ø)
aiohomekit/controller/ip/connection.py 82.98% <0.00%> (-1.05%) ⬇️

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 82a5d9d...5bb4e35. Read the comment docs.

@bdraco
Copy link
Copy Markdown
Collaborator Author

bdraco commented May 23, 2021

I think the windows test here is flakey.

I'll amend and repush to make sure its just a transient failure

@bdraco bdraco force-pushed the fix_close_testing branch from 5ef28cc to 5bb4e35 Compare May 23, 2021 14:58
@Jc2k
Copy link
Copy Markdown
Owner

Jc2k commented May 23, 2021

Yes there is a flakey test. It flakes most on Windows, but I have seen it flake on them all 😭

@Jc2k Jc2k merged commit 815cca1 into Jc2k:main May 23, 2021
@bdraco bdraco deleted the fix_close_testing branch May 23, 2021 17:49
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.

2 participants