Re: [PATCH 00/20] oeqa: Split configuration from the code

Richard Purdie
2016-08-09T12:11:35Z

On Mon, 2016-08-08 at 09:22 -0700, Jose Lamego wrote:
> Improve oeqa-selftest capabilities and UX by placing
> test configuration features and variables into a separate
> configuration file.
> Configuration file reading is done through a new library in
> oeqa.utils and specific configuration files for each test are
> stored in a separate directory.
>
> [Yocto 9389]

Looking at these changes, I'm not 100% convinced this is an improvement
in readability or experience. Right now, I can look at the test case
definition and get a reasonable idea of what its doing. After the
change, I have to read two files at once, side by side in order to have
a clue what the test does. Is that an improvement?

I'm open to persuasion on why this patchset is a good idea but the
pieces I looked at didn't seem to help much :/

Cheers,

Richard