search

jonsequitur / Its.Log

Get information out of your code at runtime to send it to log files, perf counters, consoles, services, sensors. Maximum flexibility and maintainability, minimum code.
Other
13 stars 10 forks source link

Telemetry_events_contain_timing is flaky #38

Closed PhillipPruett closed 8 years ago

PhillipPruett commented 8 years ago

the assertion in Telemetry_events_contain_timing fails every now and then due to the timing being slightly off.

PhillipPruett commented 8 years ago

Expected value to be between 1000 and 1100, but found 996. at FluentAssertions.Execution.LateBoundTestFramework.Throw(String message) in D:\Workspaces\FluentAssertions\Src\Shared\Execution\LateBoundTestFramework.cs:line 26

at FluentAssertions.Execution.AssertionScope.FailWith(String message, Object[] args) in D:\Workspaces\FluentAssertions\Src\Core\Execution\AssertionScope.cs:line 197

at FluentAssertions.Numeric.NumericAssertions`1.BeInRange(T minimumValue, T maximumValue, String because, Object[] reasonArgs) in D:\Workspaces\FluentAssertions\Src\Core\Numeric\NumericAssertions.cs:line 270

at Its.Log.Instrumentation.UnitTests.TelemetryTests.d__7.MoveNext()

at NUnit.Framework.AsyncInvocationRegion.AsyncTaskInvocationRegion.WaitForPendingOperationsToComplete(Object invocationResult)

at NUnit.Core.NUnitAsyncTestMethod.RunTestMethod()

PhillipPruett commented 8 years ago

blocking my appveyor build so i went ahead and made its assertion a bit less specific. hopefully that settles it down.

PhillipPruett commented 8 years ago

fixed