FineCodeCoverage icon indicating copy to clipboard operation
FineCodeCoverage copied to clipboard

No Glyphs on *.razor files

Open pauldavis-dwe opened this issue 1 year ago • 2 comments

Installed product versions

  • Visual Studio: 2022 Professional
  • This extension: 1.1.214

Description

After successfully getting coverage from running tests, glyphs do not show on .razor files (but do on .cs files)

Steps to recreate

  1. Run tests
  2. See no glyphs on .razor files

Current behavior

No glyphs on .razor files

Expected behavior

Glyphs on .razor files

Side Notes

  • It helps to attach a repro solution so i can test the fix : https://github.com/pauldavis-dwe/FccBugRepro2.git
  • It helps to attach the log: https://pastebin.com/7w4gUvbZ

pauldavis-dwe avatar Apr 29 '24 08:04 pauldavis-dwe

Thanks Paul for reporting and for attaching the repo. I will look at this. In the meantime you can put the C# code in a code-behind file

tonyhallett avatar Apr 29 '24 13:04 tonyhallett

Thanks, I do on my production projects

pauldavis-dwe avatar Apr 29 '24 13:04 pauldavis-dwe

Available in latest release.

tonyhallett avatar May 10 '24 16:05 tonyhallett

Erm... Glyphs in .cs files now don't work...

pauldavis-dwe avatar May 13 '24 10:05 pauldavis-dwe

Thanks Paul will check

tonyhallett avatar May 13 '24 11:05 tonyhallett

Glyphs do work although I have changed the functionality to be strict.

If I file has been changed outside of the document then the coverage marks can no longer be expected to apply. If you have the file open and change it after test execution starts then again the marks can no loner be expected to apply.

If you check the FCC output window pane there should be a message when this occurs.

I can add a boolean option to override this behaviour.

tonyhallett avatar May 13 '24 12:05 tonyhallett

If this is not the behaviour that you are witnessing then please chaeck the FCC Output Window Pane for exceptions.

tonyhallett avatar May 13 '24 12:05 tonyhallett

OK, seems to be working as expected after a reinstall. Sorry for the trouble

pauldavis-dwe avatar May 14 '24 06:05 pauldavis-dwe