Problem/Motivation

Seems that the testShouldSend seems to fail every once and a while. From my tests it seemed to be two culprits, either the time zone used by DateTime or a logical mishap within the tests.

Steps to reproduce

This was originally seeming like a problem with time zones as it seems after reconstructing the DateTime from the state, the time zone was different.

Otherwise, it may be due to the 'next tuesday' which is producing the wrong date and causing a fail. Perhaps the solution is to just move that date to 'two weeks' or similar.

Remaining tasks

  • Discover if bug is reproducible regularly.
  • Discover route of bug, CronExpression, DateTime, Logical error.
  • Create and review patch.

Comments

DerekCresswell created an issue.

derekcresswell’s picture

This seems to be entirely unrelated, though it did appear at the same time, but the SubscriptionWidget test is also failing occasionally.

I've yet to see the test fail locally. It's throwing the error when checking to see if the digests description is shown on the page (Approx L107). This is just a random string and I wonder if that has to do with it but that shouldn't be the case.

For now, a re-test seems to do the trick.