Effective implicit keep action

Author: thperret

.gitignore

@@ -4,3 +4,4 @@
 parser.out
 parsetab.py
 *.pyc
+*.egg-info/

sifter/grammar/command_list.py

@@ -22,5 +22,7 @@ def evaluate(self, message, state=None):
             # isn't required by the standard, but we might as well.
             if len(state.actions) > 0 and state.actions[-1][0] == 'stop':
                 break
+        if state.actions.implicit_keep:
+            state.actions.append('keep')
         return state.actions
 

sifter/t/evaluation_3.msg

@@ -0,0 +1,14 @@
+Date: Tue, 1 Apr 1997 09:06:31 -0800 (PST)
+From: user1@example.org
+To: roadrunner@acme.example.com
+Subject: I have a present for you
+
+Look, I'm sorry about the whole anvil thing, and I really
+didn't mean to try and drop it on you from the top of the
+cliff.  I want to try to make it up to you.  I've got some
+great birdseed over here at my place--top of the line
+stuff--and if you come by, I'll have it all wrapped up
+for you.  I'm really sorry for all the problems I've caused
+for you over the years, but I know we can work this out.
+--
+Wile E. Coyote   "Super Genius"   coyote@desert.example.org

sifter/t/evaluation_3.rules

@@ -0,0 +1,6 @@
+require "fileinto";
+if header :contains "from" "coyote" {
+        discard;
+} elsif header :contains ["subject"] ["$$$"] {
+        discard;
+}

sifter/t/test_evaluation.py

@@ -16,6 +16,12 @@ class TestEvaluateRules(unittest.TestCase):
              [('redirect', 'postmaster@example.com')]),
             ("evaluation_2.msg", "evaluation_2.rules",
              []),
+            ("evaluation_3.msg", "evaluation_1.rules",
+             [('redirect', 'field@example.com')]),
+            ("evaluation_3.msg", "evaluation_2.rules",
+             [('fileinto', ['INBOX'])]),
+            ("evaluation_3.msg", "evaluation_3.rules",
+             [('keep', None)]),
     )
 
     def setUp(self):