mingle_events 0.0.6 → 0.0.7

data/README.textile

@@ -123,12 +123,11 @@ One of the great usages of event playback is historical analysis. E.g., if you w
 It is possible to repeatedly run analysis against previously fetched events using the ProjectEventFetcher class that is only used in the internals of the standard polling mechanism.  ProjectEventFetcher does most of the real work of reading new events from the server and making them available for local processing, so it's not a bad class to get to know.
 
 <pre>
-event_fetcher = MingleEvents::ProjectEventFetcher.new('mingle', mingle_access)
-processor = MingleEvents::Processors::PutsPublisher.new
+event_fetcher = MingleEvents::ProjectEventFetcher.new('my_project', mingle_access)
 event_fetcher.all_fetched_entries.each do |e|
-  processor.process_events([e])
+  # do something interesting with each event
 end  
 </pre>
 
-That's not an ideal interface.  It's a bit awkward to push a single event into an array for processing.  We'll likely look to make this much better as we bring back batch processing of events.
+Until there's time for further documentation take a look at card_count_by_day.rb and story_count_by_day.rb in the examples folder.
 

data/lib/mingle_events/feed.rb

@@ -1,5 +1,5 @@
-require File.expand_path(File.join(File.dirname(__FILE__), 'feed', 'element_support'))
 require File.expand_path(File.join(File.dirname(__FILE__), 'feed', 'author'))
 require File.expand_path(File.join(File.dirname(__FILE__), 'feed', 'category'))
 require File.expand_path(File.join(File.dirname(__FILE__), 'feed', 'entry'))
 require File.expand_path(File.join(File.dirname(__FILE__), 'feed', 'page'))
+require File.expand_path(File.join(File.dirname(__FILE__), 'feed', 'changes'))

data/lib/mingle_events/feed/author.rb

@@ -3,9 +3,7 @@ module MingleEvents
     
     # The user who's Mingle activity triggered this event
     class Author
-    
-      include ElementSupport
-  
+      
       # The name of the author
       attr_reader :name
       # The email address of the author
@@ -16,13 +14,27 @@ module MingleEvents
       attr_reader :icon_uri
   
       def initialize(author_element)
-        @name = element_text(author_element, 'name')
-        @email = element_text(author_element, 'email', true)
-        @uri = element_text(author_element, 'uri', true)
-        @icon_uri = element_text(author_element, 'mingle:icon', true)
+        @name = author_element.at("name").inner_text
+        @email = optional_element_text(author_element, 'email')
+        @uri = optional_element_text(author_element, 'uri')        
+        @icon_uri = optional_element_text(author_element, 'icon')
+      end
+      
+      def self.from_snippet(entry_xml)
+        self.new(Nokogiri::XML(entry_xml))        
+      end
+      
+      private 
+      
+      def optional_element_text(parent_element, element_name)
+        element = parent_element.at(element_name)
+        element.nil? ? nil : element.inner_text
       end
     
     end
-    
+      
   end
-end
+end
+
+
+

data/lib/mingle_events/feed/category.rb

@@ -6,6 +6,8 @@ module MingleEvents
     # All current Mingle categories are also defined here, as constants.
     class Category
     
+      @@categories_by_mingle_term = {}
+    
       # The category's term
       attr_reader :term
       # The category's scheme
@@ -14,6 +16,16 @@ module MingleEvents
       def initialize(term, scheme)
         @term = term
         @scheme = scheme
+        @@categories_by_mingle_term[term] = self
+      end
+      
+      # lookup a category by the text value of the term (mingle scheme is assumed)
+      def self.for_mingle_term(mingle_term)
+        @@categories_by_mingle_term[mingle_term]
+      end
+      
+      def to_s
+        term
       end
     
       def ==(other)
@@ -30,7 +42,7 @@ module MingleEvents
     
       # The Atom category scheme for all Mingle categories
       MINGLE_SCHEME = 'http://www.thoughtworks-studios.com/ns/mingle#categories'
-    
+          
       # Category for any event sourced by a card
       CARD = Category.new('card', MINGLE_SCHEME)
       # Category for any event that is the creation of a new card

data/lib/mingle_events/feed/changes.rb

