Skip to content
This repository has been archived by the owner on Feb 21, 2023. It is now read-only.

Slightly improve tests execution time #643

Merged
merged 6 commits into from Sep 30, 2019
Merged

Slightly improve tests execution time #643

merged 6 commits into from Sep 30, 2019

Conversation

popravich
Copy link
Contributor

@popravich popravich commented Sep 26, 2019

What do these changes do?

Slightly improve tests running time

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • Add a new news fragment into the CHANGES/ folder

@popravich popravich changed the title Update 'writing tests' notes Slightly improve tests execution time Sep 26, 2019
@codecov
Copy link

codecov bot commented Sep 30, 2019

Codecov Report

Merging #643 into master will decrease coverage by 16.84%.
The diff coverage is 69.49%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #643       +/-   ##
=========================================
- Coverage   96.85%    80%   -16.85%     
=========================================
  Files          57     57               
  Lines        8551   8049      -502     
  Branches      578    570        -8     
=========================================
- Hits         8282   6440     -1842     
- Misses        192   1555     +1363     
+ Partials       77     54       -23
Impacted Files Coverage Δ
tests/_testutils.py 100% <ø> (+15.78%) ⬆️
tests/stream_commands_test.py 16.21% <ø> (-83.79%) ⬇️
tests/conftest.py 55.05% <0%> (-34.43%) ⬇️
tests/pubsub_receiver_test.py 98.82% <100%> (-0.04%) ⬇️
tests/pubsub_commands_test.py 96.46% <100%> (-3.54%) ⬇️
tests/generic_commands_test.py 85.02% <100%> (-14.98%) ⬇️
tests/pool_test.py 96.74% <100%> (-2.99%) ⬇️
tests/server_commands_test.py 92.09% <100%> (-7.91%) ⬇️
tests/sentinel_commands_test.py 18.82% <33.33%> (-81.18%) ⬇️
tests/sentinel_failover_test.py 12.8% <7.69%> (-75.06%) ⬇️
... and 43 more

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 bac9b42...01a2200. Read the comment docs.

@popravich popravich merged commit ecc9dc6 into master Sep 30, 2019
@popravich popravich deleted the tests_speedups branch September 30, 2019 12:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant