Skip to content

Commit d25bb1f

Browse files
authored
Fix Elasticsearch function test (#7547)
Fixes #7452
1 parent ac5199c commit d25bb1f

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

tests/Aspire.Elastic.Clients.Elasticsearch.Tests/AspireElasticClientExtensionsTest.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,9 +140,10 @@ public void CanAddClientFromEncodedConnectionString()
140140

141141
[Fact]
142142
[RequiresDocker]
143-
[ActiveIssue("https://github.com/dotnet/aspire/issues/7452")]
144143
public void ElasticsearchInstrumentationEndToEnd()
145144
{
145+
// RemoteExecutor is used because OTEL uses a static instance to capture activities
146+
146147
RemoteExecutor.Invoke(async (connectionString) =>
147148
{
148149
var builder = CreateBuilder(connectionString);
@@ -162,7 +163,8 @@ public void ElasticsearchInstrumentationEndToEnd()
162163
Assert.Single(activityList);
163164

164165
var activity = activityList[0];
165-
Assert.Equal("ping", activity.OperationName);
166+
Assert.Equal("ping", activity.DisplayName);
167+
Assert.Equal("HEAD", activity.OperationName);
166168
Assert.Contains(activity.Tags, kvp => kvp.Key == "db.system" && kvp.Value == "elasticsearch");
167169
}, DefaultConnectionString, new RemoteInvokeOptions { TimeOut = 120_000 }).Dispose();
168170
}

0 commit comments

Comments
 (0)