Codecov Report

Merging #951 into master will decrease coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #951      +/-   ##
==========================================
- Coverage    91.8%   91.69%   -0.12%     
==========================================
  Files         103      103
  Lines        4040     4046       +6
  Branches      638      639       +1
==========================================
+ Hits         3709     3710       +1     
- Misses        193      197       +4     
- Partials      138      139       +1

Impacted Files	Coverage Δ
telegram/utils/helpers.py	`82.35% <100%> (+2.35%)`	⬆️
telegram/ext/updater.py	`77.17% <100%> (+0.25%)`	⬆️
telegram/utils/request.py	`66.96% <0%> (-0.9%)`	⬇️
telegram/bot.py	`87.51% <0%> (-0.5%)`	⬇️


		# From https://stackoverflow.com/questions/2549939/get-signal-names-from-numbers-in-python
		_signames = {v: k
		for k, v in reversed(sorted(vars(signal).items()))

This makes tests fail though...

TODO: Fix warning about deprecated fixture declaration

		'telegram.ext.updater', 'INFO', 'Received signal {} (SIGTERM), '
		'stopping...'.format(signal.SIGTERM))]
		rec = caplog.records()[-1]
		assert rec.msg.startswith('Received signal {}'.format(signal.SIGTERM))

This was referenced

Mar 2, 2018

Closed

Merged