OpenRefine / OpenRefine / #7514

51%

push

GitHub Actions

Don't write newly read projects before they're modified (#6026)

* Restore previous constructor behavior

Use modified time if we're given one.

* Enhance test to cover initial read case

It wasn't testing if an unnecessary project
write occurs when projects are read, but
not written

* Make sure lastSave timestamp is initialized on load - fixes #3805

* Fix test for workspace save on project remove from PR #4796

* Save workspace when projects removed. Fixes #1418

* Refactor to clean up workspace save

- only write temp file if needed rather than writing,
  then deleting if unneeded
- check and log errors on file create/delete/rename
- use try-with-resources to avoid resource leaks

29 of 29 new or added lines in 3 files covered. (100.0%)

15658 of 28003 relevant lines covered (55.92%)

0.56 hits per line