dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.16k stars 4.71k forks source link

[wasm] MeterDisposeTest is failing #93754

Open lewing opened 1 year ago

lewing commented 1 year ago

Starting in #93337 MeterDisposeTest started failing on wasm

[13:39:53] info: [2023-10-19T13:39:53.690Z] [FAIL] Microsoft.Extensions.Diagnostics.Metrics.Tests.DefaultMetricsFactoryTests.MeterDisposeTest
[13:39:53] info: Assert.Equal() Failure: Values differ
[13:39:53] info: Expected: 10
[13:39:53] info: Actual:   0
[13:39:53] info:    at Microsoft.Extensions.Diagnostics.Metrics.Tests.DefaultMetricsFactoryTests.MeterDisposeTest()
[13:39:53] info:    at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
[13:39:53] info:    at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)
ghost commented 1 year ago

Tagging subscribers to this area: @brzvlad, @kotlarmilos See info in area-owners.md if you want to be subscribed.

Issue Details
Starting in #93337 MeterDisposeTest started failing on wasm ``` [13:39:53] info: [2023-10-19T13:39:53.690Z] [FAIL] Microsoft.Extensions.Diagnostics.Metrics.Tests.DefaultMetricsFactoryTests.MeterDisposeTest [13:39:53] info: Assert.Equal() Failure: Values differ [13:39:53] info: Expected: 10 [13:39:53] info: Actual: 0 [13:39:53] info: at Microsoft.Extensions.Diagnostics.Metrics.Tests.DefaultMetricsFactoryTests.MeterDisposeTest() [13:39:53] info: at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args) [13:39:53] info: at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr) ```
Author: lewing
Assignees: -
Labels: `arch-wasm`, `untriaged`, `area-Codegen-Interpreter-mono`
Milestone: -
SamMonoRT commented 12 months ago

@lewing @radical - I'm not sure who the right owner is, but I assigned @radical for tracking. Reach out to Vlad if indeed an interpreter issue (I don't think it is though). I also added disabled-test label, and removed the untriaged label, setting it to 9.0.0

lewing commented 2 months ago

@mkhamoyan please check if the test is still failing and update the status here

mkhamoyan commented 2 months ago

@lewing I have checked, and the test is still failing.