mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/Math
synced 2024-11-23 23:25:02 +00:00
a429c8cc35
Actual changes: * Let the special page accept the subpage parameter, e.g. [[Special:MathWikibase/Q6203]]. * Fix some of the message escaping issues in SpecialMathWikibase. Messages that are just plain text and not supposed to contain any parameters, wikitext, HTML, or something like this should be properly escaped, either via Html::rawElement( $msg->escaped() ) or better Html::element( $msg->plain ). The later approach is cheaper. Other code cleanups that don't affect production: * Reduce the amount of duplicate code in SpecialMathWikibase. * Make use of the preg_match() return value. * Remove default parameters from parent::__construct() call. * Remove some auto-generated comments. * Remove some comments that don't add new information, but just repeat what the code already says. * Make use of PHPUnit's assertCount(). We can split this patch into smaller patches, if you prefer. This should especially be done if one of the changes is controversial, to not block the other changes. Please tell me. Change-Id: I8c20014435af3d2ac9c7381fe5fd00b9fe49b2c2
120 lines
3.9 KiB
PHP
120 lines
3.9 KiB
PHP
<?php
|
|
|
|
/**
|
|
* Test the database access and core functionality of MathRenderer.
|
|
*
|
|
* @covers \MathRenderer
|
|
*
|
|
* @group Math
|
|
* @group Database
|
|
*
|
|
* @license GPL-2.0-or-later
|
|
*/
|
|
class MathDatabaseTest extends MediaWikiTestCase {
|
|
/**
|
|
* @var MathRenderer
|
|
*/
|
|
private $renderer;
|
|
const SOME_TEX = "a+b";
|
|
const SOME_HTML = "a<sub>b</sub> and so on";
|
|
const SOME_MATHML = "iℏ∂_tΨ=H^Ψ<mrow><\ci>";
|
|
const SOME_CONSERVATIVENESS = 2;
|
|
const SOME_OUTPUTHASH = 'C65c884f742c8591808a121a828bc09f8<';
|
|
|
|
/**
|
|
* creates a new database connection and a new math renderer
|
|
* TODO: Check if there is a way to get database access without creating
|
|
* the connection to the database explicitly
|
|
* function addDBData() {
|
|
* $this->tablesUsed[] = 'math';
|
|
* }
|
|
* was not sufficient.
|
|
*/
|
|
protected function setUp() : void {
|
|
parent::setUp();
|
|
// TODO: figure out why this is necessary
|
|
$this->db = wfGetDB( DB_MASTER );
|
|
$this->renderer = new MathMathML( self::SOME_TEX );
|
|
$this->tablesUsed[] = 'mathoid';
|
|
}
|
|
|
|
/**
|
|
* Checks the tex and hash functions
|
|
* @covers \MathRenderer::getInputHash
|
|
*/
|
|
public function testInputHash() {
|
|
$expectedhash = $this->db->encodeBlob( pack( "H32", md5( self::SOME_TEX ) ) );
|
|
$this->assertEquals( $expectedhash, $this->renderer->getInputHash() );
|
|
}
|
|
|
|
/**
|
|
* Helper function to set the current state of the sample renderer instance to the test values
|
|
*/
|
|
public function setValues() {
|
|
// set some values
|
|
$this->renderer->setTex( self::SOME_TEX );
|
|
$this->renderer->setMathml( self::SOME_MATHML );
|
|
}
|
|
|
|
/**
|
|
* Checks database access. Writes an entry and reads it back.
|
|
* @covers \MathRenderer::writeToDatabase
|
|
* @covers \MathRenderer::readFromDatabase
|
|
*/
|
|
public function testDBBasics() {
|
|
$this->setValues();
|
|
$this->renderer->writeToDatabase( $this->db );
|
|
$renderer2 = new MathMathML( self::SOME_TEX );
|
|
$this->assertTrue( $renderer2->readFromDatabase(), 'Reading from database failed' );
|
|
// comparing the class object does now work due to null values etc.
|
|
$this->assertEquals(
|
|
$this->renderer->getTex(), $renderer2->getTex(), "test if tex is the same"
|
|
);
|
|
$this->assertEquals(
|
|
$this->renderer->getMathml(), $renderer2->getMathml(), "Check MathML encoding"
|
|
);
|
|
$this->assertEquals(
|
|
$this->renderer->getHtmlOutput(), $renderer2->getHtmlOutput(), 'test if HTML is the same'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Checks the creation of the math table.
|
|
* @covers \MathHooks::onLoadExtensionSchemaUpdates
|
|
*/
|
|
public function testCreateTable() {
|
|
$this->setMwGlobals( 'wgMathValidModes', [ 'mathml' ] );
|
|
$this->db->dropTable( "mathoid", __METHOD__ );
|
|
$dbu = DatabaseUpdater::newForDB( $this->db );
|
|
$dbu->doUpdates( [ "extensions" ] );
|
|
$this->expectOutputRegex( '/(.*)Creating mathoid table(.*)/' );
|
|
$this->setValues();
|
|
$this->renderer->writeToDatabase();
|
|
$res = $this->db->select( "mathoid", "*" );
|
|
$row = $res->fetchRow();
|
|
$this->assertCount( 16, $row );
|
|
}
|
|
|
|
/*
|
|
* This test checks if no additional write operation
|
|
* is performed, if the entry already existed in the database.
|
|
*/
|
|
public function testNoWrite() {
|
|
$this->setValues();
|
|
$inputHash = $this->renderer->getInputHash();
|
|
$this->assertTrue( $this->renderer->isChanged() );
|
|
$this->assertTrue( $this->renderer->writeCache(), "Write new entry" );
|
|
$res = $this->db->selectField( "mathoid", "math_inputhash",
|
|
[ "math_inputhash" => $inputHash ] );
|
|
$this->assertTrue( $res !== false, "Check database entry" );
|
|
$this->assertTrue( $this->renderer->readFromDatabase(), "Read entry from database" );
|
|
$this->assertFalse( $this->renderer->isChanged() );
|
|
// modify the database entry manually
|
|
$this->db->delete( "mathoid", [ "math_inputhash" => $inputHash ] );
|
|
// the renderer should not be aware of the modification and should not recreate the entry
|
|
$this->assertFalse( $this->renderer->writeCache() );
|
|
// as a result no entry can be found in the database.
|
|
$this->assertFalse( $this->renderer->readFromDatabase() );
|
|
}
|
|
}
|