From b9c785535a020c0bc1b871bca48cee71aa2143d9 Mon Sep 17 00:00:00 2001 From: Robin Houston Date: Wed, 7 Sep 2011 20:52:44 +0100 Subject: Detect multipart/report bounce messages Add support for multipart/report bounce messages to script/handle-mail-replies. Also add a spec test for this script. --- spec/script/handle-mail-replies_spec.rb | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) create mode 100644 spec/script/handle-mail-replies_spec.rb (limited to 'spec/script/handle-mail-replies_spec.rb') diff --git a/spec/script/handle-mail-replies_spec.rb b/spec/script/handle-mail-replies_spec.rb new file mode 100644 index 000000000..856476c3f --- /dev/null +++ b/spec/script/handle-mail-replies_spec.rb @@ -0,0 +1,33 @@ +require File.expand_path(File.dirname(__FILE__) + '/../spec_helper') +require "external_command" + +def mail_reply_test(email_filename) + Dir.chdir RAILS_ROOT do + xc = ExternalCommand.new("script/handle-mail-replies", "--test") + xc.run(load_file_fixture(email_filename)) + + xc.err.should == "" + return xc + end +end + +describe "When filtering" do + it "should detect an Exim bounce" do + r = mail_reply_test("track-response-exim-bounce.email") + r.status.should == 1 + r.out.should == "user@example.com\n" + end + + it "should pass on a non-bounce message" do + r = mail_reply_test("incoming-request-bad-uuencoding.email") + r.status.should == 0 + r.out.should == "" + end + + it "should detect a multipart bounce" do + r = mail_reply_test("track-response-multipart-report.email") + r.status.should == 1 + r.out.should == "FailedUser@example.com\n" + end +end + -- cgit v1.2.3 From 71048a2bd639943801d626fa89e2dfcb2ebb7ee2 Mon Sep 17 00:00:00 2001 From: Robin Houston Date: Thu, 8 Sep 2011 01:56:43 +0100 Subject: Improved OOF detection Slightly improved detection of out-of-office messages, + tests. --- spec/script/handle-mail-replies_spec.rb | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'spec/script/handle-mail-replies_spec.rb') diff --git a/spec/script/handle-mail-replies_spec.rb b/spec/script/handle-mail-replies_spec.rb index 856476c3f..2ef8d6afa 100644 --- a/spec/script/handle-mail-replies_spec.rb +++ b/spec/script/handle-mail-replies_spec.rb @@ -29,5 +29,15 @@ describe "When filtering" do r.status.should == 1 r.out.should == "FailedUser@example.com\n" end + + it "should detect a generic out-of-office" do + r = mail_reply_test("track-response-generic-oof.email") + r.status.should == 2 + end + + it "should detect an Exchange-style out-of-office" do + r = mail_reply_test("track-response-exchange-oof-1.email") + r.status.should == 2 + end end -- cgit v1.2.3 From e6c088e6b72e5bfee26fda8d933e8d71ea672b30 Mon Sep 17 00:00:00 2001 From: Robin Houston Date: Thu, 8 Sep 2011 06:18:35 +0100 Subject: More out-of-office message detection --- spec/script/handle-mail-replies_spec.rb | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'spec/script/handle-mail-replies_spec.rb') diff --git a/spec/script/handle-mail-replies_spec.rb b/spec/script/handle-mail-replies_spec.rb index 2ef8d6afa..b3e6e5dbb 100644 --- a/spec/script/handle-mail-replies_spec.rb +++ b/spec/script/handle-mail-replies_spec.rb @@ -39,5 +39,10 @@ describe "When filtering" do r = mail_reply_test("track-response-exchange-oof-1.email") r.status.should == 2 end + + it "should detect a Lotus Domino-style out-of-office" do + r = mail_reply_test("track-response-lotus-oof-1.email") + r.status.should == 2 + end end -- cgit v1.2.3 From 94ff0bd6139f1c5eae0809dcee5e77ea728a0e19 Mon Sep 17 00:00:00 2001 From: Robin Houston Date: Thu, 8 Sep 2011 06:28:44 +0100 Subject: More out-of-office message detection --- spec/script/handle-mail-replies_spec.rb | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'spec/script/handle-mail-replies_spec.rb') diff --git a/spec/script/handle-mail-replies_spec.rb b/spec/script/handle-mail-replies_spec.rb index b3e6e5dbb..451c66252 100644 --- a/spec/script/handle-mail-replies_spec.rb +++ b/spec/script/handle-mail-replies_spec.rb @@ -44,5 +44,10 @@ describe "When filtering" do r = mail_reply_test("track-response-lotus-oof-1.email") r.status.should == 2 end + + it "should detect a Messagelabs-style out-of-office" do + r = mail_reply_test("track-response-messagelabs-oof-1.email") + r.status.should == 2 + end end -- cgit v1.2.3 From 2442a5322ee560756f521983a0f7f0de4c4aa4e1 Mon Sep 17 00:00:00 2001 From: Robin Houston Date: Thu, 8 Sep 2011 22:12:15 +0100 Subject: Detect OOF messages that identify themselves by a X-POST-MessageClass header --- spec/script/handle-mail-replies_spec.rb | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'spec/script/handle-mail-replies_spec.rb') diff --git a/spec/script/handle-mail-replies_spec.rb b/spec/script/handle-mail-replies_spec.rb index 451c66252..eae0b516b 100644 --- a/spec/script/handle-mail-replies_spec.rb +++ b/spec/script/handle-mail-replies_spec.rb @@ -49,5 +49,10 @@ describe "When filtering" do r = mail_reply_test("track-response-messagelabs-oof-1.email") r.status.should == 2 end + + it "should detect an out-of-office that has an X-POST-MessageClass header" do + r = mail_reply_test("track-response-messageclass-oof.email") + r.status.should == 2 + end end -- cgit v1.2.3