Skip to content

Commit 5dafb36

Browse files
[Infra] Fix flaky tests (#3369)
1 parent a6963eb commit 5dafb36

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

test/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests/EntityFrameworkIntegrationTests.cs

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -162,25 +162,26 @@ bool ActivityFilter(string? providerName, IDbCommand command)
162162
using var scope = SemanticConventionScope.Get(useNewConventions);
163163

164164
var activities = new List<Activity>();
165-
using var tracerProvider = Sdk.CreateTracerProviderBuilder()
165+
using (Sdk.CreateTracerProviderBuilder()
166166
.AddInMemoryExporter(activities)
167167
.AddSqlClientInstrumentation()
168168
.AddEntityFrameworkCoreInstrumentation(options => options.Filter = ActivityFilter)
169-
.Build();
170-
171-
var optionsBuilder = new DbContextOptionsBuilder<ItemsContext>();
169+
.Build())
170+
{
171+
var optionsBuilder = new DbContextOptionsBuilder<ItemsContext>();
172172

173-
this.ConfigureProvider(provider, optionsBuilder);
173+
this.ConfigureProvider(provider, optionsBuilder);
174174

175-
await using var context = new ItemsContext(optionsBuilder.Options);
175+
await using var context = new ItemsContext(optionsBuilder.Options);
176176

177-
try
178-
{
179-
await context.Database.ExecuteSqlRawAsync(commandText);
180-
}
181-
catch
182-
{
183-
// Ignore
177+
try
178+
{
179+
await context.Database.ExecuteSqlRawAsync(commandText);
180+
}
181+
catch
182+
{
183+
// Ignore
184+
}
184185
}
185186

186187
Assert.NotEmpty(activities);

test/OpenTelemetry.Instrumentation.EventCounters.Tests/EventCountersMetricsTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ private static double GetActualValue(Metric metric)
220220
Thread.Sleep(100);
221221
meterProvider.ForceFlush();
222222
metric = exportedItems.FirstOrDefault(x => x.Name == expectedInstrumentName);
223-
return metric != null;
223+
return metric != null && Math.Abs(GetActualValue(metric)) > double.Epsilon;
224224
},
225225
10_000);
226226

0 commit comments

Comments
 (0)