Unbreak TalkPageFunctionalTest

* Clear echo_event before the test. Otherwise it contains
  a 'thank-you-edit' event that offsets everything.
* Use standard $this->db
* Use standard ~~~~ signature

Bug: T67336
Change-Id: Idd09aa107d01038d03b689d9086f65a9fb7b653a
This commit is contained in:
Stephane Bisson 2019-04-19 14:05:57 -04:00
parent 5796d46565
commit 3a7e655229

View file

@ -7,37 +7,24 @@
*/ */
class EchoTalkPageFunctionalTest extends ApiTestCase { class EchoTalkPageFunctionalTest extends ApiTestCase {
/**
* @var \Wikimedia\Rdbms\IDatabase
*/
protected $dbr;
protected function setUp() { protected function setUp() {
parent::setUp(); parent::setUp();
$this->dbr = MWEchoDbFactory::getDB( DB_REPLICA ); $this->db->delete( 'echo_event', '*' );
} }
/** /**
* Creates and updates a user talk page a few times to ensure proper events are * Creates and updates a user talk page a few times to ensure proper events are
* created. The user performing the edits is self::$users['sysop']. * created. The user performing the edits is self::$users['sysop'].
* @group Broken
* @covers \EchoDiscussionParser * @covers \EchoDiscussionParser
*/ */
public function testAddCommentsToTalkPage() { public function testAddCommentsToTalkPage() {
$editor = self::$users['sysop']->getUser()->getName();
$talkPage = self::$users['uploader']->getUser()->getName(); $talkPage = self::$users['uploader']->getUser()->getName();
// A set of messages which will be inserted
$messages = [
'Moar Cowbell',
"I can haz test\n\nplz?", // checks that the parser allows multi-line comments
'blah blah',
];
$messageCount = 0; $messageCount = 0;
$this->assertCount( $messageCount, $this->fetchAllEvents() ); $this->assertCount( $messageCount, $this->fetchAllEvents() );
// Start a talkpage // Start a talkpage
$content = "== Section 8 ==\n\n" . $this->signedMessage( $editor, $messages[$messageCount] ); $content = "== Section 8 ==\n\nblah blah ~~~~\n";
$this->editPage( $talkPage, $content, '', NS_USER_TALK ); $this->editPage( $talkPage, $content, '', NS_USER_TALK );
// Ensure the proper event was created // Ensure the proper event was created
@ -49,7 +36,7 @@ class EchoTalkPageFunctionalTest extends ApiTestCase {
// Add another message to the talk page // Add another message to the talk page
$messageCount++; $messageCount++;
$content .= $this->signedMessage( $editor, $messages[$messageCount] ); $content .= "More content ~~~~\n";
$this->editPage( $talkPage, $content, '', NS_USER_TALK ); $this->editPage( $talkPage, $content, '', NS_USER_TALK );
// Ensure another event was created // Ensure another event was created
@ -61,7 +48,7 @@ class EchoTalkPageFunctionalTest extends ApiTestCase {
// Add a new section and a message within it // Add a new section and a message within it
$messageCount++; $messageCount++;
$content .= "\n\n== EE ==\n\n" . $this->signedMessage( $editor, $messages[$messageCount] ); $content .= "\n\n== EE ==\n\nhere we go with a new section ~~~~\n";
$this->editPage( $talkPage, $content, '', NS_USER_TALK ); $this->editPage( $talkPage, $content, '', NS_USER_TALK );
// Ensure this event has the new section title // Ensure this event has the new section title
@ -83,13 +70,9 @@ class EchoTalkPageFunctionalTest extends ApiTestCase {
* @return \stdClass[] All events in db sorted from oldest to newest * @return \stdClass[] All events in db sorted from oldest to newest
*/ */
protected function fetchAllEvents() { protected function fetchAllEvents() {
$res = $this->dbr->select( 'echo_event', EchoEvent::selectFields(), [], __METHOD__, [ 'ORDER BY' => 'event_id ASC' ] ); $res = $this->db->select( 'echo_event', EchoEvent::selectFields(), [], __METHOD__, [ 'ORDER BY' => 'event_id ASC' ] );
return iterator_to_array( $res ); return iterator_to_array( $res );
} }
protected function signedMessage( $name, $content = 'Moar cowbell', $depth = 1 ) {
return str_repeat( ':', $depth ) . " $content [[User:$name|$name]] ([[User talk:$name|$name]]) 00:17, 7 May 2013 (UTC)\n";
}
} }