Skip to content

Fix task store writes after parent directory removal#27

Open
baszalmstra wants to merge 1 commit into
tintinweb:masterfrom
baszalmstra:fix-task-store-missing-parent-dir
Open

Fix task store writes after parent directory removal#27
baszalmstra wants to merge 1 commit into
tintinweb:masterfrom
baszalmstra:fix-task-store-missing-parent-dir

Conversation

@baszalmstra

Copy link
Copy Markdown

Summary

  • Prevent task store mutations from failing if the backing store parent directory is removed after initialization.

Changes

  • Recreate the lock parent directory before acquiring a file lock.
  • Recreate the store parent directory before writing the temporary file.
  • Add a regression test that removes the parent directory between mutations.

Testing

  • npm test
  • npm run typecheck
  • npm run lint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant