rhtools / vminfo-parser

3 stars 4 forks source link

Humanize Unit Output #58

Closed stratus-ss closed 1 month ago

stratus-ss commented 1 month ago

feat: Units are converted from GB to TB

In order to make things more useable, once output gets over 1000GB they are converted to TB for easier reading. THIS NOT PRECISE! It divides by 1000 instead of 1024. It is meant for aesthetic changes and not for precision math.

BREAKING CHANGE: Output has changed so tests in test_main.py were broken.

Refs: #44 Refs: #59

github-actions[bot] commented 1 month ago

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py40100% 
__main__.py330%1, 3–4
analyzer.py1955472%31, 33–34, 36–37, 42–44, 100–101, 104–105, 107, 114–119, 121, 151, 157, 163, 181, 241–242, 271, 278–280, 309–310, 312, 338, 358, 364–367, 370, 393–394, 422–424, 426–427, 429, 443–444, 446, 451, 457–458
clioutput.py1084657%44, 58, 70–71, 73–79, 81–82, 107–108, 123–124, 136–139, 153–175, 177–178
config.py76396%132, 158, 189
const.py110100% 
visualizer.py1111586%52–55, 66–68, 183, 188, 216–219, 221–222
vmdata.py85890%77–78, 138–139, 150–151, 156–157
vminfo_parser.py793358%36–37, 41, 45, 48, 50–51, 60, 66–68, 77–78, 83, 96, 100, 102, 111–112, 117–118, 124, 127, 132–137, 147, 151, 157, 168
TOTAL67216275% 

Tests Skipped Failures Errors Time
154 0 :zzz: 0 :x: 0 :fire: 1m 56s :stopwatch:
stratus-ss commented 1 month ago

I added in an adjustment to the unit ranges as well since we are humanizing the output. It made more sense to include this change with the original PR as it falls within the scope of this task