search

afewmail / afew

an initial tagging script for notmuch mail
ISC License
325 stars 98 forks source link

Implement initial MailMover test suite #244

Closed kyrias closed 5 years ago

kyrias commented 5 years ago

I believe that this should cover all possible rule cases and max_age.

This is in preparation for another attempt on #147.

kyrias commented 5 years ago

Hmm, it only fails on Python 3.4 now since it didn't have run, but 3.4 went end-of-life in March, so does it need to be supported by afew?

flokli commented 5 years ago

@kyrias I think only keeping around Python 3.6 and 3.7 are fine.

I openend https://github.com/afewmail/afew/pull/245 for that.

kyrias commented 5 years ago

I've rebased and changed so that the mailbox comparisons are done using message IDs rather than filenames, since mids are both more stable, and there's no actual guarantee that the Maildir class will use the filenames as keys. Will squash that into the second commit once approved.

kyrias commented 5 years ago

And now it passed on both 3.6 and 3.7. :)