Failing manual test for ScheduledFireTimeUtc #376
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The test fails with the following output: (Test started at 09:35 UTC) (Computer put to sleep at 09:36 UTC) (Trigger should have fired at 09:37 UTC but could not because the computer was in sleep) (Computer woken up at 09:41 UTC) Event Scheduled for 03/06/2016 09:41:37 +00:00 Ticked at 03/06/2016 09:41:37 +00:00 with Difference of 00:00:00.0260000 Scheduled Fire Time Wrong - It Should Have Been Earlier Because of the Misfire Expected: greater than 00:00:30 But was: 00:00:00.0260000 A fixed job was needed to implement this test, so a fixed job factory was also added, which can be used in any circumstance when only one job is executed for all triggers. In order to ask the tester to carry out the manual part of this test at the right time a MessageBox is used, which is why the build script is modified a bit as well to include a Windows Forms reference.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ScheduledFireTimeUtc likes to stay in the present even though it should sometimes reflect the past, as detailed in this issue #375