motiro 0.6.3 → 0.6.4

data/test/fixtures/revisions.yml

@@ -0,0 +1,134 @@
+page_creation:
+  id: 1
+  page_id: 11
+  kind: common
+  modified_at: 2007-01-22 12:18:43
+  last_editor_id: 1000004 #john
+  editors: ""
+  title: This page has been created
+  text: John entered some text here
+
+page_edition:
+  id: 2
+  page_id: 11
+  kind: common
+  modified_at: 2007-01-22 13:21:26
+  last_editor_id: 1000005 #eric
+  editors: ""
+  title: This page has been edited since creation
+  text: Eric changed the text that John entered
+
+main_page_creation:
+  id: 3
+  page_id: 1
+  last_editor_id: 1000001
+  editors:  ""
+  text: = Motiro =
+        
+        This is project motiro
+
+johns_page_creation:
+  id: 4
+  page_id: 2
+  last_editor_id: 1000004
+  editors: john
+  text: This is John's page
+  
+bob_and_erics_page_creation:
+  id: 5
+  page_id: 3
+  last_editor_id: 1000001 #bob
+  editors: bob eric
+  text: This is Bob and Eric's page
+  
+nobodys_page_creation:
+  id: 6
+  page_id: 4
+  last_editor_id:
+  editors: ""
+  text: This page was created before the wiki access control was in place
+  
+test_page_creation:
+  id: 7
+  page_id: 5
+  kind: common
+  last_editor_id: 1000001
+  editors: ""
+  title: 
+  text: This is the page text
+  
+non_matching_title_page_creation:
+  id: 8
+  page_id: 6
+  kind: common
+  last_editor_id: 1000001
+  editors: ""
+  title: The title does not match the name
+  text: This is the page text
+  
+list_last_modified_features_page_creation:
+  id: 9
+  page_id: 7
+  kind: feature
+  last_editor_id: 1000001
+  modified_at: 2033-12-27 16:47:00
+  editors: ""
+  title: List last modified features
+  text: Motiro should list the last modified feature suggestion pages on the
+        main page
+
+provide_rss_feed_for_last_modified_pages_page_creation:
+  id: 10
+  page_id: 8
+  kind: feature
+  last_editor_id: 1000001
+  modified_at: 2033-12-27 12:32:48
+  editors: ""
+  title: Provide RSS feed for last modified pages
+  text: There could be an RSS feed for the last modified pages
+  
+provide_rss_feed_for_last_modified_pages_page_edition:
+  id: 11
+  page_id: 8
+  kind: feature
+  modified_at: 2033-12-27 12:33:48
+  last_editor_id: 1000004
+  editors: ""
+  title: Provide RSS feed for last modified pages
+  text: There could be an RSS feed for the last modified pages
+
+events_are_just_special_pages_too_page_creation:
+  id: 12
+  page_id: 9
+  kind: feature
+  last_editor_id: 1000001
+  modified_at: 2033-12-25 18:41:12
+  editors: ""
+  title: Events are just special wiki pages too
+  text: Event descriptions can be treated just like other regular wiki pages,
+        in the same way that feature suggestion pages are
+        
+events_are_just_special_pages_too_page_edition:
+  id: 13
+  page_id: 9
+  kind: feature
+  modified_at: 2033-12-25 18:43:31
+  last_editor_id: 1000005
+  editors: ""
+  title: Events are just special wiki pages too
+  text: Event descriptions can be treated just like other regular wiki pages,
+        in the same way that feature suggestion pages are
+        
+release_event_creation:
+  id: 14
+  page_id: 10
+  kind: event
+  modified_at: 2007-01-18 13:28:47
+  last_editor_id: 1000004 #john
+  editors: ""
+  title: Fourteenth release
+  happens_at: 2007-01-24
+  text: The 0.6.1 release will be a bug fixing one and will introduce editable
+        event pages. This is included as part of the everything is a wiki page
+        motto.
+ 

data/test/functional/root_controller_test.rb

@@ -6,7 +6,7 @@ class RootController; def rescue_action(e) raise e end; end
 
 class RootControllerTest < Test::Unit::TestCase
 
-  fixtures :users, :pages
+  fixtures :users, :pages, :revisions
 
   def setup
     @controller = RootController.new
@@ -17,7 +17,7 @@ class RootControllerTest < Test::Unit::TestCase
   def test_version_number
     get :index, :locale => 'en'
     
