search

reportportal / agent-net-vstest

Logger extension for VsTest
Apache License 2.0
8 stars 10 forks source link

Merge launches #60

Closed heilwood closed 3 years ago

heilwood commented 3 years ago

Hi,

Is it possible to merge two different launches which is running in parallel? Found for specflow agent and nunit, but is it possible to merge using vstest agent?

nvborisenko commented 3 years ago

The same approach https://github.com/reportportal/commons-net/blob/master/docs/Configuration.md#reporting-experience

heilwood commented 3 years ago

Can you explain please how to use regular files or env. variables. For example where i should keep ReportPortal_Launch_Id.txt with value? Currently i'm creating the launch using reportportal api and powershell script from azureDevops pipeline step in another step i writing this launch id to file and saving to bin/Debug and nothing happens, still separate launches

nvborisenko commented 3 years ago

You are doing right, but file name should be ReportPortal_Launch_Id, without .txt extension.

heilwood commented 3 years ago

Looks like i'm doing something wrong: Attached screenshots of my configuration. File created, id added to file, but launches still running in parallel.

startLaunch saveLaunchIdToFile savedFileWithLaunch launches_in_reportportal

nvborisenko commented 3 years ago

You put launch number instead of uuid. UUID is GUID type.

image

If it doesn't work please share your agent`s version.

heilwood commented 3 years ago

Switched to UUID, same issue. Also in my project i have ReportPortal.config.json, maybe here is conflict?

image image

nvborisenko commented 3 years ago

Reproduced it. The issue that the agent doesn't read your file at all.

https://github.com/reportportal/agent-net-vstest/blob/3644d3f530c4d5e2e8f226e262010f0e51f45454/src/ReportPortal.VSTest.TestLogger/ReportPortalLogger.cs#L49

Definitely should be fixed.

heilwood commented 3 years ago

Reproduced it. The issue that the agent doesn't read your file at all.

https://github.com/reportportal/agent-net-vstest/blob/3644d3f530c4d5e2e8f226e262010f0e51f45454/src/ReportPortal.VSTest.TestLogger/ReportPortalLogger.cs#L49

Definitely should be fixed.

Didn't get it.. Should i update vstet testLogger nuget package to get this fix?

nvborisenko commented 3 years ago

Fixed in v3.1.1 please verify.

heilwood commented 3 years ago

thank you very much, bug has been fixed