Invictum / serenity-reportportal-integration

Serenity TAF integration with Report Portal
Apache License 2.0
20 stars 19 forks source link

Selenium logs are collected even when their reporting is disabled #101

Closed HajekOndrej closed 4 years ago

HajekOndrej commented 4 years ago

Hi, thanks for this great RP integration for Serenity.

I have found a little issue, that Selenium logs are collected no matter what LogUnits are enabled. They are collected even when reportportal logging is disabled with rp.enable=false.

This would normally not be a problem, but when the webdriver is Appium, Appium logs are collected for each step, which can take a lot of time and I generally don't need that.

Do you have any idea how can I disable that?

Thanks.

Invictum commented 4 years ago

Hi @HajekOndrej For now integration always tries to collect logs for UI tests, this is done for avoiding several configuration points. Seems like issue can be fixed, but it require a little rework.

Invictum commented 4 years ago

Implemented. Will be available in 1.5.0+