Commit graph

27 commits

Author SHA1 Message Date
Ori Livneh 9cb10b050e Update test for API change in PS10 of I17ef1f5197
Use $this->getMutableTestUser() instead of $this->getTestUser( true ).

Change-Id: Idb41b4842d0c95801dcd5476fa2f3e45153f9c71
2016-05-23 20:16:08 -07:00
Ori Livneh 111f48f982 Update tests for I17ef1f519
This is a prerequisite for I17ef1f519, since the tests break otherwise. The
change is backward-compatible since it checks for the presence of the
getTestUser method introduced by that change.

Change-Id: I8bbc9ebf32a89157fa9c60fd33151a10e7598af8
2016-05-24 02:38:12 +00:00
addshore 5c0b21593f Make revisions in setUp for ApiRevThankIntTest
Bug: T134582
Change-Id: I5317616f99f1344557fd818d0b00073a82197d05
2016-05-07 11:54:04 +00:00
addshore 6d470cad44 Rename ApiRevThankTest to reflect file name
Change-Id: I4b99e42999416643e17a8bcebe7bf00e70ca18ef
2016-05-06 15:23:30 +01:00
James D. Forrester 03b8f88fff build: Add doxygen, use composer for phpcs, make pass
Change-Id: I939d6f4d989cf21c0f817d262bf3e4acedcf2ffe
2016-04-25 15:42:04 +00:00
Kunal Mehta 9aa92841a6 Fix ApiRevThankTest::testValidRequest()
Bug: T130973
Change-Id: I3fc094fa0aeb7d5928fd8033bc3eefbceea72e16
2016-03-29 15:12:49 -07:00
addshore 57236b1809 @author Addshore everywhere for me
Change-Id: I424b8ba94a46d1ad430914aaf8d883af0df712a1
2016-01-27 11:06:17 +01:00
Glaisher 703642e40e Hide thanks button for botedits if bots are not allowed to receive thanks in mobile as well
Also only load 'ext.thanks.mobilediff' module if the recipient is not an IP.

Bug: T118686
Change-Id: I096d3b05a46590db426ae038f17f6e180fa5bf37
2015-11-24 22:47:56 +05:00
addshore 9b864432fd Remove use of deprecated TestUser->user
Change-Id: I10f1809b08a6c39685dc3af8088019483f307e6d
2015-08-11 01:27:50 +00:00
Matthias Mullie ddf1310eea Flow container needs to be reset before each test
Pimple throws an exception when overriding something that was
already fetched from container.
This is currently causing some extensions trouble.

To prevent shared state between tests the flow container 
needs to be reset in the setUp() routine of all tests
accesing the container.

Bug: T91967
Change-Id: Ic80b137e6ca2f8ba6ad34ce0bdbb1319b81c0a6e
2015-03-09 17:27:01 +00:00
jdlrobson e02c334880 Add tests for rendering of thanks button on mobile diff page
This has broken many times and I want this test to ensure its protection
in future.

Bug: T86687
Change-Id: Iea01c5c7c1ba00de47182521eeba86c8aef03522
2015-01-28 12:48:08 -08:00
Brad Jorsch a0e4ccfa30 Add i18n for API module help
MediaWiki core change I04b1a384 added support for i18n of API module
help. This takes advantage of that while still maintaining backwards
compatibility with earlier versions of MediaWiki.

Once support for MediaWiki before 1.25 is dropped, the methods marked
deprecated in this patch may be removed.

Change-Id: I67395aff48185f3e09da31b51a08aa2541fe6a17
2014-11-03 13:32:23 -08:00
jenkins-bot 160b5a3aab Merge "Repair thanks tests" 2014-09-18 23:44:34 +00:00
Erik Bernhardson a22b23287f Repair thanks tests
Change-Id: Ia0b34ece38ae219e5ee2c49b3d98490a503f536b
2014-09-16 13:27:58 -07:00
Matthias Mullie 2395bffdf9 Use board Title instead of topic Title
Since we've recently introduced a topic namespace, the notification looked like:
"Username thanked you for your comment in "Topic title" on Topic:UUID"

