renaming methods for clarity

throws exception when trying to fetch using base class

lib/fetcher/base.rb

@@ -9,15 +9,23 @@ def initialize(options={})
 
     def fetch
       establish_connection
-      get_message
+      get_messages
       close_connection
     end
 
     protected
 
-    def establish_connection; true; end
-    def get_message; true; end
-    def close_connection; true; end
+    def establish_connection
+      raise NotImplementedError, "This method should be overridden by subclass"
+    end
+
+    def get_messages
+      raise NotImplementedError, "This method should be overridden by subclass"
+    end
+
+    def close_connection
+      raise NotImplementedError, "This method should be overridden by subclass"
+    end
 
   end
 end

lib/fetcher/imap.rb

@@ -4,18 +4,18 @@ module Fetcher
   class Imap < Base
 
     def initialize(options={})
-      @authentication = options.delete(:authentication)
+      @authentication = options.delete(:authentication) || 'PLAIN'
       super(options)
     end
 
     protected
 
     def establish_connection
       @connection = Net::IMAP.new(@server)
-      @connection.authenticate((@authentication || 'PLAIN'), @username, @password)
+      @connection.authenticate(@authentication, @username, @password)
     end
 
-    def get_message
+    def get_messages
       @connection.select('INBOX')
       @connection.search(['ALL']).each do |message_id|
         msg = @connection.fetch(message_id,'RFC822')[0].attr['RFC822']

lib/fetcher/pop.rb

@@ -16,14 +16,16 @@ def establish_connection
       @connection.start(@username, @password)
     end
 
-    def get_message
+    def get_messages
       unless @connection.mails.empty?
         @connection.each_mail do |msg|
+          # Process the message
           begin
             @receiver.receive(msg.pop)
           rescue
             # Store the message for inspection if the receiver errors
           end
+          # Delete message from server
           msg.delete
         end
       end

test/fetcher_test.rb

@@ -1,4 +1,5 @@
-require File.dirname(__FILE__) + '/../../../../config/boot'
+# require File.dirname(__FILE__) + '/../../../../config/boot'
+require 'rubygems'
 require 'test/unit'
 require 'mocha'
 require 'fetcher'
@@ -20,7 +21,7 @@ def test_should_set_configuration_instance_variables
     assert_equal @receiver, @fetcher.instance_variable_get(:@receiver)
   end
 
-  def test_should_fetch_message
-    assert @fetcher.fetch
+  def test_should_require_subclass
+    assert_raise(NotImplementedError) { @fetcher.fetch }
   end
 end