moved message formatting to separate Class

lib/epub_validator/format_error_message.rb

@@ -0,0 +1,18 @@
+module EpubValidator
+  class FormatErrorMessage
+    def process_message(message)
+      return ['Passed.'] if message.match('No errors or warnings detected')
+
+      m_array = message.split(/\n/)
+
+      # clean up all useless info
+      m_array.delete_if do |s|
+        s.empty? or
+          s.match('^Epubcheck Version.*') or
+          s.match('^Check finished.*')
+      end
+
+      m_array.unshift('FAILED!')
+    end
+  end
+end

spec/epub_validator/format_error_message_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+module EpubValidator
+  describe FormatErrorMessage do
+    context "when it recieves a missing file message" do
+      it "should return a 'FAILED!' message as an array" do
+        message = "ERROR: test.epub: I/O error: test.epub (No such file or directory)"
+        formatted_message = ["FAILED!", "ERROR: test.epub: I/O error: test.epub (No such file or directory)"]
+        errm = FormatErrorMessage.new
+        errm.process_message(message).should eq(formatted_message)
+      end
+    end
+    context "when it recieves a valid file message" do
+      it "should return 'Passed.' message as an array" do
+        message = "Epubcheck Version 1.2\n\nNo errors or warnings detected\n"
+        errm = FormatErrorMessage.new
+        errm.process_message(message).should eq(['Passed.'])
+      end
+    end
+    context "when it recieves an invalid file message" do
+      it "should return error message as an array" do
+        message = "Epubcheck Version 1.2\n\nERROR: book.epub: resource OEBPS/stylesheets/handbookish.css is missing\n\nCheck finished with warnings or errors!"
+        formatted_message = ["FAILED!", "ERROR: book.epub: resource OEBPS/stylesheets/handbookish.css is missing"]
+        errm = FormatErrorMessage.new
+        errm.process_message(message).should eq(formatted_message)
+      end
+    end
+  end
+end