We want the board name instead of the link to the topic page, like:
"Username thanked you for your comment in "Topic title" on Talk:Boardname"

Change-Id: If45c01be3b0a9d67572932253a0103a2480fc823
2014-08-20 12:42:32 +02:00
wctaiwan dfb639af3e Call getAlphadecimal() directly
Relying on UUID::__toString() now triggers a notice, causing
unrelated patches to fail. This patch addresses the issue.

Change-Id: Ia94826a0e600adf8afd5908af75772b596ab8d0a
2014-04-15 03:54:00 +00:00
bsitu d9def310b3 Fix unit test in ApiFlowThankIntegrationTest.php
Looks like this is the only spot to fix for rev_type_id

Change-Id: Ie89b7d8090ca1c759f320d33d407fccdc7eb66c1
2014-04-15 01:44:51 +00:00
Erik Bernhardson 486ada0c56 PHPUnit mock builder doesnt accept leading \
Change-Id: Icab8d2475009604dd5b71ad816b5f959a8c7cc34
2014-04-08 00:01:54 +00:00
Benjamin Chen fbaed178a5 Add integration test for flowthank API
The test suite sets up various mocks for Flow classes and inserts
them into Flow\Container, so the API module retrieves the correct
mock classes instead.

Change-Id: I9c5e910e288f3dd72c66e68ca558265b8ca8ce5e
2014-03-14 19:16:11 +08:00
wctaiwan 98726b568b Add "Thanks" for comments on Flow boards
bug: 61930
Change-Id: Id37a14e3c75c63419fc34e0b7c2e21f74b3fa875
Co-authored-by: Bencmq <bencmqwiki@gmail.com>
2014-03-14 11:03:11 +00:00
addshore 4cdfa4e1f8 Add basic unit test for dieOnBadUser api method
Change-Id: Ic9fd4e63b0a7e3508c600f34decb8ce18abc1597
2014-01-24 13:13:25 -08:00
jenkins-bot 31437cf89d Merge "Add Special:Thanks as a non-JS fallback" 2013-12-19 05:43:11 +00:00
jenkins-bot 3573062156 Merge "Thanks is not using namespaces" 2013-12-06 08:00:55 +00:00
Kunal Mehta 06360b6d0d Add Special:Thanks as a non-JS fallback
If the user has JavaScript disabled, clicking
on "thank" will send the user to Special:Thanks
with the revision id already filled in. When
submitted, the form calls the API internally
and then shows the user an error or confirmation
message.

Special:Thanks is not a listed special page
since it's mainly intended as a fallback.

The API was modified to return the user name of the
editor who is being thanked if a new notification
was created.

Bug: 49161
Change-Id: I7ba4664b92bb0da425784350487ed0e7ca352b4e
2013-12-05 23:23:21 -08:00
theopolisme 72c0bf5d52 Perform checks before thanking via API
If the user attempts to thank an anonymous
user, a bot (when $wgThanksSendToBots is not true),
or him/herself, return an error. This is consistent
with the restrictions imposed in insertThankLink().

Bug: 56954
Change-Id: I9e8ae030a8ff1cc312bf98671e593c7367ca0ea2
2013-11-24 17:41:21 -06:00
Kunal Mehta 6240343103 Thanks is not using namespaces
Added in 422f0f6062

Change-Id: I8809348d30fdd8b569affd00cc061de22b38e6dc
2013-11-22 23:25:25 -08:00
addshore 422f0f6062 Add really basic API tests
This also includes the unit test hooks

Jenkings job will be added with I6ec2edf62b9

Change-Id: Id085db7096db420416e9fba72a65586d021f1c76
2013-11-04 16:38:23 +00:00