corvus-dotnet / corvus-dotnet/Corvus.Monitoring

Test may be presuming too much about AI operation IDs

Open
#15 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
5
Forks
0
PR merge metrics
No merged PRs in 30d

Description

When building with Application Insights 2.11, we get these errors:

```
Expected string length 55 but was 51. Strings differ at index 0.
Expected: "00-a080bab656ea934a9728001ad5135437-80505fb4142fb048-00"
But was: "|a080bab656ea934a9728001ad5135437.80505fb4142fb048."
```

This implies there has been some change in the ID format. The intention was for the tests to have no preconceived notions about how these IDs are structured, and just to check that things like parent/child relationships reflect the expected structure. So there's probably something not quite right in these tests.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.