Closed jduss4 closed 1 year ago
Merging #1625 (50a37e7) into main (031595c) will increase coverage by
0.02%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## main #1625 +/- ##
==========================================
+ Coverage 94.11% 94.14% +0.02%
==========================================
Files 66 66
Lines 4115 4135 +20
==========================================
+ Hits 3873 3893 +20
Misses 242 242
see 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Hold off on merging this until we can discuss how the estimated hours for weekly allocation are being calculated
Let me know if you'd like me to squash the commits!
Description
Adds weekly allocation estimated hour equivalent into existing queries / structures that calculate utilization across time and organizations. That is, where previously only
hours_billed
was being used for theutilization
page, now we also includetotal_allocation_hours
to combine into abillable
annotation.Updated tests where necessary and created new tests for time (monthly utilization with two timecards) and organization (all timecards from last week).
Closes https://github.com/18F/tock/issues/1592
Additional information
Compare the utilization for two users before and after this change. The utilization and billed hours (numerator) change, because weekly allocation's estimated hours are being added to the mix.
On current main branch:
This branch:
NOTE Older timecards with incorrect
target_hours
will appear with inflated utilizationThis change does not appear to change any of the analytics pages or visualizations