Screenshot added via NUnit TestContext.AddTestAttachment is displayed twice in GHPReport

[Steffx115]

Steps to reproduce
Add screenshot to NUnit TestContext via AddtestAttachment method

Expected behavior
Screenshot should only be displayed once

Desktop (please complete the following information):

• OS: Windows 10 Enterprise 64 bit
• Browser: Chrome 77.0.3865.90 64 bit

[elv1s42]

Hi @Steffx115 !

Can you please clarify how do you take screenshots in your tests?
Are you doing it inside [TearDown] method or inside [Test] method?
Are you using only TestContext.AddTestAttachment(...) method without ScreenHelper.SaveScreenshot(...) method?

These details will help me to reproduce the issue.

Thank you

[Steffx115]

Hi,

We have a class called TestBase ,from which every TestClass is inherited, inside this class we take the screenshot inside the [TearDown] method, and we only use the TestContext.AddTestAttachment(...) method to save the screenshot

[elv1s42]

Hi @Steffx115,

I've tried to reproduce the issue on my side and there are no duplicates.
I used this code to reproduce the issue:

The report has only one screenshot in this case.

[Steffx115]

Did u let the Test fail so the Screenshot is taken from the TearDown method?
Because we add the ScreenShot int he teardown using TestContext.AddTestAttachment

[elv1s42]

Hi @Steffx115 !

Sorry, my bad.

I've tried to reproduce it once again with the correct repro steps - failing test with screenshot taken in the [Teardown] method:

The test is now failing:

But still one screenshot is taken:

Please note that I'm using the prepared base64 string instead of the real screenshot for tests to work faster:

Thank you

[Steffx115]

The only difference i notice now is, that we use .bmp as file extension for saving the screenshot

EDIT: i changed the file extension to .png, but this had no effect on this issue

[Steffx115]

additional Info: Opening the test result via the VS Test explorer also only shows one Screenshot as test attachment

[elv1s42]

Hi @Steffx115 !
Still no luck with trying to reproduce the issue.
Can you please try to run this test to see if you'll get the screenshot twice for my example as well?

Thank you

[Steffx115]

hi @elv1s42 , i just noticed that you didnt reproduce this with the class structure that I use
you would need to move the [TestFixture] declaration and the [Teardown] method, which takes the screenshot, to another class and inherit from it.

So then you would have public class Tests : TestBase which contains only the tests, and no Teardown and no TestFixture declaration