umputun / remark42 / 28544463906

84%

Build Type

push

github

Committed by umputun

Commit Message

fix: make user deletion idempotent for users without comments

deleteUser now succeeds for a user who has no comments (e.g. one who only logged
in) instead of failing on the missing per-user bucket. In hard mode the per-user
bucket is deleted, tolerating bbolt's ErrBucketNotFound so a bucket left behind by
an earlier partial removal is still removed; the comment-deletion failure path now
wraps the actual error.

Because the engine cannot distinguish a valid login-only user from a never-existed
one, deletion is idempotent: /admin/deleteme returns 200 for an unknown (but validly
signed) token rather than 400. The deleteme test is updated to this contract, engine
tests cover hard and soft deletion of login-only and unknown users, and the API docs
note the idempotent behaviour.

Coverage Stats

9 of 12 new or added lines in 1 file covered. (75.0%)

21 existing lines in 2 files now uncovered.

6412 of 7594 relevant lines covered (84.44%)

39.26 hits per line