@@ -0,0 +1,76 @@
+module MingleEvents  
+  module Feed
+    
+    # Enumerable detail for each change specified in the entry's content section 
+    class Changes
+      
+      include Enumerable
+      
+      def initialize(changes_element)
+        @changes_element = changes_element
+      end
+      
+      def each        
+        (@changes ||= parse_changes).each{|c| yield c}
+      end
+      
+      private
+      
+      def parse_changes
+        changes = []
+        @changes_element.search("change").map do |change_element|
+          category = Category.for_mingle_term(change_element["type"])
+          changes <<  Change.new(category).build(change_element)
+        end
+        changes
+      end
+      
+      class Change
+        
+        def initialize(category)
+          @category = category
+        end
+        
+        def build(element)          
+          element_to_hash(element)
+                                    
+          raw_hash_from_xml = element_to_hash(element)
+          
+          raw_hash_from_xml[:change].merge({
+            :category => @category, 
+            :type => @category
+          })
+        end
+        
+        private
+    
+        def element_to_hash(element, hash = {}) 
+          hash_for_element = (hash[element.name.to_sym] ||= {})
+                   
+          element.attribute_nodes.each do |a|
+            hash_for_element[a.name.to_sym] = a.value
+          end
+                   
+          element.children.each do |child|
+            next unless child.is_a?(Nokogiri::XML::Element)
+            
+            if child.children.count{|c| c.is_a?(Nokogiri::XML::Element)} > 0
+              element_to_hash(child, hash_for_element)
+            else
+              hash_for_element[child.name.to_sym] = if child["nil"] && child["nil"] == "true"
+                nil
+              else
+                child.inner_text
+              end
+            end
+          end 
+          
+          hash         
+        end
+                        
+      end      
+      
+
+    end    
+  end
+end

data/lib/mingle_events/feed/entry.rb

@@ -9,6 +9,10 @@ module MingleEvents
       def initialize(entry_element)
         @entry_element = entry_element
       end
+      
+      def self.from_snippet(entry_xml)
+        self.new(Nokogiri::XML(entry_xml).remove_namespaces!)        
+      end
   
       # The raw entry XML from the Atom feed
       def raw_xml
@@ -18,7 +22,7 @@ module MingleEvents
       # The Atom entry's id value. This is the one true identifier for the entry,
       # and therefore the event.
       def entry_id
-        @entry_id ||= @entry_element.at('id').inner_text
+        @entry_id ||= @entry_element.at("id").inner_text
       end
       alias :event_id :entry_id
   
@@ -29,20 +33,30 @@ module MingleEvents
   
       # The time at which entry was created, i.e., the event was triggered
       def updated
-        @updated ||= Time.parse(@entry_element.at('updated').inner_text)
+        @updated ||= Time.parse(@entry_element.at("updated").inner_text)
       end
   
       # The user who created the entry (triggered the event), i.e., changed project data in Mingle
       def author 
-        @author ||= Author.new(@entry_element.at('author'))
+        @author ||= Author.new(@entry_element.at("author"))
       end
     
       # The set of Atom categoies describing the entry
       def categories
-        @categories ||= @entry_element.search('category').map do |category_element|
-          Category.new(category_element.attribute('term').text, category_element.attribute('scheme').text)
+        @categories ||= @entry_element.search("category").map do |category_element|
+          Category.new(category_element["term"], category_element["scheme"])
         end
       end
+      
+      # The array of changes for this entry. Each change is a hash with redundant :type and
+      # :category entries specifying the category to which the change maps.  
+      # Change detail is contained in nested hashes with keys mapping exactly to the XML
+      # as described in http://www.thoughtworks-studios.com/mingle/3.3/help/mingle_api_events.html. 
+      # The data in the change hashes reflect only what is in the XML as encriching them would 
+      # require potentially many calls to the Mingle server resulting in very slow processing. 
+      def changes
+        @changes ||= Changes.new(@entry_element.at("content/changes"))
+      end
   
       # Whether the entry/event was sourced by a Mingle card
       def card?
@@ -60,7 +74,7 @@ module MingleEvents
       # The version number of the card or page that was created by this event. (For now, only 
       # working with cards.)
       def version
-        @version ||= CGI.parse(URI.parse(card_version_resource_uri).query)['version'].first.to_i
+        @version ||= CGI.parse(URI.parse(card_version_resource_uri).query)["version"].first.to_i
       end
   
       # The resource URI for the card version that was created by this event. Throws error if not card event.
@@ -94,14 +108,14 @@ module MingleEvents
           "link[@rel='http://www.thoughtworks-studios.com/ns/mingle#event-source'][@type='application/vnd.mingle+xml']"
         )
         # TODO: improve this bit of parsing :)
-        card_number_element.attribute('href').text.split('/').last.split('.')[0..-2].join.to_i
+        card_number_element["href"].split('/').last.split('.')[0..-2].join.to_i
       end
   
       def parse_card_version_resource_uri
         card_number_element = @entry_element.at(
           "link[@rel='http://www.thoughtworks-studios.com/ns/mingle#version'][@type='application/vnd.mingle+xml']"
         )
