avocado icon indicating copy to clipboard operation
avocado copied to clipboard

Post test plugins

Open richtja opened this issue 2 years ago • 3 comments

Infrastructure for creating post-test plugins. Such plugin will be able to add tasks after the test task.

Introduces first post test plugin which is using this infrastructure the sysinfo plugin. It adds sysinfo gathering into the suite run. It creates pre and post sysinfo tasks in a suite which will be run before and after the testing. Those tasks will gather sysinfo by sysinfo-runner and save it into the test-results folder.

Reference: #5204, #3877 Signed-off-by: Jan Richter [email protected]

richtja avatar Aug 03 '22 12:08 richtja

This pull request introduces 1 alert when merging 797cd72bdf8edaab095871248f112441c92c162d into 5a0c5b2348da450397287a0954e4c335c0d590a9 - view on LGTM.com

new alerts:

  • 1 for `__eq__` not overridden when adding attributes

lgtm-com[bot] avatar Aug 03 '22 12:08 lgtm-com[bot]

This pull request introduces 1 alert when merging 360110f845867fa9f23897fd3b6ec9f465be0e10 into 5a0c5b2348da450397287a0954e4c335c0d590a9 - view on LGTM.com

new alerts:

  • 1 for `__eq__` not overridden when adding attributes

lgtm-com[bot] avatar Aug 03 '22 15:08 lgtm-com[bot]

Hi @clebergnu thanks for your rewire. I tried to explain my approach during creation of the PR and also problems which I can see. I am not satisfied with this solution, but I wasn't able to come up with something better. So I will appreciate any ideas how to solve this.

richtja avatar Aug 23 '22 09:08 richtja

Hi @clebergnu and @beraldoleal, thanks for your reviews, I created v2 #5630, please have a look.

richtja avatar Mar 28 '23 15:03 richtja