Ryan10145 has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/404062 )

Change subject: Unit test NewsletterDb::getNewsletter
......................................................................

Unit test NewsletterDb::getNewsletter

Bug: T183810
Change-Id: I59d331ddda77d527496c2d61d52c785b1ac3ebbf
---
M tests/NewsletterDbTest.php
1 file changed, 37 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Newsletter 
refs/changes/62/404062/1

diff --git a/tests/NewsletterDbTest.php b/tests/NewsletterDbTest.php
index 007e23c..44d9012 100644
--- a/tests/NewsletterDbTest.php
+++ b/tests/NewsletterDbTest.php
@@ -394,6 +394,43 @@
        }
 
        /**
+        * @covers NewsletterDb::getNewsletter
+        */
+       public function testGetNewsletter() {
+               $mockWriteDb = $this->getMockIDatabase();
+               $newsletter = $this->getTestNewsletter();
+
+               $mockResWrapper = $this->getMockBuilder( 'ResultWrapper' )
+                       ->disableOriginalConstructor()
+                       ->getMock();
+               $mockResWrapper->expects( $this->once() )
+                       ->method( 'current' )
+                       ->will( $this->returnValue(
+                               (Object)[
+                                       'nl_id' => $newsletter->getId(),
+                                       'nl_name' => $newsletter->getName(),
+                                       'nl_desc' => 
$newsletter->getDescription(),
+                                       'nl_main_page_id' => 
$newsletter->getPageId(),
+                               ]
+                       ) );
+
+               $mockWriteDb
+                       ->expects( $this->once() )
+                       ->method( 'select' )
+                       ->with(
+                               'nl_newsletters',
+                               [ 'nl_id', 'nl_name', 'nl_desc', 
'nl_main_page_id' ],
+                               [ 'nl_id' => $newsletter->getId(), 'nl_active' 
=> 1 ]
+                       )
+                       ->will( $this->returnValue( $mockResWrapper ) );
+
+               $table = new NewsletterDb( $this->getMockLoadBalancer( 
$mockWriteDb ) );
+
+               $result = $table->getNewsletter( $newsletter->getId() );
+               $this->assertEquals( $newsletter, $result );
+       }
+
+       /**
         * @covers NewsletterDb::getNewsletterFromName
         */
        public function testGetNewsletterFromName() {

-- 
To view, visit https://gerrit.wikimedia.org/r/404062
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I59d331ddda77d527496c2d61d52c785b1ac3ebbf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Newsletter
Gerrit-Branch: master
Gerrit-Owner: Ryan10145 <chang.ryan10...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to