Add a refresh button on the dialog box

[mustafaarif109]

Installed product versions

• Visual Studio: [example 2015 Professional]
• This extension: [example 1.1.21]

Description

I want to suggest the team to add a refresh button on the dialog box as it is sometimes not very quick and causes confusion.

Steps to recreate

1. Add a new method to test coverage
2. Wait for the coverage to change

Current behavior

It will change but takes time and sometimes it won't until you restart visual studio

Expected behavior

There would be a refresh button on dialog box that once clicked will refresh the code coverage automatically and that would be faster and without any confusion

Side Notes

• It helps to attach a repro solution so i can test the fix
• It helps to attach the log (send it to my mailbox if you need todo so privately : fortunengwenya@gmail.com)
• It helps to attach a screen recoding video of steps to recreate (sometimes a video is worth a thousand words)
• Please consider giving this project a star and become one of the stagazers
  https://github.com/FortuneN/FineCodeCoverage/stargazers
• Please consider buying me a coffee
  https://paypal.me/FortuneNgwenya (Once) OR https://liberapay.com/FortuneN/donate (Recurring)
• Please consider rating and/or reviewing
  https://marketplace.visualstudio.com/items?itemName=FortuneNgwenya.FineCodeCoverage&ssr=false#review-details

[tonyhallett]

I want to suggest the team to add a refresh button on the dialog box as it is sometimes not very quick and causes confusion.

When you say confusion, do you mean that the gutter display is not updating and the stats do not account for any new methods ?

I think that what you are asking for is FCC to track dirty state so if you change the code in the sut the gutters are removed etc ?

[mustafaarif109]

Hi Tony Hope you are doing good. I mean to say that the gutter displays doesn't update at times even after code changes in a certain method or even after adding a new class. In such case we need to close and restart visual studio so I recommended a hard refresh button on the dialog box to reset gutters forcefully or trigger an even that updates the display. Thank you! Regards Mustafa Arif

…

On Thu, May 13, 2021, 3:13 PM Tony Hallett ***@***.***> wrote: I want to suggest the team to add a refresh button on the dialog box as it is sometimes not very quick and causes confusion. When you say confusion, do you mean that the gutter display is not updating and the stats do not account for any new methods ? I think that what you are asking for is FCC to track dirty state so if you change the code in the sut the gutters are removed etc ? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#132 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIKM33XCYJV7ZACHHJ6C2ILTNOQ43ANCNFSM43UBXIYQ> .

[tonyhallett]

You just highlighted an issue where the gutters and stats are not cleared when you run coverage.

This has been fixed by https://ci.appveyor.com/api/buildjobs/a414oyg4owqxa7b5/artifacts/FineCodeCoverage.vsix

Gutters and stats are relevant to your code as it was when you run your tests ( and coverage was initiated ).
After you have changed the code, the only way to get correct results is to run tests again from the test explorer.

I will add a button for now but in future will probably replace with file watching.

[mustafaarif109]

Hi Tony Thanks for the clarification Have a nice day! Happy Coding 😃 Regards Mustafa Arif

…

On Thu, May 13, 2021, 9:02 PM Tony Hallett ***@***.***> wrote: You just highlighted an issue where the gutters and stats are not cleared when you run coverage. This has been fixed by https://ci.appveyor.com/api/buildjobs/a414oyg4owqxa7b5/artifacts/FineCodeCoverage.vsix Gutters and stats are relevant to your code as it was when you run your tests ( and coverage was initiated ). After you have changed the code, the only way to get correct results is to run tests again from the test explorer. I will add a button for now but in future will probably replace with file watching. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#132 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIKM33WTG5PG7SBR4K4CVIDTNPZZ7ANCNFSM43UBXIYQ> .

[tonyhallett]

Updated vsix with a clear ui button
https://ci.appveyor.com/api/buildjobs/fpc9ps3xwtkx2rj3/artifacts/FineCodeCoverage.vsix

This will be on the marketplace in the next couple of days.

[mustafaarif109]

Hi Tony Hope you are fine! Is there a delay in the release of this version on the marketplace as the website is still showing the old version. Thank you! Regards Mustafa Arif

…

On Thu, May 13, 2021 at 10:04 PM Tony Hallett ***@***.***> wrote: Updated vsix with a clear ui button https://ci.appveyor.com/api/buildjobs/fpc9ps3xwtkx2rj3/artifacts/FineCodeCoverage.vsix This will be on the marketplace in the next couple of days. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#132 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIKM33SJHEVN46ODJKIAWZ3TNQBATANCNFSM43UBXIYQ> .

[tonyhallett]

Is there a delay in the release of this version on the marketplace as the
website is still showing the old version

Yes I have some other tasks that are more urgent.

@FortuneN
Please can you update the marketplace !

[tonyhallett]

@FortuneN please do not release

[FortuneN]

On Thu, 20 May 2021 at 22:10, Tony Hallett ***@***.***> wrote: @FortuneN <https://github.com/FortuneN> please do not release — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#132 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABLAATYXA7OQ7RLYYS7SDKLTOVUDDANCNFSM43UBXIYQ> .

😆