Closed jregchiu closed 4 years ago
As a follow-up I think the time it takes to run the test suite is already output (at least for Chris' sample repo)
@jregchiu can you clarify this issue?
@lydiachoi pretty sure we've covered this by collecting the heap and runtime?
Before ./get-coverage is run, we should run another script to instrument the test files with logging to print out memory usage (and maybe wall clock info to figure out runtime, if the test runner doesn't output that info already?)
I suspect it'll be something similar to that exercise in class to instrument code