-        card_number_element.attribute('href').text
+        card_number_element["href"]
       end
   
     end

data/lib/mingle_events/feed/page.rb

@@ -14,45 +14,24 @@ module MingleEvents
       end
   
       def entries
-        @entries ||= page_as_document.search('feed/entry').map do |entry_element|
+        @entries ||= page_as_document.search('entry').map do |entry_element|
           Entry.new(entry_element)
         end
       end
   
       def next
-        next_url_element = page_as_document.at("feed/link[@rel='next']")
+        next_url_element = page_as_document.at("link[@rel='next']")
         if next_url_element.nil?
           nil
         else
-          Page.new(next_url_element.attribute('href').text, @mingle_access)
-        end
-      end
-    
-      def previous
-        previous_url_element = page_as_document.at("feed/link[@rel='previous']")
-        if previous_url_element.nil?
-          nil
-        else
-          Page.new(previous_url_element.attribute('href').text, @mingle_access)
-        end
-      end
-    
-      def archived?
-        URI.parse(url).query && url == page_as_document.at("feed/link[@rel='self']").attribute('href').text
-      end
-    
-      def closest_archived_page
-        if archived? 
-          self
-        else
-          self.next
+          Page.new(next_url_element["href"], @mingle_access)
         end
       end
   
       private    
   
       def page_as_document
-        @page_as_document ||= Nokogiri::XML(@mingle_access.fetch_page(@url))
+        @page_as_document ||= Nokogiri::XML(@mingle_access.fetch_page(@url)).remove_namespaces!
       end
     
     end

data/lib/mingle_events/project_event_fetcher.rb

@@ -81,7 +81,7 @@ module MingleEvents
     
     def current_state_entry(info_file_key)
       if info_file = load_current_state[info_file_key]
-        entry_for_xml(YAML.load(File.new(info_file))[:entry_xml])
+        Feed::Entry.from_snippet(YAML.load(File.new(info_file))[:entry_xml])
       else
         nil
       end
@@ -106,7 +106,7 @@ module MingleEvents
       current_state = load_current_state
       last_fetched_entry = if current_state[:last_fetched_entry_info_file]
         last_fetched_entry_info = YAML.load(File.new(current_state[:last_fetched_entry_info_file]))
-        Feed::Entry.new(Nokogiri::XML(last_fetched_entry_info[:entry_xml]).at('/entry'))
+        Feed::Entry.from_snippet(last_fetched_entry_info[:entry_xml])
       else 
         nil
       end
@@ -120,10 +120,6 @@ module MingleEvents
       end
     end
     
-    def entry_for_xml(entry_xml)
-      Feed::Entry.new(Nokogiri::XML(entry_xml).at('/entry'))
-    end
-    
     class Entries
       
       include Enumerable
@@ -132,16 +128,12 @@ module MingleEvents
         @first_info_file = first_info_file
         @last_info_file = last_info_file
       end
-      
-      def entry_for_xml(entry_xml)
-        Feed::Entry.new(Nokogiri::XML(entry_xml).at('/entry'))
-      end
-      
+          
       def each(&block)
         current_file = @first_info_file
         while current_file
           current_entry_info = YAML.load(File.new(current_file))
-          yield(entry_for_xml(current_entry_info[:entry_xml]))
+          yield(Feed::Entry.from_snippet(current_entry_info[:entry_xml]))
           break if File.expand_path(current_file) == File.expand_path(@last_info_file)
           current_file = current_entry_info[:next_entry_file_path]
         end

data/test/mingle_events/feed/author_test.rb

@@ -6,15 +6,15 @@ module MingleEvents
     class AuthorTest < Test::Unit::TestCase
   
       def test_parse_attributes
-        element = Nokogiri.XML(%{
-          <author xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+        author_xml = %{
+          <author >
             <name>Sammy Soso</name>
             <email>sammy@example.com</email>
             <uri>https://mingle.example.com/api/v2/users/233.xml</uri>
             <mingle:icon>https://mingle.example.com/user/icon/233/profile.jpg</mingle:icon>
-          </author>})
-        
-        author = Author.new(element)
+          </author>
+        }        
+        author = Author.from_snippet(author_xml)
         assert_equal("Sammy Soso", author.name)
         assert_equal("sammy@example.com", author.email)
         assert_equal("https://mingle.example.com/api/v2/users/233.xml", author.uri)
@@ -22,11 +22,12 @@ module MingleEvents
       end
     
       def test_parse_attributes_when_no_optional_fields
