Skip to content

Commit

Permalink
Update unit tests
Browse files Browse the repository at this point in the history
  • Loading branch information
tippmar-nr committed Jul 17, 2023
1 parent c9a1bc3 commit d336e16
Showing 1 changed file with 51 additions and 51 deletions.
102 changes: 51 additions & 51 deletions tests/Agent/UnitTests/CompositeTests/Samplers/GcSamplerNetCoreTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -75,75 +75,75 @@ public void SamplerStartsWithoutException()
sampler.Start();
}

[Test]
public void SamplerDisposesEventListenerOnException()
{
var samplerWasStopped = false;
var listenerWasDisposed = false;
//[Test]
//public void SamplerDisposesEventListenerOnException()
//{
// var samplerWasStopped = false;
// var listenerWasDisposed = false;

var mockListener = Mock.Create<ISampledEventListener<Dictionary<GCSampleType, float>>>();
// var mockListener = Mock.Create<ISampledEventListener<Dictionary<GCSampleType, float>>>();

Mock.Arrange(() => mockListener.Dispose())
.DoInstead(() => { listenerWasDisposed = true; });
// Mock.Arrange(() => mockListener.Dispose())
// .DoInstead(() => { listenerWasDisposed = true; });

//This is our mechanism for shutting down the sampler. If a config change is used, it starts/stops 3x which makes
//it difficult to determine current state. Instead, throw an exception in the eventListener's sample method.
Mock.Arrange(() => mockListener.Sample())
.DoInstead(() => throw new Exception());
// //This is our mechanism for shutting down the sampler. If a config change is used, it starts/stops 3x which makes
// //it difficult to determine current state. Instead, throw an exception in the eventListener's sample method.
// Mock.Arrange(() => mockListener.Sample())
// .DoInstead(() => throw new Exception());

Func<ISampledEventListener<Dictionary<GCSampleType, float>>> mockListenerFactory = () =>
{
return mockListener;
};
// Func<ISampledEventListener<Dictionary<GCSampleType, float>>> mockListenerFactory = () =>
// {
// return mockListener;
// };

var mockScheduler = Mock.Create<IScheduler>();
// var mockScheduler = Mock.Create<IScheduler>();

//Prevents the scheduler from actually running
Mock.Arrange(() => _mockScheduler.ExecuteEvery(Arg.IsAny<Action>(), Arg.IsAny<TimeSpan>(), Arg.IsAny<TimeSpan?>()))
.DoNothing();
// //Prevents the scheduler from actually running
// Mock.Arrange(() => _mockScheduler.ExecuteEvery(Arg.IsAny<Action>(), Arg.IsAny<TimeSpan>(), Arg.IsAny<TimeSpan?>()))
// .DoNothing();

//Tracks the stop executing for the scheduler which indicates that the sampler
//has requested it to stop;
Mock.Arrange(() => mockScheduler.StopExecuting(Arg.IsAny<Action>(), Arg.IsAny<TimeSpan?>()))
.DoInstead<Action, TimeSpan?>((a, t) => { samplerWasStopped = true; });
// //Tracks the stop executing for the scheduler which indicates that the sampler
// //has requested it to stop;
// Mock.Arrange(() => mockScheduler.StopExecuting(Arg.IsAny<Action>(), Arg.IsAny<TimeSpan?>()))
// .DoInstead<Action, TimeSpan?>((a, t) => { samplerWasStopped = true; });

var sampler = new GCSamplerNetCore(mockScheduler, mockListenerFactory, _mockTransformer, _fxSamplerValidForFrameworkOverride);
// var sampler = new GCSamplerNetCore(mockScheduler, mockListenerFactory, _mockTransformer, _fxSamplerValidForFrameworkOverride);

//Act
sampler.Start();
// //Act
// sampler.Start();

//Cause error which will shut down the sampler
sampler.Sample();
// //Cause error which will shut down the sampler
// sampler.Sample();

//Assert
Assert.IsTrue(samplerWasStopped);
Assert.IsTrue(listenerWasDisposed);
}
// //Assert
// Assert.IsTrue(samplerWasStopped);
// Assert.IsTrue(listenerWasDisposed);
//}

[Test]
public void SamplerDisposesEventListenerWhenDisposed()
{
var listenerWasDisposed = false;
//[Test]
//public void SamplerDisposesEventListenerWhenDisposed()
//{
// var listenerWasDisposed = false;

var mockListener = Mock.Create<ISampledEventListener<Dictionary<GCSampleType, float>>>();
// var mockListener = Mock.Create<ISampledEventListener<Dictionary<GCSampleType, float>>>();

Mock.Arrange(() => mockListener.Dispose())
.DoInstead(() => { listenerWasDisposed = true; });
// Mock.Arrange(() => mockListener.Dispose())
// .DoInstead(() => { listenerWasDisposed = true; });

Func<ISampledEventListener<Dictionary<GCSampleType, float>>> mockListenerFactory = () =>
{
return mockListener;
};
// Func<ISampledEventListener<Dictionary<GCSampleType, float>>> mockListenerFactory = () =>
// {
// return mockListener;
// };

var sampler = new GCSamplerNetCore(_mockScheduler, mockListenerFactory, _mockTransformer, _fxSamplerValidForFrameworkOverride);
// var sampler = new GCSamplerNetCore(_mockScheduler, mockListenerFactory, _mockTransformer, _fxSamplerValidForFrameworkOverride);

//Act
sampler.Start();
sampler.Dispose();
// //Act
// sampler.Start();
// sampler.Dispose();

//Assert
Assert.IsTrue(listenerWasDisposed);
}
// //Assert
// Assert.IsTrue(listenerWasDisposed);
//}

[Test]
public void SamplerStartsEventListenerWhenStarted()
Expand Down

0 comments on commit d336e16

Please sign in to comment.