Add tests for avatars, banners and more #102
Loading…
Reference in New Issue
No description provided.
Delete Branch "more-tests"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This should cover all the important fields, but its entirely possible that I missed something. All tests are passing for me.
@ -47,0 +54,4 @@expect(commentOne.content).toBe(commentTwo.content);expect(commentOne.creator_name).toBe(commentTwo.creator_name);expect(commentOne.community_actor_id).toBe(commentTwo.community_actor_id);expect(commentOne.published).toBe(commentTwo.published);I'm a little bit worried about published here, but its probably being set manually, so its fine. The test would fail otherwise.
@ -47,0 +54,4 @@expect(commentOne.content).toBe(commentTwo.content);expect(commentOne.creator_name).toBe(commentTwo.creator_name);expect(commentOne.community_actor_id).toBe(commentTwo.community_actor_id);expect(commentOne.published).toBe(commentTwo.published);I'm a little bit worried about published here, but its probably being set manually, so its fine. The test would fail otherwise.
@ -18,0 +33,4 @@expect(communityOne.category_id).toBe(communityTwo.category_id);expect(communityOne.removed).toBe(communityTwo.removed);expect(communityOne.deleted).toBe(communityTwo.deleted);}Good idea on these helper functions.
@ -41,0 +47,4 @@expect(postOne.ap_id).toBe(postTwo.ap_id);expect(postOne.name).toBe(postTwo.name);expect(postOne.body).toBe(postTwo.body);expect(postOne.url).toBe(postTwo.url);These are sometimes both undefined, but hopefully the check is done correctly.
@ -41,0 +52,4 @@expect(postOne.embed_title).toBe(postTwo.embed_title);expect(postOne.embed_description).toBe(postTwo.embed_description);expect(postOne.embed_html).toBe(postTwo.embed_html);expect(postOne.published).toBe(postTwo.published);Published time worries me a bit here too.
It passed.