Skip to content

Commit 3ee09e7

Browse files
committed
fix filtering on empty text post records
1 parent c93d631 commit 3ee09e7

File tree

2 files changed

+30
-3
lines changed

2 files changed

+30
-3
lines changed

lib/vrhose/ingestor.ex

+1-1
Original file line numberDiff line numberDiff line change
@@ -479,7 +479,7 @@ defmodule VRHose.Ingestor do
479479
|> Enum.any?()}
480480
else
481481
# filter out posts without any text
482-
{text, false}
482+
{text, true}
483483
end
484484
end
485485

test/vrhose/post_flags_test.exs

+29-2
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,34 @@ defmodule VRHose.PostFlagsTest do
1919
"""
2020
|> Jason.decode!()
2121

22-
{_, accepted?} = VRHose.Ingestor.run_filters(post)
23-
assert not accepted?
22+
{_, filtered?} = VRHose.Ingestor.run_filters(post)
23+
assert filtered?
24+
end
25+
26+
test "filters out these posts i think" do
27+
posts_to_filter = [
28+
"""
29+
{"text":"","$type":"app.bsky.feed.post","embed":{"$type":"app.bsky.embed.recordWithMedia","media":{"$type":"app.bsky.embed.images","images":[{"alt":"","image":{"$type":"blob","ref":{"$link":"bafkreiboaz6mjvtaw2nv2d6plixlc6w4c3mkzractx4klzwbmw7go56a6e"},"mimeType":"image/jpeg","size":544839},"aspectRatio":{"width":934,"height":931}},{"alt":"","image":{"$type":"blob","ref":{"$link":"bafkreidk37e7uecrekovsm3pxdbgkuovimswvyk67kd2yb4oxicat3efny"},"mimeType":"image/jpeg","size":487014},"aspectRatio":{"width":750,"height":945}},{"alt":"","image":{"$type":"blob","ref":{"$link":"bafkreigzkjdviuo4plac6ccx56d3uamw3ym5tve336tp23ln6dzqlhktju"},"mimeType":"image/jpeg","size":74021},"aspectRatio":{"width":960,"height":736}},{"alt":"","image":{"$type":"blob","ref":{"$link":"bafkreicvoonwjowxyr4io67zi5237cxmpymr5opnduc62dfqvgwmb4ru34"},"mimeType":"image/jpeg","size":162080},"aspectRatio":{"width":960,"height":887}}]},"record":{"$type":"app.bsky.embed.record","record":{"cid":"bafyreia3e4orqipmnbrznrml5bjdllwsjtccaqd5ii7jqvpn6z5tjmik7u","uri":"at://did:plc:vowtbiwp2g2rqsq3ftrv6gqz/app.bsky.feed.post/3ld2e45ecl222"}}},"langs":["en"],"createdAt":"2024-12-14T04:49:25.192Z"}
30+
""",
31+
"""
32+
{"text":"","$type":"app.bsky.feed.post","embed":{"$type":"app.bsky.embed.recordWithMedia","media":{"$type":"app.bsky.embed.external","external":{"uri":"https://open.spotify.com/album/5hzSssDOGvtV8LtqHDdoDS?si=IyOLfdTGQlW6Pre2fcGLxg","thumb":{"$type":"blob","ref":{"$link":"bafkreiebiydw5yekm2rlsrdlatqhkvay7bzorecegotohqjexi5jgz3bdy"},"mimeType":"image/jpeg","size":277394},"title":"Xeno","description":"Santino Le Saint · EP · 2018 · 3 songs"}},"record":{"$type":"app.bsky.embed.record","record":{"cid":"bafyreiegqnwmxhb4lraxccmgsp7bf4w7xm4y47x2yob7igtkxfmw2imc7a","uri":"at://did:plc:rqehws5hhimzu47hkjal7i5w/app.bsky.feed.post/3ld4lmfer722e"}}},"langs":["en"],"createdAt":"2024-12-14T05:00:18.373Z"}
33+
""",
34+
"""
35+
{"text":"","$type":"app.bsky.feed.post","embed":{"$type":"app.bsky.embed.recordWithMedia","media":{"$type":"app.bsky.embed.external","external":{"uri":"https://media.tenor.com/WB_3sRiDCiAAAAAC/lonely-island-the-creep.gif?hh=280&ww=498","thumb":{"$type":"blob","ref":{"$link":"bafkreifl6wnsejq3m7lguxpwtwgywbjanmodbquecp4b63jvijph5t6t2m"},"mimeType":"image/jpeg","size":102167},"title":"Lonely Island The Creep GIF","description":"ALT: Lonely Island The Creep GIF"}},"record":{"$type":"app.bsky.embed.record","record":{"cid":"bafyreifjiprcysvxoiko2vo7uknboosj232bzwlno2bbeztbj37tiwzt7m","uri":"at://did:plc:egiwo4lte2phjkajxjd4a4ba/app.bsky.feed.post/3ldae4tez5k2x"}}},"langs":["en"],"createdAt":"2024-12-14T05:02:00.713Z"}
36+
""",
37+
"""
38+
{"text":"","$type":"app.bsky.feed.post","embed":{"$type":"app.bsky.embed.recordWithMedia","media":{"$type":"app.bsky.embed.video","video":{"$type":"blob","ref":{"$link":"bafkreihigop2xzvlktxcfjajihrfuvyuybhkofeiyzlr7xf2kr7onmr72e"},"mimeType":"video/mp4","size":1983468},"aspectRatio":{"width":888,"height":1920}},"record":{"$type":"app.bsky.embed.record","record":{"cid":"bafyreidx3drzczjgeq24lfvbtokl7qokctlag2clq2yx5uyodaoyiloe5i","uri":"at://did:plc:za4pbpzm45dsgw6pcyad6yyy/app.bsky.feed.post/3ld4rptp6v22x"}}},"langs":["en"],"createdAt":"2024-12-14T05:03:04.194Z"}
39+
""",
40+
"""
41+
{"text":"","$type":"app.bsky.feed.post","embed":{"$type":"app.bsky.embed.recordWithMedia","media":{"$type":"app.bsky.embed.images","images":[{"alt":"","image":{"$type":"blob","ref":{"$link":"bafkreih77l3xhgf6kj6nfluqux5eq423c2wkk3xqo4cz62xwc3fkyg7rnu"},"mimeType":"image/jpeg","size":255422},"aspectRatio":{"width":1440,"height":1443}}]},"record":{"$type":"app.bsky.embed.record","record":{"cid":"bafyreid4wzky7xogqeuhebb63ez237ktfelkgznfjy67uwi6b3ob2workm","uri":"at://did:plc:5o6k7jvowuyaquloafzn3cfw/app.bsky.feed.post/3ldadyhkmic2s"}}},"langs":["en"],"createdAt":"2024-12-14T05:03:26.689Z"}
42+
"""
43+
]
44+
45+
posts_to_filter
46+
|> Enum.map(fn post_json_text ->
47+
post = Jason.decode!(post_json_text)
48+
{_, filtered?} = VRHose.Ingestor.run_filters(post)
49+
assert filtered?
50+
end)
2451
end
2552
end

0 commit comments

Comments
 (0)