Page MenuHomePhabricator

introduce a script that calulcates coverage of spec test suite
Needs ReviewPublic

Authored by bu5hm4n on Sep 5 2019, 7:04 AM.

Details

Summary

this script tells you:

  • how many interfaces are inherited by widgets, but not tested.
  • how many interface tests are available but not applied
  • how many interfaces are successfully tested on the widgets

Additionally a table is printed that shows which widget misses which
interfaces.

Depends on D9850

Diff Detail

Repository
rEFL core/efl
Branch
master
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 13137
bu5hm4n created this revision.Sep 5 2019, 7:04 AM
bu5hm4n requested review of this revision.Sep 5 2019, 7:04 AM
e "./widget_interface_coverage.py", line 4, in <module>
    from ete3 import Tree, TreeStyle, AttrFace, CircleFace, faces, NodeStyle
ModuleNotFoundError: No module named 'ete3'

Also, don't you want to put the file in scripts or src/scripts ?

I want to keep them out there, the scripts folder does contain build related stuff, maybe i can move it at some point, for now i would leave it where it is.

to install ete3: pip install ete3

The src/scripts folder also contains some doc generation scripts. I am not sure we want to pollute even more our root...

Please describe in the script that it depends on ete3 and PyQt5.

zmike requested changes to this revision.Mon, Sep 23, 1:48 PM

should be in scripts/

This revision now requires changes to proceed.Mon, Sep 23, 1:48 PM

As said in the other revision, i do not want to have them in scripts/. Lets keep the discussion in D9851. (But i would prefer to move all statistic-related scripts to something that also tells that, in *one* go)

bu5hm4n planned changes to this revision.Sat, Oct 5, 9:07 AM

This is just broken now. Needs investigation.

bu5hm4n updated this revision to Diff 25909.Sun, Oct 6, 12:36 AM
bu5hm4n edited the summary of this revision. (Show Details)

repair from recent eolian breaks

bu5hm4n updated this revision to Diff 25911.Sun, Oct 6, 1:16 AM
bu5hm4n edited the summary of this revision. (Show Details)

update commit msg