Pull out RichMessageExtractor

No need to have it embedded
  when it should be unit tested!

app/services/rich_message_extractor.rb

@@ -0,0 +1,28 @@
+class RichMessageExtractor
+
+  RICH_MESSAGE_FIELDS_REGEX = /([^>@]*)@?([^>]*)>?(.*)/
+
+  def initialize(message)
+    @message = message
+  end
+
+  def description
+    fields[1].strip
+  end
+
+  def context
+    fields[2].strip
+  end
+
+  def project
+    stripped = fields[3].strip
+    stripped.blank? ? nil : stripped
+  end
+
+  private
+
+    def fields
+      @message.match(RICH_MESSAGE_FIELDS_REGEX)
+    end
+
+end

app/services/todo_from_rich_message.rb

@@ -1,34 +1,5 @@
 class TodoFromRichMessage
 
-  class RichMessageExtractor
-
-    RICH_MESSAGE_FIELDS_REGEX = /([^>@]*)@?([^>]*)>?(.*)/
-
-    def initialize(message)
-      @message = message
-    end
-
-    def description
-      fields[1].strip
-    end
-
-    def context
-      fields[2].strip
-    end
-
-    def project
-      stripped = fields[3].strip
-      stripped.blank? ? nil : stripped
-    end
-
-    private
-
-      def fields
-        @message.match(RICH_MESSAGE_FIELDS_REGEX)
-      end
-
-  end
-
   attr_reader :user, :default_context_id, :description, :notes
 
   def initialize(user, default_context_id, description, notes)

test/models/rich_message_extractor_test.rb

@@ -0,0 +1,55 @@
+require 'test/unit'
+require 'active_support/core_ext/object/blank'
+require_relative '../../app/services/rich_message_extractor.rb'
+
+class RichMessageExtractorTest < Test::Unit::TestCase
+
+  def test_message_with_all_options
+    message = "ohai@some-context>in-this-project"
+    extractor = RichMessageExtractor.new(message)
+    assert_equal "ohai", extractor.description
+    assert_equal "some-context", extractor.context
+    assert_equal "in-this-project", extractor.project
+  end
+
+  def test_message_without_project
+    message = "ohai @ some-context"
+    extractor = RichMessageExtractor.new(message)
+    assert_equal "ohai", extractor.description
+    assert_equal "some-context", extractor.context
+    assert_equal nil, extractor.project
+  end
+
+  def test_message_without_project
+    message = " ohai @ some-context"
+    extractor = RichMessageExtractor.new(message)
+    assert_equal "ohai", extractor.description
+    assert_equal "some-context", extractor.context
+    assert_equal nil, extractor.project
+  end
+
+  def test_message_without_project_or_context
+    message = "ohai"
+    extractor = RichMessageExtractor.new(message)
+    assert_equal "ohai", extractor.description
+    assert_equal "", extractor.context
+    assert_equal nil, extractor.project
+  end
+
+  def test_message_without_anything
+    message = ""
+    extractor = RichMessageExtractor.new(message)
+    assert_equal "", extractor.description
+    assert_equal "", extractor.context
+    assert_equal nil, extractor.project
+  end
+
+  def test_message_with_just_a_context
+    message = "@some-context"
+    extractor = RichMessageExtractor.new(message)
+    assert_equal "", extractor.description
+    assert_equal "some-context", extractor.context
+    assert_equal nil, extractor.project
+  end
+
+end