X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=blobdiff_plain;f=tests%2Fphpunit%2Fincludes%2Fapi%2FApiTestCase.php;h=a5ee7ddb8c9f4c84ab83bd4273524e1f2072d727;hp=8e53b55940134ac648c28135959b5228a380f3ef;hb=54d50ef3921dc9f30dc4d863ddc471dc564998e9;hpb=767042c3e68adb29513f9983c385a1e63110279f

diff --git a/tests/phpunit/includes/api/ApiTestCase.php b/tests/phpunit/includes/api/ApiTestCase.php
index 8e53b55940..a5ee7ddb8c 100644
--- a/tests/phpunit/includes/api/ApiTestCase.php
+++ b/tests/phpunit/includes/api/ApiTestCase.php
@@ -56,6 +56,28 @@ abstract class ApiTestCase extends MediaWikiLangTestCase {
 		return $page->doEditContent( ContentHandler::makeContent( $text, $title ), $summary );
 	}
 
+	/**
+	 * Revision-deletes a revision.
+	 *
+	 * @param Revision|int $rev Revision to delete
+	 * @param array $value Keys are Revision::DELETED_* flags.  Values are 1 to set the bit, 0 to
+	 *   clear, -1 to leave alone.  (All other values also clear the bit.)
+	 * @param string $comment Deletion comment
+	 */
+	protected function revisionDelete(
+		$rev, array $value = [ Revision::DELETED_TEXT => 1 ], $comment = ''
+	) {
+		if ( is_int( $rev ) ) {
+			$rev = Revision::newFromId( $rev );
+		}
+		RevisionDeleter::createList(
+			'revision', RequestContext::getMain(), $rev->getTitle(), [ $rev->getId() ]
+		)->setVisibility( [
+			'value' => $value,
+			'comment' => $comment,
+		] );
+	}
+
 	/**
 	 * Does the API request and returns the result.
 	 *
@@ -72,6 +94,7 @@ abstract class ApiTestCase extends MediaWikiLangTestCase {
 	 * @param string|null $tokenType Set to a string like 'csrf' to send an
 	 *   appropriate token
 	 *
+	 * @throws ApiUsageException
 	 * @return array
 	 */
 	protected function doApiRequest( array $params, array $session = null,
@@ -98,6 +121,10 @@ abstract class ApiTestCase extends MediaWikiLangTestCase {
 		}
 
 		if ( $tokenType !== null ) {
+			if ( $tokenType === 'auto' ) {
+				$tokenType = ( new ApiMain() )->getModuleManager()
+					->getModule( $params['action'], 'action' )->needsToken();
+			}
 			$params['token'] = ApiQueryTokens::getToken(
 				$wgUser, $sessionObj, ApiQueryTokens::getTokenTypeSalts()[$tokenType]
 			)->toString();
@@ -141,45 +168,34 @@ abstract class ApiTestCase extends MediaWikiLangTestCase {
 	 * @return array Result of the API call
 	 */
 	protected function doApiRequestWithToken( array $params, array $session = null,
-		User $user = null, $tokenType = 'csrf'
+		User $user = null, $tokenType = 'auto'
 	) {
 		return $this->doApiRequest( $params, $session, false, $user, $tokenType );
 	}
 
-	protected function doLogin( $testUser = 'sysop' ) {
+	/**
+	 * Previously this would do API requests to log in, as well as setting $wgUser and the request
+	 * context's user.  The API requests are unnecessary, and the global-setting is unwanted, so
+	 * this method should not be called.  Instead, pass appropriate User values directly to
+	 * functions that need them.  For functions that still rely on $wgUser, set that directly.  If
+	 * you just want to log in the test sysop user, don't do anything -- that's the default.
+	 *
+	 * @param TestUser|string $testUser Object, or key to self::$users such as 'sysop' or 'uploader'
+	 * @deprecated since 1.31
+	 */
+	protected function doLogin( $testUser = null ) {
+		global $wgUser;
+
 		if ( $testUser === null ) {
 			$testUser = static::getTestSysop();
 		} elseif ( is_string( $testUser ) && array_key_exists( $testUser, self::$users ) ) {
-			$testUser = self::$users[ $testUser ];
+			$testUser = self::$users[$testUser];
 		} elseif ( !$testUser instanceof TestUser ) {
-			throw new MWException( "Can not log in to undefined user $testUser" );
+			throw new MWException( "Can't log in to undefined user $testUser" );
 		}
 
-		$data = $this->doApiRequest( [
-			'action' => 'login',
-			'lgname' => $testUser->getUser()->getName(),
-			'lgpassword' => $testUser->getPassword() ] );
-
-		$token = $data[0]['login']['token'];
-
-		$data = $this->doApiRequest(
-			[
-				'action' => 'login',
-				'lgtoken' => $token,
-				'lgname' => $testUser->getUser()->getName(),
-				'lgpassword' => $testUser->getPassword(),
-			],
-			$data[2]
-		);
-
-		if ( $data[0]['login']['result'] === 'Success' ) {
-			// DWIM
-			global $wgUser;
-			$wgUser = $testUser->getUser();
-			RequestContext::getMain()->setUser( $wgUser );
-		}
-
-		return $data;
+		$wgUser = $testUser->getUser();
+		RequestContext::getMain()->setUser( $wgUser );
 	}
 
 	protected function getTokenList( TestUser $user, $session = null ) {