-        element = Nokogiri.XML(%{
-          <author xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+        author_xml = %{
+          <author>
             <name>Sammy Soso</name>
-          </author>})
-        author = Author.new(element)
+          </author>
+        }
+        author = Author.from_snippet(author_xml)
         assert_equal("Sammy Soso", author.name)
         assert_nil(author.email)
         assert_nil(author.uri)

data/test/mingle_events/feed/category_test.rb

@@ -13,6 +13,10 @@ module MingleEvents
         assert_not_equal(Category::CARD, Object.new)
         assert_equal(:foo, {Category::CARD => :foo}[Category.new('card', 'http://www.thoughtworks-studios.com/ns/mingle#categories')])
       end
+      
+      def test_can_lookup_by_mingle_term
+        assert_equal(Category::CARD_CREATION, Category.for_mingle_term('card-creation'))
+      end
     
     end
   

data/test/mingle_events/feed/changes_test.rb

@@ -0,0 +1,344 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'test_helper'))
+
+module MingleEvents
+  module Feed
+    
+    class ChangesTest < Test::Unit::TestCase
+      
+      def test_parse_multiple_changes
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="card-creation"/>
+                <change type="card-type-change">
+                  <old_value nil="true"></old_value>
+                  <new_value>
+                    <card_type url="https://mingle.example.com/api/v2/projects/atlas/card_types/27.xml">
+                      <name>Defect</name>
+                    </card_type>
+                  </new_value>
+                </change>
+                <change type="name-change">
+                  <old_value nil="true"></old_value>
+                  <new_value>A New Card</new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+        
+        # check that all the changes are built
+        assert_equal 3, entry.changes.count        
+        [Category::CARD_CREATION, Category::CARD_TYPE_CHANGE, Category::NAME_CHANGE].each do |change_type|
+          assert entry.changes.find{|c| c[:category] == change_type}
+        end
+        
+        # check that a change's detail is built
+        card_type_change = entry.changes.find{|c| c[:category] == Category::CARD_TYPE_CHANGE}
+        assert_equal('Defect', card_type_change[:new_value][:card_type][:name])
+      end
+            
+      def test_parse_name_change_from_nil
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="name-change">
+                  <old_value nil="true" />
+                  <new_value>Basic email integration</new_value>
+                </change
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+        
+        change = entry.changes.first
+        assert_equal(Category::NAME_CHANGE, change[:type])
+        assert_equal(Category::NAME_CHANGE, change[:category])
+        assert_nil change[:old_value]
+        assert_equal("Basic email integration", change[:new_value])
+      end
+      
+      def test_parse_name_change
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="name-change">
+                  <old_value>Work with email</old_value>
+                  <new_value>Basic email integration</new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+        
+        change = entry.changes.first        
+        assert_equal(Category::NAME_CHANGE, change[:type])
+        assert_equal(Category::NAME_CHANGE, change[:category])
+        assert_equal("Work with email", change[:old_value])
+        assert_equal("Basic email integration", change[:new_value])       
+      end
+      
+      def test_parse_type_info_when_no_custom_builder_specified
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="card-creation"/>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+        
+        assert_equal(Category::CARD_CREATION, entry.changes.first[:type])
+        assert_equal(Category::CARD_CREATION, entry.changes.first[:category])
+      end
+      
+      def test_parse_card_type_change_from_nil
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="card-type-change">
+                  <old_value nil="true"></old_value>
+                  <new_value>
+                    <card_type url="https://mingle.example.com/api/v2/projects/atlas/card_types/27.xml">
+                      <name>Defect</name>
+                    </card_type>
+                  </new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+        
+        change = entry.changes.first
+        
+        assert_equal(Category::CARD_TYPE_CHANGE, change[:type])
+        assert_equal(Category::CARD_TYPE_CHANGE, change[:category])
+        assert_nil change[:old_value]
+        assert_equal("Defect", change[:new_value][:card_type][:name])
+        assert_equal("https://mingle.example.com/api/v2/projects/atlas/card_types/27.xml", change[:new_value][:card_type][:url])
+      end
+      
+      def test_parse_card_type_change
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="card-type-change">
+                  <old_value>
+                    <card_type url="https://mingle.example.com/api/v2/projects/atlas/card_types/30.xml">
+                      <name>Story</name
+                    </card_type>
+                  </old_value>
+                  <new_value>
+                    <card_type url="https://mingle.example.com/api/v2/projects/atlas/card_types/27.xml">
+                      <name>Defect</name>
+                    </card_type>
+                  </new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+        
+        change = entry.changes.first
+        assert_equal(Category::CARD_TYPE_CHANGE, change[:type])
+        assert_equal(Category::CARD_TYPE_CHANGE, change[:category])
+        assert_equal("Story", change[:old_value][:card_type][:name])
+        assert_equal("https://mingle.example.com/api/v2/projects/atlas/card_types/30.xml", change[:old_value][:card_type][:url])
+        assert_equal("Defect", change[:new_value][:card_type][:name])
+        assert_equal("https://mingle.example.com/api/v2/projects/atlas/card_types/27.xml", change[:new_value][:card_type][:url])
+      end
+      
+      def test_parse_card_type_change_to_deleted_type
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="card-type-change">
+                  <old_value>
+                    <card_type url="https://mingle.example.com/api/v2/projects/atlas/card_types/30.xml">
+                      <name>Story</name>
+                    </card_type>
+                  </old_value>
+                  <new_value>
+                    <deleted_card_type>
+                      <name>Card</name>
+                    </deleted_card_type>
+                  </new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+                
+        change = entry.changes.first
+        assert_equal(Category::CARD_TYPE_CHANGE, change[:type])
+        assert_equal(Category::CARD_TYPE_CHANGE, change[:category])
+        assert_equal("Story", change[:old_value][:card_type][:name])
+        assert_equal("https://mingle.example.com/api/v2/projects/atlas/card_types/30.xml", change[:old_value][:card_type][:url])
+        assert_equal("Card", change[:new_value][:deleted_card_type][:name])
+        assert_equal(nil, change[:new_value][:deleted_card_type][:url])
+      end     
+      
+      def test_parse_card_property_change
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="property-change">
+                  <property_definition 
+                      url="http://mingle.example.com/api/v2/projects/atlas/property_definitions/10418.xml">
+                    <name>Priority</name>
+                    <position nil="true"></position>
+                    <data_type>string</data_type>
+                    <is_numeric type="boolean">false</is_numeric>
+                  </property_definition>
+                  <old_value>nice</old_value>
+                  <new_value>must</new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+                
+        change = entry.changes.first
+        assert_equal(Category::PROPERTY_CHANGE, change[:type])
+        assert_equal(Category::PROPERTY_CHANGE, change[:category])
+        assert_equal(
+          "http://mingle.example.com/api/v2/projects/atlas/property_definitions/10418.xml", 
+          change[:property_definition][:url]
+        )
+        assert_equal("Priority", change[:property_definition][:name])
+        assert_equal("nice", change[:old_value])
+        assert_equal("must", change[:new_value])
+      end  
+      
+      def test_parse_card_property_change_from_nil
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="property-change">
+                  <property_definition 
+                      url="http://mingle.example.com/api/v2/projects/atlas/property_definitions/10418.xml">
+                    <name>Priority</name>
+                    <position nil="true"></position>
+                    <data_type>string</data_type>
+                    <is_numeric type="boolean">false</is_numeric>
+                  </property_definition>
+                  <old_value nil="true"></old_value>
+                  <new_value>must</new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+                
+        change = entry.changes.first
+        assert_nil(change[:old_value])
+        assert_equal("must", change[:new_value])
+      end   
+      
+      def test_parse_card_property_change_to_nil
+        element_xml_text = %{
+          <entry>
+            <content type="application/vnd.mingle+xml">
+              <changes>
+                <change type="property-change">
+                  <property_definition 
+                      url="http://mingle.example.com/api/v2/projects/atlas/property_definitions/10418.xml">
+                    <name>Priority</name>
+                    <position nil="true"></position>
+                    <data_type>string</data_type>
+                    <is_numeric type="boolean">false</is_numeric>
+                  </property_definition>
+                  <old_value>nice</old_value>
+                  <new_value nil="true"></new_value>
+                </change>
+              </changes>
+            </content>
+          </entry>}
+        entry = Entry.from_snippet(element_xml_text)
+                
+        change = entry.changes.first
+        assert_equal("nice", change[:old_value])
+        assert_nil(change[:new_value])
+      end  
+      
+      def test_foo
+        foo = %{
+          <entry>
+              <id>https://mingle09.thoughtworks.com/projects/mingle/events/index/1344945</id>
+              <title>story #67 CRUD Project created</title>
+              <updated>2006-11-13T05:45:06Z</updated>
+              <author>
+                <name>Jon Tirsen</name>
+                <email>jtirsen@thoughtworks.com</email>
+                <uri>https://mingle09.thoughtworks.com/api/v2/users/10040.xml</uri>
+              </author>
+              <link href="https://mingle09.thoughtworks.com/api/v2/projects/mingle/cards/67.xml" rel="http://www.thoughtworks-studios.com/ns/mingle#event-source" type="application/vnd.mingle+xml" title="story #67"/>
+              <link href="https://mingle09.thoughtworks.com/projects/mingle/cards/67" rel="http://www.thoughtworks-studios.com/ns/mingle#event-source" type="text/html" title="story #67"/>
+              <link href="https://mingle09.thoughtworks.com/api/v2/projects/mingle/cards/67.xml?version=1" rel="http://www.thoughtworks-studios.com/ns/mingle#version" type="application/vnd.mingle+xml" title="story #67 (v1)"/>
+              <link href="https://mingle09.thoughtworks.com/projects/mingle/cards/67?version=1" rel="http://www.thoughtworks-studios.com/ns/mingle#version" type="text/html" title="story #67 (v1)"/>
+              <category term="card" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
+              <category term="card-creation" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
+              <category term="card-type-change" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
+              <category term="description-change" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
+              <category term="name-change" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
+              <category term="property-change" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
+              <content type="application/vnd.mingle+xml">
+                <changes xmlns="http://www.thoughtworks-studios.com/ns/mingle">
+                  <change type="card-creation"/>
+                  <change type="card-type-change">
+                    <old_value nil="true"/>
+                    <new_value>
+                      <card_type url="https://mingle09.thoughtworks.com/api/v2/projects/mingle/card_types/10134.xml">
+                        <name>story</name>
+                      </card_type>
+                    </new_value>
+                  </change>
+                  <change type="description-change">
+                  </change>
+                  <change type="name-change">
+                    <old_value nil="true"/>
+                    <new_value>CRUD Project</new_value>
+                  </change>
+                  <change type="property-change">
+                    <property_definition url="https://mingle09.thoughtworks.com/api/v2/projects/mingle/property_definitions/10380.xml">
+                      <name>release</name>
+                      <position nil="true"/>
+                      <data_type>string</data_type>
+                      <is_numeric type="boolean">false</is_numeric>
+                    </property_definition>
+                    <old_value nil="true"/>
+                    <new_value>0.92</new_value>
+                  </change>
+                  <change type="property-change">
+                    <property_definition url="https://mingle09.thoughtworks.com/api/v2/projects/mingle/property_definitions/10381.xml">
+                      <name>Priority</name>
+                      <position nil="true"/>
+                      <data_type>string</data_type>
+                      <is_numeric type="boolean">false</is_numeric>
+                    </property_definition>
+                    <old_value nil="true"/>
+                    <new_value>Should</new_value>
+                  </change>
+                </changes>
+              </content>
+            </entry>
+        }
+        
+        entry = Entry.from_snippet(foo)
+        entry.changes.each do |c|
+          puts c
+        end
+      end 
+      
+    end        
+  end
+end

