The RevisionSlider is an extension for MediaWiki that has just been deployed on all Wikipedias and other Wikimedia websites as a beta feature. The extension was developed by Wikimedia Germany as part of their focus on technical wishes of the German speaking Wikimedia community. This post will look at the RevisionSliders design, development and use so far.
Chrome Extension Generator
by yeoman is an npm package that can be used to very easily scaffold out a Chrome extension. Over the past year I have poked and prodded at a few chrome extensions, and ended up publishing one to display download counts on GitHub. I highly recommend this generator, it creates everything you need out of the box and also enables easy set-up of permissions, actions as well as auto rebuild and reload into Chrome for testing.
So it turns out the at() method doesn’t quite do what I had initially thought….
I have recently been working on some tests for the new Newsletter extension for Mediawiki, specifically to test the NewslettterTablePager class. This thing extends the TablePager class in Mediawiki which is designed to make displaying information from a database table on a special page on a mediawiki site easy, and also easily enable things such as sorting.
The code interacts with the database and gets a ResultWrapper object, and the Pager uses the numRows(), seek() and fetchObject() methods, all of which I thought would be incredibly simple to mock.
My first attempt where I first notice I have been thinking about the at() method all wrong can be seen below:
|
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 |
private function getMockDatabase( array $resultObjects ) { $mockResult = $this->getMock( 'ResultWrapper' ); $mockResult->expects( $this->atLeastOnce() ) ->method( 'numRows' ) ->will( $this->returnValue( count( $resultObjects ) ) ); $mockResult->expects( $this->any() ) ->method( 'seek' ); foreach ( $resultObjects as $index => $resultObject ) { $mockResult->expects( $this->at( $index ) ) ->method( 'fetchObject' ) ->will( $this->returnValue( $resultObject ) ); } $mockDb = $this->getMock( 'IDatabase' ); $mockDb->expects( $this->atLeastOnce() ) ->method( 'select' ) ->will( $this->returnValue( $mockResult ) ); return $mockDb; } |
This methods returns a mock Database that the Pager will use. As you can see the only parameter is an array of objects to be returned by fetchObject() and I am using the at() method provided by phpunit to return each object at the index that it is stored in the array. This is when I discovered that at() in phpunit does not work in the way I first thought…
at() refers to the index of calls made to the mocked object as a whole. This means that in the code sample above, all of the calles to numRows() and seek() are increasing the current call counter index for the object and thus my mocked fetchObject() method is never returning the correct value or returning null.
In my second attempt I made a guess that phpunit might allow multiple method mocks to stack and thus the return values of those methods be returned in the order that they were created. Thus I changed my loop to simply use any():
|
1 2 3 4 5 |
foreach ( $resultObjects as $index => $resultObject ) { $mockResult->expects( $this->any() ) ->method( 'fetchObject' ) ->will( $this->returnValue( $resultObject ) ); } |
But of course this also does not work and this result in the same $resultObject being returned for all calls.
I ended up having to to do something a little bit nasty (in my opinion) and use returnCallback() and use a private member of the testcase within the callback as a call counter / per method index:
|
1 2 3 4 5 6 7 8 |
$testcase = $this; $mockResult->expects( $this->any() ) ->method( 'fetchObject' ) ->will( $this->returnCallback( function () use ( $testCase, $resultObjects ) { $obj = $resultObjects[$testCase->mockSeekCounter]; $testCase->mockSeekCounter =+ 1; return $obj; } ) ); |
It would be great if phpunit would have some form of per method index expectation!
Some of the code examples here are slightly cut down, the full change can be seen at https://gerrit.wikimedia.org/r/#/c/234727/ in the NewsletterTablePagerTest file.
If only I had Googled the issue sooner! http://www.andrejfarkas.com/2012/07/phpunit-at-method-to-check-method-invocation-at-certain-index/
GitHub tracks the number of downloads for all assets (files) that are attached to a release, but GitHub currently makes it very hard for users to get at this information. The number of downloads is currently only accessible through the API.
I noticed this many months ago when wondering how many people were downloading the new C++ version of Huggle. At the time I started coming up with some odd little script that I could set running somewhere checking the download counts and updating a static list, I even thought about just uploading the build files to some other site that tracked this information.
A few days ago I took my first look at developing chrome extensions for a referencing tool for Wikidata, and thus today I thought, why not just make an extension for chrome that adds the download counts to the GitHub releases page!
© 2017 Addshore
Theme by Anders Noren — Up ↑
Recent Comments