-    assert_tag :content => /Motiro version 0.6.3/
+    assert_tag :content => /Motiro version 0.6.4/
   end
   
 

data/test/functional/wiki_controller_test.rb

@@ -26,7 +26,7 @@ class WikiControllerTest < Test::Unit::TestCase
 
   include TestConfiguration
   
-  fixtures :users, :pages
+  fixtures :users, :pages, :revisions
 
   def setup
     @controller = WikiController.new
@@ -75,8 +75,7 @@ class WikiControllerTest < Test::Unit::TestCase
       page_provider.should_receive(:find_by_name).
           with('TestPage').
           once.
-          and_return(Page.new(:name => 'TestPage',
-                                :text => ''))
+          and_return(Page.new(:name => 'TestPage'))
       @controller = WikiController.new(page_provider)
 
       ensure_logged_in
@@ -86,8 +85,8 @@ class WikiControllerTest < Test::Unit::TestCase
   end
   
   def test_authentication_required_for_edition
-     get :edit, { :page_name => 'TestPage' }
-     assert_redirected_to(:controller => 'account', :action => 'login')
+     get :edit, :page_name => 'TestPage'
+     assert_redirected_to(:action => 'show', :page_name => 'TestPage')
   end
   
   def test_only_original_author_can_change_editors_list
@@ -264,17 +263,62 @@ class WikiControllerTest < Test::Unit::TestCase
     assert_xml_element "//div[@id = 'crumbs']/a[@href = '/report/older/events' and text() = 'Events']"
     assert_xml_element "//div[@id = 'crumbs']/a[@href = '/wiki/show/#{release_event.name}' and text() = '#{release_event.title}']"
   end
+  
+  def test_shows_history_summary_and_details_for_new_wiki_pages
+    log_as :bob
+    post :save, :page_name => 'RevisedPage',
+                :btnSave => true, 
+                :page => { :title => 'The title will be changed',
+                           :text => 'Some very boring text',
+                           :kind => 'common',
+                           :editors => '' }
+    post :save, :page_name => 'RevisedPage',
+                :btnSave => true, 
+                :page => { :title => 'The title was changed',
+                           :text => 'A little more exciting text',
+                           :kind => 'common',
+                           :editors => '' }
+
+    log_out                       
+
+    get :history, :page_name => 'RevisedPage'
+    
+    assert_tag :tag => 'a', :attributes => {
+                 :href => @controller.url_for(
+                   :controller => 'wiki', :action => 'show',
+                   :page_name => 'RevisedPage', :revision => '1')},
+               :content => /The title will be changed/
+    assert_tag :content => /The title was changed/
+
+    get :show, :page_name => 'RevisedPage', :revision => '1'
+    assert_tag :content => /Some very boring text/
+
+    get :show, :page_name => 'RevisedPage', :revision => '0'
+    assert_tag :content => /A little more exciting text/
+  end
+  
+  def test_displays_old_text_when_showing_revisions
+    get :show, :page_name => pages('changed_page').name, :revision => '0'
+    assert_tag :content => revisions('page_edition').text
 
+    get :show, :page_name => pages('changed_page').name, :revision => '1'
+    assert_tag :content => revisions('page_creation').text
+  end
+  
 private
 
   def log_as(user_name)
     @request.session[:user] = users(user_name)
   end
   
+  def log_out
+    @request.session[:user] = nil
+  end
+  
   def mocked_page
-    Page.new(:name => 'MockedPage', :title => 'Mocked page',
-             :last_editor => nil, :modified_at => nil, :kind => 'common',
-             :text => "You've been mocked!\n\n--- pt-br ----\n\nVocê foi enganado!")
+    Page.new(:name => 'MockedPage').
+      revise(bob, now, :title => 'Mocked page', :kind => 'common',
+                       :text => "You've been mocked!\n\n--- pt-br ----\n\nVocê foi enganado!")
   end
   
 end

data/test/test_helper.rb

@@ -1,3 +1,20 @@
+#  Motiro - A project tracking tool
+#  Copyright (C) 2006-2007  Thiago Arrais
+#  
+#  This program is free software; you can redistribute it and/or modify
+#  it under the terms of the GNU General Public License as published by
+#  the Free Software Foundation; either version 2 of the License, or
+#  any later version.
+#  
+#  This program is distributed in the hope that it will be useful,
+#  but WITHOUT ANY WARRANTY; without even the implied warranty of
+#  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#  GNU General Public License for more details.
+#  
+#  You should have received a copy of the GNU General Public License
+#  along with this program; if not, write to the Free Software
+#  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
 ENV["RAILS_ENV"] = "test"
 
 $:.push File.expand_path(File.dirname(__FILE__))
@@ -33,4 +50,16 @@ class Test::Unit::TestCase
   self.use_instantiated_fixtures  = false
 
   # Add more helper methods to be used by all tests here...
+  def bob; users('bob'); end
+  def john; users('john'); end
+  def now; Time.local(2007, 3, 27, 23, 41, 38); end
+
+  def revise_named_page(attrs)
+    Page.new(:name => 'SomePage').revise(bob, now, attrs)
+  end
+
+  def revise_brand_new_page(attrs)
+    Page.new(:name => nil).revise(bob, now, attrs)
+  end
+
 end

data/test/unit/page_test.rb

@@ -18,91 +18,92 @@
 require File.dirname(__FILE__) + '/../test_helper'
 
 class PageTest < Test::Unit::TestCase
-  fixtures :pages
-
+  fixtures :pages, :revisions, :users
+  
   def test_is_open_to_all
-    attrs = { :name => 'SomePage', :text => 'Page text' }
-    assert !Page.new(attrs.merge(:editors => 'john')).is_open_to_all?
-    assert  Page.new(attrs.merge(:editors => '')).is_open_to_all?
-    assert  Page.new(attrs.merge(:editors => '  ')).is_open_to_all?
-    assert !Page.new(attrs.merge(:editors => '  john ')).is_open_to_all?
-    assert  Page.new(attrs.merge(:editors => "\n")).is_open_to_all?
+    attrs = { :text => 'Page text' }
+    assert !revise_named_page(attrs.merge(:editors => 'john')).is_open_to_all?
+    assert  revise_named_page(attrs.merge(:editors => '')).is_open_to_all?
+    assert  revise_named_page(attrs.merge(:editors => '  ')).is_open_to_all?
+    assert !revise_named_page(attrs.merge(:editors => '  john ')).is_open_to_all?
+    assert  revise_named_page(attrs.merge(:editors => "\n")).is_open_to_all?
   end
   
   def test_fresh_page_is_open_to_all
-    page = Page.new(:name => 'FreshPage')
-    
-    assert page.is_open_to_all?
+    assert Page.new(:name => 'FreshPage').is_open_to_all?
+  end
+  
+  def test_fresh_page_shows_default_text
+    assert_equal WIKI_NOT_FOUND_TEXT, Page.new(:name => 'FreshPage').text
+  end
+  
+  def test_default_text_for_main_page_shows_some_appraisal
+    assert_equal CONGRATS_TEXT, Page.new(:name => 'MainPage').text
   end
   
   def test_fresh_page_has_empty_editors_list
-    page = Page.new(:name => 'AnotherFreshPage')
-    
-    assert_equal 0, page.editors.size
+    assert_equal 0, Page.new(:name => 'AnotherFreshPage').editors.size
   end
   
   def test_page_name_is_title_camelized
-    page = Page.new(:title => 'How to write a Motiro page')
-    
-    assert_equal 'HowToWriteAMotiroPage', page.name
+    assert_equal 'HowToWriteAMotiroPage',
+                 revise_brand_new_page(:title => 'How to write a Motiro page').name
   end
   
   def test_user_entered_page_name_takes_precedence_over_camelized_title
     page = Page.new(:name => 'MyFirstPageName')
-    page.title = 'My first page title'
+    page.revise(bob, now, :title => 'My first page title')
     
     assert_equal 'MyFirstPageName', page.name  
-    
-    page = Page.new(:title => 'My second page title')
-    page.name = 'MySecondPageName'
-    
-    assert_equal 'MySecondPageName', page.name
   end
   
   def test_converts_utf8_decorated_vowels_when_generating_name
-    assert_equal 'MinhaPagina', Page.new(:title => 'Minha página').name
+    assert_equal 'MinhaPagina',
+                 revise_brand_new_page(:title => 'Minha página').name
     assert_equal 'AaaaaEeeeIiiiiYyyOooooUuuuu',
-                 Page.new(:title => 'ãàáâä èéêë ĩìíîï ÿýý õòóôö ũùúûü').name
+                 revise_brand_new_page(:title => 'ãàáâä èéêë ĩìíîï ÿýý õòóôö ũùúûü').name
     assert_equal 'AaaaaEeeeIiiiiYyOooooUuuuu',
-                 Page.new(:title => 'ÃÀÁÂÄ ÈÉÊË ĨÌÍÎÏ ÝŸ ÕÒÓÔÖ ŨÙÚÛÜ').name
+                 revise_brand_new_page(:title => 'ÃÀÁÂÄ ÈÉÊË ĨÌÍÎÏ ÝŸ ÕÒÓÔÖ ŨÙÚÛÜ').name
   end
   
   def test_converts_utf8_decorated_consonants_when_generating_name
-    assert_equal 'ElNino', Page.new(:title => 'El Niño').name
+    assert_equal 'ElNino', revise_brand_new_page(:title => 'El Niño').name
     assert_equal 'NnCcSrlzNnCcSrlz',
-                 Page.new(:title => 'ñń ćç śŕĺź ÑŃ ÇĆ ŚŔĹŹ').name
+                 revise_brand_new_page(:title => 'ñń ćç śŕĺź ÑŃ ÇĆ ŚŔĹŹ').name
   end
 
   def test_drops_non_alpha_numeric_characters_when_generating_name
     assert_equal 'ThisHasLotsOfPunctuation',
-                 Page.new(:title => 'This, has. lots! of? punctuation').name
+                 revise_brand_new_page(:title => 'This, has. lots! of? punctuation').name
     assert_equal 'TitleWithSomeStrangeCharacters',
-                 Page.new(:title => 'Title%with$some&strange*characters').name
+                 revise_brand_new_page(:title => 'Title%with$some&strange*characters').name
   end
 
-  def test_resolve_clashing_page_names
-    fst_page = Page.new(:title => 'My first Motiro page', :text => '')
+  def test_resolves_clashing_page_names
+    fst_page = revise_brand_new_page(:title => 'My first Motiro page', :text => '')
     assert_equal 'MyFirstMotiroPage', fst_page.name
     fst_page.save
     
-    snd_page = Page.new(:title => 'My first Motiro page', :text => '')
+    snd_page = revise_brand_new_page(:title => 'My first Motiro page', :text => '')
     assert_equal 'MyFirstMotiroPage2', snd_page.name
     snd_page.save
     
-    trd_page = Page.new(:title => 'My first Motiro page')
+    trd_page = revise_brand_new_page(:title => 'My first Motiro page')
     assert_equal 'MyFirstMotiroPage3', trd_page.name
   end
   
   def test_uses_place_holder_title_when_nil
-    page = Page.new
-    
-    assert_equal PLACE_HOLDER_TITLE.t, page.title
+    assert_equal PLACE_HOLDER_TITLE.t, Page.new.title
+  end
+  
+  def test_uses_place_holder_title_for_new_pages
+    assert_equal PLACE_HOLDER_TITLE.t, Page.new.title    
   end
   
-  def test_uses_place_holder_title_when_empty
-    page = Page.new(:title => '')
+  def test_uses_kind_based_title_when_revised_with_empty_title
+    page = revise_brand_new_page(:kind => 'feature', :title => '')
     
-    assert_equal PLACE_HOLDER_TITLE.t, page.title    
+    assert_equal 'Feature page', page.title    
   end
   
   def test_translates_default_title
@@ -112,27 +113,21 @@ class PageTest < Test::Unit::TestCase
   end
   
   def test_saves_default_title_based_on_page_kind_when_title_is_place_holder
-    common_page = Page.new
+    common_page = Page.new.revise(bob, now, :title => PLACE_HOLDER_TITLE.t)
     
-    common_page.save
     assert_equal 'Common page', common_page.title
     
-    feature_page = Page.new(:kind => 'feature')
-    feature_page.save
+    feature_page = revise_brand_new_page(:kind => 'feature')
     assert_equal 'Feature page', feature_page.title
   end
   
   def test_numbers_titles_when_already_used
-    Page.new.save
-    
-    snd_page = Page.new
-    snd_page.save
+    revise_brand_new_page({})
     
+    snd_page = revise_brand_new_page({})
     assert_equal 'Common page 2', snd_page.title
     
-    trd_page = Page.new
-    trd_page.save       
-
+    trd_page = revise_brand_new_page({})
     assert_equal 'Common page 3', trd_page.title
   end
   
@@ -142,4 +137,131 @@ class PageTest < Test::Unit::TestCase
     assert_equal 'Yet another motiro page', page.title
   end
   
+  def test_revising_new_page_should_record_original_author_id_and_editors
+    john = users(:john)
+    time = Time.local(2007, 3, 15, 17, 3, 22)
+    page = Page.new(:name => 'BrandNew').revise(john, time,
+                                                :kind => 'feature',
+                                                :title => 'Brand new page',
+                                                :text => 'This is a new page',
+                                                :editors => 'john bob eric')
+                 
+    assert_equal john, page.last_editor
+    assert_equal john, page.original_author
+    assert_equal time, page.modified_at
+    assert_equal 'feature', page.kind
+    assert_equal 'Brand new page', page.title
+    assert_equal 'This is a new page', page.text
+    assert_equal 'john bob eric', page.editors
+  end
+  
+  def test_revising_old_page_should_not_record_original_author_id
+    john = users('john')
+    time = Time.local(2007, 3, 15, 17, 3, 22)
+    page = pages(:bob_and_erics_page).revise(john, time,
+                                             :title => "Bob and Eric's page",
+                                             :text => 'Do not touch')
+                 
+    assert_equal john, page.last_editor
+    assert_equal users('bob'), page.original_author
+    assert_equal time, page.modified_at
+    assert_equal "Bob and Eric's page", page.title
+    assert_equal 'Do not touch', page.text
+  end
+  
+  def test_only_original_author_should_be_able_to_change_editors_list
+    john = users('john')
+    editors = pages(:bob_and_erics_page).editors 
+    page = pages(:bob_and_erics_page).revise(john, Time.local(2007, 3, 15, 17),
+                                             :title => "Bob and Eric's page",
+                                             :text => 'Do not touch',
+                                             :editors => 'john lennon')
+                 
+    assert_not_equal 'john lennon', editors
+    assert_equal editors, page.editors
+  end
+  
+  def test_keeps_revision_record
+    page = create_page_with_one_revision
+    assert_equal 1, page.revisions.size
+
+    page.revise(users('john'), Time.local(2007, 3, 15, 10, 19, 36),
+                :title => 'Revised page',
+                :text => 'Page revision number 2',
+                :editors => '')
+    
+    assert_equal 2, page.revisions.size
+    assert_equal 'Page revision number 2', page.revisions.first.text
+    assert_equal 'Page revision number 1', page.revisions.last.text
+  end
+  
+  def test_copies_previous_editors_list_when_not_provided
+    page = create_page_with_one_revision
+    previous_editors = page.editors
+    
+    page.revise(users('bob'), Time.now, :title => page.title,
+                                        :text => 'Bob was here')
+    
+    assert_equal previous_editors, page.editors
+    assert_equal previous_editors, page.revisions.first.editors
+    assert_equal previous_editors, page.revisions.last.editors
+  end
+  
+  def test_most_recent_revision_comes_first
+    page = pages('changed_page')
+    assert_equal revisions('page_creation'), page.revisions[1]
+    assert_equal revisions('page_edition'), page.revisions[0]
+  end
+  
+  def test_records_original_author_for_pages_without_author
+    page = pages('nobodys_page')
+    bob = users('bob')
+    assert_nil page.original_author
+    
+    page.revise(bob, now, :title => page.title, :text => 'Bob was here')
+                                        
+    assert_equal bob, page.original_author                      
+  end
+  
+  def test_copies_previous_kind_when_not_provided
+    page = pages('list_last_modified_features_page')
+    
+    page.revise(bob, now, :text => 'New feature page text')
+    
+    assert_equal 'feature', page.kind
+  end
+  
+  def test_stores_revisions_for_happens_at
+    event = pages('release_event')
+    old_time = event.happens_at
+    new_time = Date.new(2007, 4, 4)
+    
+    event.revise(bob, now, :happens_at => new_time)
+    
+    assert_equal new_time, event.happens_at
+    assert_equal new_time, event.revisions.first.happens_at
+    assert_equal old_time, event.revisions.last.happens_at
+  end
+  
+  def test_rbab
+    date = Date.new(2007, 4, 4)
+    page = revise_brand_new_page(:kind => 'event',
+                                 :text => 'Something will happen',
+                                 :happens_at => date)
+    
+    assert_equal date, page.happens_at
+  end
+  
+  #TODO it seems that event pages do not get their happens_at attribute revised
+  
+private
+  
+  def create_page_with_one_revision
+    Page.new(:name => 'RevisedPage').revise(
+      users('john'), Time.local(2007, 3, 15, 9, 15, 53),
+      :title => 'Revised page',
+      :text => 'Page revision number 1',
+      :editors => '')
+  end
+
 end

data/test/unit/revision_test.rb

@@ -0,0 +1,32 @@
+#  Motiro - A project tracking tool
+#  Copyright (C) 2006-2007  Thiago Arrais
+#  
+#  This program is free software; you can redistribute it and/or modify
+#  it under the terms of the GNU General Public License as published by
+#  the Free Software Foundation; either version 2 of the License, or
+#  any later version.
+#  
+#  This program is distributed in the hope that it will be useful,
+#  but WITHOUT ANY WARRANTY; without even the implied warranty of
+#  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#  GNU General Public License for more details.
+#  
+#  You should have received a copy of the GNU General Public License
+#  along with this program; if not, write to the Free Software
+#  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+require File.dirname(__FILE__) + '/../test_helper'
+
+class RevisionTest < Test::Unit::TestCase
+  fixtures :users
+
+  def test_revisions_original_author_is_page_original_author
+    p = revise_brand_new_page(:title => 'My Title', :text => 'Original content')
+    p.revise(john, now, :text => 'Modified content')
+    
+    assert_equal bob, p.original_author
+    assert_equal bob, p.revisions.first.original_author
+    assert_equal bob, p.revisions.last.original_author
+  end
+
+end

data/test/unit/user_test.rb

@@ -107,9 +107,9 @@ class UserTest < Test::Unit::TestCase
     u = User.new(:login => 'john')
     attrs = { :name => 'TestPage', :text => '' }
 
-    assert  u.can_edit?(Page.new(attrs.merge(:editors => 'john')))
-    assert !u.can_edit?(Page.new(attrs.merge(:editors => 'eric')))
-    assert  u.can_edit?(Page.new(attrs.merge(:editors => '  ')))
+    assert  u.can_edit?(revise_brand_new_page(:editors => 'john'))
+    assert !u.can_edit?(revise_brand_new_page(:editors => 'eric'))
+    assert  u.can_edit?(revise_brand_new_page(:editors => '  '))
   end
   
   def test_original_author_can_always_edit_page
@@ -117,7 +117,7 @@ class UserTest < Test::Unit::TestCase
     john = users('john')
     bobs_page = pages('bob_and_erics_page')
     
-    bobs_page.editors = 'john'
+    bobs_page.revise(bob, now, :editors => 'john')
     assert john.can_edit?(bobs_page)
     assert bob.can_edit?(bobs_page)
   end

data/test/unit/wiki_reporter_test.rb

@@ -35,6 +35,8 @@ require File.dirname(__FILE__) + '/../test_helper'
 
 class WikiReporterTest < Test::Unit::TestCase
 
+  fixtures :users
+  
   def button; MotiroReporter.button; end
 
   def test_title
@@ -68,9 +70,7 @@ class WikiReporterTest < Test::Unit::TestCase
       mod_time = Time.local(2007, 1, 19, 15, 23, 8)
       page_provider.should_receive(:find).
         zero_or_more_times.
-        and_return([Page.new(:name => 'FeaturePage',
-                             :last_editor => User.new(:login => 'arrais'),
-                             :modified_at => mod_time,
+        and_return([Page.new(:name => 'FeaturePage').revise(bob, mod_time, 
                              :title => 'Feature page')])
       hl = FeaturesReporter.new(:page_provider => page_provider).headlines[0]
       assert_equal mod_time, hl.happened_at
@@ -82,10 +82,9 @@ class WikiReporterTest < Test::Unit::TestCase
       planned_date = Date.civil(2007, 1, 19)
       page_provider.should_receive(:find).
         zero_or_more_times.
-        and_return([Page.new(:name => 'FeaturePage',
-                             :last_editor => User.new(:login => 'arrais'),
+        and_return([Page.new(:name => 'EventPage').revise(bob, now,
                              :happens_at => planned_date,
-                             :title => 'Feature page')])
+                             :title => 'Event page')])
       hl = EventsReporter.new(:page_provider => page_provider).headlines[0]
       assert_equal planned_date.to_t, hl.happened_at
     end