data/test/mingle_events/feed/entry_test.rb

@@ -7,7 +7,7 @@ module MingleEvents
   
       def test_parse_basic_attributes
         element_xml_text = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <id>https://mingle.example.com/projects/mingle/events/index/234443</id>
             <title>Page Special:HeaderActions changed</title>
             <updated>2011-02-03T08:12:42Z</updated>
@@ -16,10 +16,9 @@ module MingleEvents
               <email>sammy@example.com</email>
               <uri>https://mingle.example.com/api/v2/users/233.xml</uri>
             </author>
-          </entry>}
-        element = Nokogiri::XML(element_xml_text)
-      
-        entry = Entry.new(element)
+          </entry>
+        }
+        entry = Entry.from_snippet(element_xml_text)      
         # assert_equal(element_xml_text.inspect, entry.raw_xml.inspect) 
         assert_equal("https://mingle.example.com/projects/mingle/events/index/234443", entry.entry_id)
         assert_equal("Page Special:HeaderActions changed", entry.title)
@@ -29,13 +28,11 @@ module MingleEvents
     
       def test_parse_categories
         element_xml_text = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <category term="foo" scheme='http://tws.com/ns#mingle' />
             <category term="bar" scheme="http://tws.com/ns#go" />
           </entry>}
-        element = Nokogiri::XML(element_xml_text)
-        
-        entry = Entry.new(element)
+        entry = Entry.from_snippet(element_xml_text)
         assert_equal(
           [Category.new('foo', 'http://tws.com/ns#mingle'), Category.new('bar', 'http://tws.com/ns#go')],
           entry.categories
@@ -48,29 +45,25 @@ module MingleEvents
         # that the card number is derived from a single, precise position
       
         element_xml_text = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <category term="card" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
             <link href="https://mingle.example.com/projects/atlas/cards/102" rel="http://www.thoughtworks-studios.com/ns/mingle#event-source" type="text/html" title="bug #103"/>
             <link href="https://mingle.example.com/api/v2/projects/atlas/cards/104.xml?version=7" rel="http://www.thoughtworks-studios.com/ns/mingle#version" type="application/vnd.mingle+xml" title="bug #105 (v7)"/>
             <link href="https://mingle.example.com/api/v2/projects/atlas/cards/106.xml" rel="http://www.thoughtworks-studios.com/ns/mingle#event-source" type="application/vnd.mingle+xml" title="bug #107"/>
             <link href="https://mingle.example.com/projects/atlas/cards/108?version=17" rel="http://www.thoughtworks-studios.com/ns/mingle#version" type="text/html" title="bug #109 (v7)"/>
           </entry>}
-        element = Nokogiri::XML(element_xml_text)
-      
-        entry = Entry.new(element)
+        entry = Entry.from_snippet(element_xml_text)
         assert_equal(106, entry.card_number)
         assert_equal(7, entry.version)
       end
     
       def test_card_number_and_version_throws_error_when_event_not_related_to_a_card      
         element_xml_text = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <category term="page" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
           </entry>}
-        element = Nokogiri::XML(element_xml_text)
-      
-        entry = Entry.new(element)
-      
+        entry = Entry.from_snippet(element_xml_text)
+        
         begin
           entry.card_number
           fail("Should not have been able to retrieve a card number for non card-related event!")
@@ -93,27 +86,23 @@ module MingleEvents
         # that the card number is derived from a single, precise position
       
         element_xml_text = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <category term="card" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
             <link href="https://mingle.example.com/projects/atlas/cards/102" rel="http://www.thoughtworks-studios.com/ns/mingle#event-source" type="text/html" title="bug #103"/>
             <link href="https://mingle.example.com/api/v2/projects/atlas/cards/104.xml?version=7" rel="http://www.thoughtworks-studios.com/ns/mingle#version" type="application/vnd.mingle+xml" title="bug #105 (v7)"/>
             <link href="https://mingle.example.com/api/v2/projects/atlas/cards/106.xml" rel="http://www.thoughtworks-studios.com/ns/mingle#event-source" type="application/vnd.mingle+xml" title="bug #107"/>
             <link href="https://mingle.example.com/projects/atlas/cards/108?version=7" rel="http://www.thoughtworks-studios.com/ns/mingle#version" type="text/html" title="bug #109 (v7)"/>
           </entry>}
-        element = Nokogiri::XML(element_xml_text)
-      
-        entry = Entry.new(element)
+        entry = Entry.from_snippet(element_xml_text)
         assert_equal('https://mingle.example.com/api/v2/projects/atlas/cards/104.xml?version=7', entry.card_version_resource_uri)
       end
     
       def test_card_version_resource_uri_throws_error_when_not_card_event
         element_xml_text = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <category term="page" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
           </entry>}
-        element = Nokogiri::XML(element_xml_text)
-      
-        entry = Entry.new(element)
+        entry = Entry.from_snippet(element_xml_text)
         begin
           entry.card_version_resource_uri
           fail("Should not have been able to retrieve a card version resource URI for non card-related event!")
@@ -124,37 +113,35 @@ module MingleEvents
     
       def test_entry_id_aliased_as_event_id
         element_xml_text = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <id>https://mingle.example.com/projects/mingle/events/index/234443</id>
           </entry>}
-        element = Nokogiri::XML(element_xml_text)
-      
-        entry = Entry.new(element)
+        entry = Entry.from_snippet(element_xml_text)
         assert_equal('https://mingle.example.com/projects/mingle/events/index/234443', entry.event_id)
         assert_equal(entry.entry_id, entry.event_id)
       end
       
       def test_entry_id_determines_equality
         element_xml_text_1 = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <id>https://mingle.example.com/projects/mingle/events/index/234443</id>
             <category term="page" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
           </entry>}
-        entry_1 = Entry.new(Nokogiri::XML(element_xml_text_1))
+        entry_1 = Entry.from_snippet(element_xml_text_1)
         
         element_xml_text_2 = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <id>https://mingle.example.com/projects/mingle/events/index/234443</id>
             <category term="card" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
           </entry>}
-        entry_2 = Entry.new(Nokogiri::XML(element_xml_text_2))
+        entry_2 = Entry.from_snippet(element_xml_text_2)
         
         element_xml_text_3 = %{
-          <entry xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
+          <entry>
             <id>https://mingle.example.com/projects/mingle/events/index/234</id>
             <category term="card" scheme="http://www.thoughtworks-studios.com/ns/mingle#categories"/>
           </entry>}
-        entry_3 = Entry.new(Nokogiri::XML(element_xml_text_3))
+        entry_3 = Entry.from_snippet(element_xml_text_3)
         
         assert entry_1.eql?(entry_2)
         assert entry_1 == entry_2

data/test/mingle_events/feed/page_test.rb

