ApiLogout: Follow up Icb674095
authorBrad Jorsch <bjorsch@wikimedia.org>
Thu, 25 Apr 2019 13:49:01 +0000 (09:49 -0400)
committerReedy <reedy@wikimedia.org>
Thu, 25 Apr 2019 14:50:14 +0000 (15:50 +0100)
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


No differences found