about summary refs log tree commit homepage
path: root/t
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2016-12-10 01:09:50 +0000
committerEric Wong <e@80x24.org>2016-12-10 03:29:07 +0000
commitc6a8fdf71e2c336f6e591c6f4005c8e922c8f2df (patch)
treefd15d7ce7045145797a3161df26b5e20d336bd7f /t
parentdecbc59daa3b08e58e749b5bd88a9d0cbbb7e14a (diff)
downloadpublic-inbox-c6a8fdf71e2c336f6e591c6f4005c8e922c8f2df.tar.gz
Since we use SearchMsg from Xapian data, we can be
assured we do not get self-referential {references}
field.

However, we may need to be more careful when checking
has_descendent for loops, as blindly calling add_child
could open us up to that possibility...
Diffstat (limited to 't')
-rw-r--r--t/thread-cycle.t8
1 files changed, 0 insertions, 8 deletions
diff --git a/t/thread-cycle.t b/t/thread-cycle.t
index 0e1ecfe2..b0844490 100644
--- a/t/thread-cycle.t
+++ b/t/thread-cycle.t
@@ -70,14 +70,6 @@ SKIP: {
         is($check, $st, 'Mail::Thread output matches');
 }
 
-@msgs = map { bless $_, 'PublicInbox::SearchMsg' } (
-        { mid => 'a@b' },
-        { mid => 'b@c', references => '<a@b> <b@c>' },
-        { mid => 'd@e', references => '<d@e>' },
-);
-
-is(thread_to_s(\@msgs), "a\@b\n b\@c\nd\@e\n", 'ok with self-references');
-
 done_testing();
 
 sub thread_to_s {