DiffEngine icon indicating copy to clipboard operation
DiffEngine copied to clipboard

With Rider, when multiple tests fail, only two windows are shown (sequentially) and other tests permanently fail until IDE restart

Open cmeeren opened this issue 1 year ago • 12 comments

Minimal repro solution: TestProject.zip

  1. Open a single instance of Rider with the repro solution (do not open other solutions in other windows, to minimize confounders)
  2. Run all tests
  3. One diff window opens. Accept the changes and close.
  4. Another diff window opens. Accept the changes and close.
  5. Bug: Observe that no other diff windows open.
  6. Re-run all the tests
  7. Bug: Observe that two of the tests fail, but no diff windows open.
  8. Close Rider
  9. Bug: Observe that Rider immediately restarts, with the splash screen, and then a diff window shows. Accept the changes and close. It is possible this then repeats for the last "pending" diff window.

Using Rider 2024.1.2.

cmeeren avatar May 15 '24 07:05 cmeeren

can u try setting solution opening to "new window"

image

SimonCropp avatar Jun 25 '24 11:06 SimonCropp

if that doesnt work, i think u need to rais it with the jetbrains peeps.

SimonCropp avatar Jun 25 '24 11:06 SimonCropp

Are you sure "Reopen projects on startup" is relevant here? Only the last bug I described has anything to do with IDE restarts. The bugs described in steps 5 and 7 do not have anything to do with the IDE restarting.

cmeeren avatar Jun 25 '24 11:06 cmeeren

for me step 3 does not repro

One diff window opens. Accept the changes and close.

when i run all tests, all diff open.

Are you sure "Reopen projects on startup" is relevant here

no. i am guessing

given i cant repro, i will need you to debug into it to work out what is wrong

SimonCropp avatar Jun 25 '24 11:06 SimonCropp

or, alternatively, pick a different diff tool. My preference is Beyond Compare

SimonCropp avatar Jun 25 '24 11:06 SimonCropp

also, a long shot, but i did fix a on process launching in 15.4.4. can u try that version?

SimonCropp avatar Jun 26 '24 09:06 SimonCropp

also, a long shot, but i did fix a on process launching in 15.4.4. can u try that version?

Not sure how. I use this through the Verify Support plugin. So I guess that needs to be updated? Currently on version 2024.1.2 of that plugin, and I don't see an option of updating.

cmeeren avatar Jun 26 '24 13:06 cmeeren

I use this through the Verify Support plugin.

the launching of diffs during a test run is not handled by the plugin. you can get diffengine 15.4.4 by updating to Verify.Xunit 25.0.4

SimonCropp avatar Jun 27 '24 00:06 SimonCropp

Thank you for the clarification. I tried updating, but there was no change in behavior. I also see that the setting "New window" was already selected.

cmeeren avatar Jun 27 '24 05:06 cmeeren

given DiffEngine is just calling the rider command line i dont think there is much more i can do hear. I suggest you raise it with JetBrains

SimonCropp avatar Jun 27 '24 21:06 SimonCropp

perhaps @khalidabuhakmeh or @matkoch can help us

image

SimonCropp avatar Jun 28 '24 03:06 SimonCropp

Let me bubble this up to the right folks.

khalidabuhakmeh avatar Jun 28 '24 18:06 khalidabuhakmeh

This seems to have been fixed in Rider 2024.2 (at least 2024.2.1). Thanks a lot! 😁

cmeeren avatar Aug 26 '24 12:08 cmeeren