Using self.next_run.time() instead of self.at_time when "making sure we run at the specified time *today*"

[AnezeR]

An if-statement that subtracts a day from self.next_run was using self.at_time (a value not affected by timezone).
Thus, when applying timezone to self.next_run a day was incorrectly subtracted, like in #582 (comment) or, on the opposite, not subtracted, like in #579 (comment).
Tests for these issues are added as well.

[coveralls]

Coverage: 99.713%. Remained the same when pulling 97c4dd3 on AnezeR:patch-1 into a3c2074 on dbader:master.

[AnezeR]

py39: install_deps> python -I -m pip install mock mypy pytest pytest-cov types-pytz
  ERROR: HTTP error 502 while getting https://files.pythonhosted.org/packages/27/6f/fbb16f260a7107ee98871d83f6793d88c4311b7189b10ef239da0b2181db/mock-5.0.2-py3-none-any.whl (from https://pypi.org/simple/mock/) (requires-python:>=3.6)
ERROR: Could not install requirement mock from https://files.pythonhosted.org/packages/27/6f/fbb16f260a7107ee98871d83f6793d88c4311b7189b10ef239da0b2181db/mock-5.0.2-py3-none-any.whl because of HTTP error 502 Server Error: Gateway Error for url: https://files.pythonhosted.org/packages/27/6f/fbb16f260a7107ee98871d83f6793d88c4311b7189b10ef239da0b2181db/mock-5.0.2-py3-none-any.whl for URL https://files.pythonhosted.org/packages/27/6f/fbb16f260a7107ee98871d83f6793d88c4311b7189b10ef239da0b2181db/mock-5.0.2-py3-none-any.whl (from https://pypi.org/simple/mock/) (requires-python:>=3.6)

@SijmenHuizenga, is this why the tests are failing, or is there something with the code?

[yotamoron]

Hi,

Is there an ETA on this issue?

Thanks.

[orientalperil]

Can this get merged and released? This is a very serious bug.

[joshkoehler]

Would be great to get this in. Is there an estimate for when it will merge? Thanks!