Bug 9931

Summary: oe-selftest: Increase organization of code
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Humberto Ibarra <humberto.ibarra.lopez>
Component: Scripts and ToolsAssignee: Mariano Lopez <mariano.lopez>
Status: RESOLVED DUPLICATE QA Contact:
Severity: enhancement    
Priority: Medium CC: benjamin.esquivel, leonardo.sandoval.gonzalez, ross.burton, sgw
Version: 2.2   
Target Milestone: 2.3   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)
Bug Depends on: 9930    
Bug Blocks: 6370    

Description Humberto Ibarra 2016-07-13 19:06:37 UTC
oe-selftest code is starting to grow considerably, and the inner functions could be delegated to different libraries on different files. Maybe a plugin design might benefit this command. The goal is to separate the code of oe-selftest into functional units and call them from a short and clear file.
Comment 1 Benjamin Esquivel 2016-11-03 22:13:40 UTC
Mariano, would you agree that this bug can be closed as dup in favor of the oeqa rewrite?
Comment 2 Mariano Lopez 2016-11-03 22:47:16 UTC
I agree, let's close this bug as duplicated of bug 10599, migration of selftest to the new framework

*** This bug has been marked as a duplicate of bug 10599 ***
Comment 3 Leonardo Sandoval Gonzalez 2016-11-04 21:18:59 UTC
(In reply to comment #2)
> I agree, let's close this bug as duplicated of bug 10599, migration of
> selftest to the new framework
> 
> *** This bug has been marked as a duplicate of bug 10599 ***

Does the oeqa2 refactor includes the runner (oe-selftest)? if not, this bug is valid.
Comment 4 Mariano Lopez 2016-11-04 21:37:54 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > I agree, let's close this bug as duplicated of bug 10599, migration of
> > selftest to the new framework
> > 
> > *** This bug has been marked as a duplicate of bug 10599 ***
> 
> Does the oeqa2 refactor includes the runner (oe-selftest)? if not, this bug
> is valid.

It does