diff --git a/tests/ConsumerTest.php b/tests/ConsumerTest.php index 642ae5f95..4f63fa269 100644 --- a/tests/ConsumerTest.php +++ b/tests/ConsumerTest.php @@ -96,10 +96,19 @@ protected function tearDown() { } protected function deleteTestActivities() { - $query = DB::prepare('DELETE FROM `*PREFIX*activity` WHERE `app` = ?'); - $query->execute(array('test')); - $query = DB::prepare('DELETE FROM `*PREFIX*activity_mq` WHERE `amq_appid` = ?'); - $query->execute(array('test')); + $query = \OC::$server->getDatabaseConnection()->getQueryBuilder(); + $query->delete('activity') + ->where($query->expr()->eq( + 'app', $query->createNamedParameter('test') + )); + $query->execute(); + + $query = \OC::$server->getDatabaseConnection()->getQueryBuilder(); + $query->delete('activity_mq') + ->where($query->expr()->eq( + 'amq_appid', $query->createNamedParameter('test') + )); + $query->execute(); } public function receiveData() { diff --git a/tests/MailQueueHandlerTest.php b/tests/MailQueueHandlerTest.php index a4b5f1def..98f4f1333 100644 --- a/tests/MailQueueHandlerTest.php +++ b/tests/MailQueueHandlerTest.php @@ -80,6 +80,9 @@ class MailQueueHandlerTest extends TestCase { /** @var ILogger|\PHPUnit_Framework_MockObject_MockObject */ protected $logger; + /** @var IDateTimeFormatter|\PHPUnit_Framework_MockObject_MockObject */ + protected $dateTimeFormatter; + protected function setUp() { parent::setUp(); @@ -89,6 +92,7 @@ protected function setUp() { $this->legacyParser = $this->createMock(LegacyParser::class); $this->config = $this->createMock(IConfig::class); $this->logger = $this->createMock(ILogger::class); + $this->dateTimeFormatter = $this->createMock(IDateTimeFormatter::class); $connection = \OC::$server->getDatabaseConnection(); $query = $connection->prepare('INSERT INTO `*PREFIX*activity_mq` ' @@ -118,6 +122,15 @@ protected function setUp() { $event->expects($this->any()) ->method('setSubject') ->willReturnSelf(); + $event->expects($this->any()) + ->method('getIcon') + ->willReturn(''); + $event->expects($this->any()) + ->method('getParsedSubject') + ->willReturn(''); + $event->expects($this->any()) + ->method('getRichSubject') + ->willReturn(''); $event->expects($this->any()) ->method('getRichSubjectParameters') ->willReturn([]); @@ -145,7 +158,7 @@ protected function setUp() { ->method('createMessage') ->willReturn($this->message); $this->mailQueueHandler = new MailQueueHandler( - $this->createMock(IDateTimeFormatter::class), + $this->dateTimeFormatter, $connection, $this->dataHelper, $this->mailer, @@ -281,6 +294,10 @@ public function testSendEmailToUser() { [null] ); + $this->dateTimeFormatter->expects($this->any()) + ->method('formatDateTimeRelativeDay') + ->willReturn('relative'); + $users = self::invokePrivate($this->mailQueueHandler, 'getAffectedUsers', [1, $maxTime, false, null]); $this->assertEquals([$user], $users); self::invokePrivate($this->mailQueueHandler, 'sendEmailToUser', [$user, $email, 'en', 'UTC', $maxTime]);