You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
More often than not we use a lot of code which is repetitive across tests. The obvious solution is to use avocado framework, and yes we should continue to do so. But there are still quite a few use cases which still cannot go into avocado but still repeats per test.
A basic example would be steps to install packages for each test. There should be a few more which can still be moved as helpers. @narasimhan-v@PraveenPenguin Your thoughts please
The text was updated successfully, but these errors were encountered:
Push as much common code to avocado utils as possible, as is the current model.
When some code is common but can not go into avocado, it is better to have a discussion in avocado and come up with a semi-util / helper model for it in avocado itself. There are more pros to doing that than this approach of helpers. Once we have a good number of helpers which can not go into avocado, then we can think of having a helper in misc-tests.
This discussion has come up in the past, and has not gained enough momentum for further action I guess. So, I suggest to start with listing the most convincing helper modules and start an RFC in avocado, and based on that we can revisit this issue. We can start listing such helpers there.
I think this we discuss i think some time back as well as @narasimhan-v said please try to push everything in avocado itself and this repo only for test case
More often than not we use a lot of code which is repetitive across tests. The obvious solution is to use avocado framework, and yes we should continue to do so. But there are still quite a few use cases which still cannot go into avocado but still repeats per test.
A basic example would be steps to install packages for each test. There should be a few more which can still be moved as helpers.
@narasimhan-v @PraveenPenguin Your thoughts please
The text was updated successfully, but these errors were encountered: