Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DependencyInjectionEventSourceTests.EmitsDynamicMethodBuiltEvent test failure on Windows in CI #35753

Open
safern opened this issue May 2, 2020 · 2 comments · May be fixed by #107465
Open
Labels
area-Extensions-DependencyInjection disabled-test The test is disabled in source code against the issue in-pr There is an active PR which will close this issue when it is merged
Milestone

Comments

@safern
Copy link
Member

safern commented May 2, 2020

Builds

Build Pull Request Test Failure Count
#626777 Rolling 1
#626849 #35712 1
#626950 Rolling 1

Configurations

  • netcoreapp5.0-Windows_NT-Debug-x64-Mono_release-Windows.10.Amd64.Server19H1.ES.Open
  • netcoreapp5.0-Windows_NT-Release-x64-Mono_release-Windows.10.Amd64.Server19H1.Open
  • netcoreapp5.0-Windows_NT-Release-x86-CoreCLR_release-Windows.10.Amd64.ServerRS5.Open

Helix Logs

Build Pull Request Console Core Test Results Run Client
#626777 Rolling console.log testResults.xml run_client.py
#626849 #35712 console.log testResults.xml run_client.py
#626950 Rolling console.log testResults.xml run_client.py
 Microsoft.Extensions.DependencyInjection.DependencyInjectionEventSourceTests.EmitsDynamicMethodBuiltEvent [FAIL]
      System.InvalidOperationException : Sequence contains more than one matching element
      Stack Trace:
        /_/src/libraries/System.Linq/src/System/Linq/ThrowHelper.cs(22,0): at System.Linq.ThrowHelper.ThrowMoreThanOneMatchException()
        /_/src/libraries/System.Linq/src/System/Linq/Single.cs(74,0): at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source, Func`2 predicate)
        /_/src/libraries/Microsoft.Extensions.DependencyInjection/tests/DependencyInjectionEventSourceTests.cs(206,0): at Microsoft.Extensions.DependencyInjection.DependencyInjectionEventSourceTests.EmitsDynamicMethodBuiltEvent()

Will disable the test in the meantime.

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this issue. Please help me learn by adding exactly one area label.

@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added the untriaged New issue has not been triaged by the area owner label May 2, 2020
@safern safern added blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' area-Microsoft.Extensions labels May 2, 2020
@safern
Copy link
Member Author

safern commented May 2, 2020

Disabling the test in #35690

@safern safern added disabled-test The test is disabled in source code against the issue and removed blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' labels May 5, 2020
@maryamariyan maryamariyan removed the untriaged New issue has not been triaged by the area owner label Jun 22, 2020
@maryamariyan maryamariyan added this to the Future milestone Jun 22, 2020
koenigst added a commit to koenigst/runtime that referenced this issue Sep 6, 2024
koenigst added a commit to koenigst/runtime that referenced this issue Sep 6, 2024
* Filtered events in listener by the service provider hash.
This prevents events from other test classes to impact the event source tests.
* Reused the listener to improve performance.
koenigst added a commit to koenigst/runtime that referenced this issue Sep 6, 2024
* Filtered events in the listener by the service provider hash.
This prevents events from other test classes impacting the event source tests.
* Reused the listener to improve performance.

Fixes dotnet#35753
@dotnet-policy-service dotnet-policy-service bot added the in-pr There is an active PR which will close this issue when it is merged label Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Extensions-DependencyInjection disabled-test The test is disabled in source code against the issue in-pr There is an active PR which will close this issue when it is merged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants