The test seems correct. There seems to be something fairly odd going
on with respect to versioning; for example,
ContentManagerTest.deleteAllPages() fails because it deletes all pages
ok, but at the last instant it recreates the pages as empty.
My guess is that it was a bug in ContentManager that priha 0.6.x
revealed; and I also have a nagging suspicion it has something to do
with ReferenceManager, as it seems to me that the pages are recreated
by the event handlers as a side effect. This could happen if the page
is first deleted, then save()d, then a event handler adds a property
to it and recreates the Node as a sanity check or something. Haven't
yet had time to peek into it very deeply.
/Janne
On Dec 1, 2009, at 15:44 , Andrew Jaquith wrote:
Janne, can I get a quick sanity check on something?
// Save two versions of the test page
m_engine.saveText( "Test", "This is the first version" );
m_engine.saveText( "Test", "This is the second version" );
// Make sure they both saved ok
WikiPage v1 = m_engine.getPage( "Test", 1 );
WikiPage v2 = m_engine.getPage( "Test", 2 );
assertNotNull( "Did not save page Test, v1!", v1 );
assertNotNull( "Did not save page Test, v2!", v2 );
assertEquals( "This is the first version",
m_engine.getPureText( v1 ).trim() );
The last assertion is failing. When WikiEngine.getPureText(v1) is
called, it is returning "This is the second version", which seems
wrong . It should return the first version, right?
Back when the trunk tests all ran clean (aka "the good old days"),
this test worked also. So it seems to me that some recent changes
caused the results to change. So was the test wrong all along, or is
there a bug somewhere?
Andrew
