diff options
author | Eric Wong <e@80x24.org> | 2021-02-09 07:09:33 -0100 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-02-10 06:59:07 +0000 |
commit | fa3f0cbcd1af5008e56c77e3c46ab60b5eca3a13 (patch) | |
tree | 318cbf3b843d0a33829ecf1fbb6c8becbe0b719e /t | |
parent | cf3d02714d560cfeab1c5582ad2e5a11542cd649 (diff) | |
download | public-inbox-fa3f0cbcd1af5008e56c77e3c46ab60b5eca3a13.tar.gz |
MdirReader now handles files in "$MAILDIR/new" properly and is stricter about what it accepts. eml_from_path is also made robust against FIFOs while eliminating TOCTOU races with between stat(2) and open(2) calls.
Diffstat (limited to 't')
-rw-r--r-- | t/mdir_reader.t | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/t/mdir_reader.t b/t/mdir_reader.t new file mode 100644 index 00000000..51b38af4 --- /dev/null +++ b/t/mdir_reader.t @@ -0,0 +1,22 @@ +#!perl -w +# Copyright (C) 2021 all contributors <meta@public-inbox.org> +# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt> +use PublicInbox::TestCommon; +require_ok 'PublicInbox::MdirReader'; +*maildir_basename_flags = \&PublicInbox::MdirReader::maildir_basename_flags; +*maildir_path_flags = \&PublicInbox::MdirReader::maildir_path_flags; + +is(maildir_basename_flags('foo'), '', 'new valid name accepted'); +is(maildir_basename_flags('foo:2,'), '', 'cur valid name accepted'); +is(maildir_basename_flags('foo:2,bar'), 'bar', 'flags name accepted'); +is(maildir_basename_flags('.foo:2,bar'), undef, 'no hidden files'); +is(maildir_basename_flags('fo:o:2,bar'), undef, 'no extra colon'); +is(maildir_path_flags('/path/to/foo:2,S'), 'S', 'flag returned for path'); +is(maildir_path_flags('/path/to/.foo:2,S'), undef, 'no hidden paths'); +is(maildir_path_flags('/path/to/foo:2,'), '', 'no flags in path'); + +# not sure if there's a better place for eml_from_path +use_ok 'PublicInbox::InboxWritable', qw(eml_from_path); +is(eml_from_path('.'), undef, 'eml_from_path fails on directory'); + +done_testing; |