reqnroll / Reqnroll.VisualStudio

Visual Studio extension for Reqnroll - open-source .NET BDD framework
https://reqnroll.net
BSD 3-Clause "New" or "Revised" License
15 stars 5 forks source link

Flaky CI test execution issues after package updates #40

Open gasparnagy opened 1 month ago

gasparnagy commented 1 month ago

Used Visual Studio

Visual Studio 2022

Are the latest Visual Studio updates installed?

Yes

Content of reqnroll.json (if present)

No response

Issue Description

When I worked on upgrading the packages locally, I also observed that after upgrading a few, some tests related to logging became flaky. At that time it seemed that the problem is with the ApprovalTests dependency (5.9.0 -> 6.0.0), so I left it as v5.9.0.

Now after the package upgrades are re-applied, the builds fail again with the same errors. (So probably was nothing to do with ApprovalTests.) This build I have even tried 3 times, all 3 failed.

The affected test class seems to be the Diagnostics.LoggingTests class, that is NOT an approval test. I reviewed the tests, but haven't found anything bad. They use a mock file system, so I don't understand how this problem is caused. But it seems we need to investigate this further or if nothing happens rewrite these tests with some other (?) method.

Steps to Reproduce

run CI or unit tests locally

Link to a project repository that reproduces the issue

No response

UL-ChrisGlew commented 1 month ago

Just to summarize the discussion @gasparnagy and I were having over at #39

It doesn't look like some of the unit test flakiness present is as a result of the package updates that occured recently in the extension - I am seeing some slight flakiness in the test executions before the package updates went through (commit #e221b):

image

gasparnagy commented 17 hours ago

Examples from CI: