Manuscript (Case [Issue]22180) File System - Add error margin to FileReferenceAttributeTests
Posted by
Pharo Issue Tracker on
Sep 28, 2018; 4:15pm
URL: https://forum.world.st/Manuscript-Case-Issue-22180-File-System-Add-error-margin-to-FileReferenceAttributeTests-tp5085516.html
Manuscript Notification
|
Hi Christophe,
Unfortunately I don't have an example to point to. It's happened a few times in my virtualbox windows environment, and once in the CI. The logs are a long gone.
My idea was to modify DiskFileAttributesTestsResources>>setUp to subtract 30 seconds from the beforeCreationTime and add 30 seconds to the afterCreationTime.
This would mean that any small variation in timestamps, up to 30 seconds, e.g. due to caching, would be ignored, but the main purpose of the tests, ensuring that the primitive is basically working and that timezones are being calculated correctly, are still being tested.
Cheers, Alistair
|
|
|
Priority: 3 – Must Fix
|
|
Status: Working On
|
|
Assigned to: Everyone
|
|
Milestone: Pharo7.0
|
Go to Case
|
|
|
A Manuscript case was edited by Alistair Grant.
Case ID: 22180
Title: Add error margin to FileReferenceAttributeTests
Status: Working On
Category: Bug
Project: File System
Area: 1. Pharo Image
Priority: 3 - Must Fix
Milestone: Pharo7.0: 9/2/2018 (Past)
Assigned To: Everyone
URL: https://pharo.manuscript.com/f/cases/22180
Hi Christophe,
Unfortunately I don't have an example to point to. It's happened a few times in my virtualbox windows environment, and once in the CI. The logs are a long gone.
My idea was to modify DiskFileAttributesTestsResources>>setUp to subtract 30 seconds from the beforeCreationTime and add 30 seconds to the afterCreationTime.
This would mean that any small variation in timestamps, up to 30 seconds, e.g. due to caching, would be ignored, but the main purpose of the tests, ensuring that the primitive is basically working and that timezones are being calculated correctly, are still being tested.
Cheers,
Alistair
You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.manuscript.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&command=view&ixBug=22180) from this case.
_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
https://lists.gforge.inria.fr/mailman/listinfo/pharo-bugtracker