ApiLogout: Follow up Icb674095
authorBrad Jorsch <bjorsch@wikimedia.org>
Thu, 25 Apr 2019 13:49:01 +0000 (09:49 -0400)
committerBrad Jorsch <bjorsch@wikimedia.org>
Thu, 25 Apr 2019 14:00:24 +0000 (10:00 -0400)
commit8e3bcb67f486defd13e0e6885e7641acd7620dc0
treefd780af2f1456d2a635a11f1fedd1fc1a6c63054
parent9045e4b17ffc1023564008053554312b706888af
ApiLogout: Follow up Icb674095

This implements getWebUITokenSalt(), as mentioned in T25227#2008199 and
implemented in F3328897. Somehow it didn't make it into Icb674095.

This also fixes some issues in the unit test:
* Properly link the user to the request's Session so User::doLogout()
  won't log a warning. This also gives use to the otherwise-unneeded
  implementation of setUp(), and lets us get rid of the broken call to
  User::newFromId() that was passing an IP address rather than a user ID.
* Privatize some internal methods.
* Use setExpectedApiException() instead of manually catching and
  hard-coding the English exception message.
* Also assert that the bad token error didn't result in a logout.

Bug: T25227
Change-Id: I2aecfba821cca3c367c5e7e8d188a88197fb82d2
includes/api/ApiLogout.php
tests/phpunit/includes/api/ApiLogoutTest.php