Fix WalltimeHandler crash on incomplete OUTCAR + new test

[janosh]

closes materialsproject/pymatgen#2251

[codecov-commenter]

Codecov Report

❌ Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.97%. Comparing base (a305609) to head (93d1ca1).
⚠️ Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
src/custodian/vasp/handlers.py	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #412      +/-   ##
==========================================
- Coverage   53.02%   52.97%   -0.05%     
==========================================
  Files          39       39              
  Lines        3508     3511       +3     
==========================================
  Hits         1860     1860              
- Misses       1648     1651       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[janosh]

@shyuep could you take a look? also @esoteric-ephemera curious if you've seen this error before and if so, how you worked around it

[shyuep]

I am fine with this. Merged.

[Andrew-S-Rosen]

@janosh: FYI that better practice here would be to do monkeypatch.chdir(tmp_path) to not modify the path of the main process in the test suite.

[Andrew-S-Rosen]

Never mind. I see this is addressed by the tearDown call and is done throughout the module.

[janosh]

strongly agree. been meaning to overhaul the custodian test suite for a long time specifically to fix this