Skip to content

feat: fakes derive create/edit/delete write responses#70

Merged
aeharding merged 1 commit into
mainfrom
feat/fake-write-responses
Jul 6, 2026
Merged

feat: fakes derive create/edit/delete write responses#70
aeharding merged 1 commit into
mainfrom
feat/fake-write-responses

Conversation

@aeharding

Copy link
Copy Markdown
Owner

Wave 3 #6 — the highest-value follow-up: derived write responses unblock the most pinned Voyager specs (composing's reply-renders, edit-updates-thread, delete-removes, create-navigates).

Seed posts/comments gain a mutable deleted flag. The fakes derive createPost / createComment / editPost / editComment / deletePost / deleteComment: they mutate the seed store (new entities show up in getPosts/getComments, edits change content, deletes set the flag) and return the updated view. Comments created with parent_id nest via path. Auth-gated; both providers (v1 uses body→content and DELETE verbs; PieFed uses body/title and POST /delete routes).

Provider-matrix test drives the full create→edit→delete cycle for posts and comments through a real client on both lemmyv1 and piefed. Also folds in the comment like/save end-to-end coverage that was reviewed in #69 but merged before it landed.

438 tests, typecheck, lint, build clean.

Seed posts/comments gain a mutable `deleted` flag; the fakes derive
createPost/createComment/editPost/editComment/deletePost/deleteComment —
mutating the seed store (new entities appear in lists, edits change
content, deletes set the flag) and returning the updated view. Comments
created with parent_id nest via path. Auth-gated.

Consumers can drop hand-built create/edit/delete wire responses and
assert the rendered result through the derived fake. Provider-matrix test
covers the full create→edit→delete cycle for posts and comments on both
providers; also folds in the comment like/save coverage from #69.
@aeharding
aeharding merged commit 636af29 into main Jul 6, 2026
1 check passed
@aeharding
aeharding deleted the feat/fake-write-responses branch July 6, 2026 04:16
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