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

Roslyn.VisualStudio.NewIntegrationTests.CSharp.CSharpSourceGenerators.InvokeNavigateToForGeneratedFile is Flaky #60477

Closed
jmarolf opened this issue Mar 30, 2022 · 4 comments
Assignees
Milestone

Comments

@jmarolf
Copy link
Contributor

jmarolf commented Mar 30, 2022

Build Information

Build: https://dev.azure.com/dnceng/9ee6d478-d288-47f7-aacc-f6e6d082ae6d/_build/results?buildId=1689247
Build error leg: VS_Integration_Debug_64 / Build and Test

Error Message

Exception Message

Assert.Equal() Failure
          ↓ (pos 0)
Expected: HelloWorld.cs [generated]
Actual:   Program.cs ⬤
          ↑ (pos 0)

Runfo Tracking Issue: CSharpSourceGenerators.InvokeNavigateToForGeneratedFile is Flaky

Build Definition Kind Run Name

Build Result Summary

Day Hit Count Week Hit Count Month Hit Count
0 0 0
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Mar 30, 2022
@jinujoseph jinujoseph added Test Test failures in roslyn-CI Flaky and removed untriaged Issues and PRs which have not yet been triaged by a lead labels Mar 30, 2022
@jinujoseph jinujoseph added this to the 17.3 milestone Mar 30, 2022
@jinujoseph
Copy link
Contributor

cc @sharwell for info

333fred added a commit to 333fred/roslyn that referenced this issue Jul 25, 2022
Collectively, these tests have failed 60 times in the past week. Runfo bugs:

dotnet#57423
dotnet#60477
dotnet#61367
333fred added a commit that referenced this issue Jul 25, 2022
Collectively, these tests have failed 60 times in the past week. Runfo bugs:

#57423
#60477
#61367
@jasonmalinowski jasonmalinowski modified the milestones: 17.3, 17.4 Aug 26, 2022
@jasonmalinowski
Copy link
Member

@JoeRobich Any chance your other fixes for Navigate To might have addressed this?

@JoeRobich
Copy link
Member

I did update this test as part of the NavigateTo fixes. Opened a PR to see.

@jasonmalinowski
Copy link
Member

We merged #64626 so we'll see if this is better now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants