jenkins-bot has submitted this change and it was merged.

Change subject: Added a quick test for getAllQueuedJobs().
......................................................................


Added a quick test for getAllQueuedJobs().

Change-Id: Ia39461bc1e2356da1a694bf1898b8ca8e4200ff8
---
M tests/phpunit/includes/jobqueue/JobQueueTest.php
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Demon: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/phpunit/includes/jobqueue/JobQueueTest.php 
b/tests/phpunit/includes/jobqueue/JobQueueTest.php
index a905f06..820f8c5 100644
--- a/tests/phpunit/includes/jobqueue/JobQueueTest.php
+++ b/tests/phpunit/includes/jobqueue/JobQueueTest.php
@@ -106,6 +106,9 @@
                $queue->flushCaches();
                $this->assertEquals( 2, $queue->getSize(), "Queue size is 
correct ($desc)" );
                $this->assertEquals( 0, $queue->getAcquiredCount(), "No jobs 
active ($desc)" );
+               $jobs = iterator_to_array( $queue->getAllQueuedJobs() );
+               $this->assertEquals( 2, count( $jobs ), "Queue iterator size is 
correct ($desc)" );
+
 
                $job1 = $queue->pop();
                $this->assertFalse( $queue->isEmpty(), "Queue is not empty 
($desc)" );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia39461bc1e2356da1a694bf1898b8ca8e4200ff8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Demon <[email protected]>
Gerrit-Reviewer: IAlex <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to