@@ -7,7 +7,7 @@ module MingleEvents
     
       def test_entries_are_enumerable   
         latest_entries_page = Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml', stub_mingle_access)
-      
+              
         assert_equal([
           'https://mingle.example.com/projects/atlas/events/index/103',
           'https://mingle.example.com/projects/atlas/events/index/101',
@@ -26,56 +26,7 @@ module MingleEvents
         last_page = Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=1', stub_mingle_access)
         assert_nil(last_page.next)
       end
-    
-      def test_previous_page_returns_the_page_of_entries_as_specified_by_previous_link
-        current_page = Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=1', stub_mingle_access)
-        previous_page = current_page.previous
-        assert_equal('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=2', previous_page.url)
-        assert_equal('https://mingle.example.com/projects/atlas/events/index/99', previous_page.entries.first.entry_id)
-      end
-    
-      def test_previous_page_when_on_latest_entries
-        latest_entries_page = Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml', stub_mingle_access)
-        assert_nil(latest_entries_page.previous)
-      end
-    
-      def test_can_determine_whether_archived
-        assert !Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml', stub_mingle_access).archived?
-        assert !Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=3', stub_mingle_access).archived?
-        assert Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=1', stub_mingle_access).archived?
-      end
-    
-      def test_can_determine_closest_archived_page
-        assert_equal 'https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=2',
-          Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml', stub_mingle_access).closest_archived_page.url
-        assert_equal 'https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=2',
-          Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=2', stub_mingle_access).closest_archived_page.url      
-      end
-    
-      def test_can_determine_when_there_is_no_closest_archived_page
-        mingle_access = StubMingleAccess.new
-        mingle_access.register_page_content('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=1',
-          %{
-  <feed xmlns="http://www.w3.org/2005/Atom" xmlns:mingle="http://www.thoughtworks-studios.com/ns/mingle">
-    <link href="https://mingle.example.com/api/v2/projects/event_tester/feeds/events.xml" rel="current"/>
-    <link href="https://mingle.example.com/api/v2/projects/event_tester/feeds/events.xml" rel="self"/>
-    <entry>
-      <id>https://mingle.example.com/projects/event_tester/events/index/390</id>
-      <title>Card #2 Card Two created</title>
-      <updated>2011-08-02T22:27:38Z</updated>
-      <author><name>David</name></author>
-    </entry>
-    <entry>
-      <id>https://mingle.example.com/projects/event_tester/events/index/389</id>
-      <title>Card #1 Card One created</title>
-      <updated>2011-08-02T22:27:36Z</updated>
-      <author><name>David</name></author>
-    </entry>
-  </feed>          
-          })
-        assert_nil Page.new('https://mingle.example.com/api/v2/projects/atlas/feeds/events.xml?page=1', mingle_access).closest_archived_page
-      end
-  
+      
     end
   
   end

data/test/mingle_events/project_event_fetcher_test.rb

@@ -76,7 +76,7 @@ module MingleEvents
           <author><name>Bob</name></author>
         </entry>
       }
-      Feed::Entry.new(Nokogiri::XML(entry_xml).at('/entry'))
+      Feed::Entry.from_snippet(entry_xml)
     end
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: mingle_events
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 17
   prerelease: 
   segments: 
   - 0
   - 0
-  - 6
-  version: 0.0.6
+  - 7
+  version: 0.0.7
 platform: ruby
 authors: 
 - David Rice
@@ -59,7 +59,7 @@ files:
 - Gemfile
 - lib/mingle_events/feed/author.rb
 - lib/mingle_events/feed/category.rb
-- lib/mingle_events/feed/element_support.rb
+- lib/mingle_events/feed/changes.rb
 - lib/mingle_events/feed/entry.rb
 - lib/mingle_events/feed/page.rb
 - lib/mingle_events/feed.rb
@@ -83,6 +83,7 @@ files:
 - README.textile
 - test/mingle_events/feed/author_test.rb
 - test/mingle_events/feed/category_test.rb
+- test/mingle_events/feed/changes_test.rb
 - test/mingle_events/feed/entry_test.rb
 - test/mingle_events/feed/page_test.rb
 - test/mingle_events/poller_test.rb

data/lib/mingle_events/feed/element_support.rb

@@ -1,19 +0,0 @@
-module MingleEvents
-  module Feed
-    
-    # Provides some helpers for pulling values from Nokogiri Elems
-    module ElementSupport
-    
-      def element_text(parent_element, element_name, optional = false)
-        element = parent_element.at(".//#{element_name}")
-        if optional && element.nil?
-          nil
-        else
-          element.inner_text
-        end
-      end
-    
-    end
-    
-  end
-end