Merge "Add tests for BaseMappings"

src/WikiTexVC/MMLmappings/BaseMappings.php

@@ -778,19 +778,15 @@ class BaseMappings {
 		// Just an empty private constructor, for singleton pattern
 	}
 
-	public static function removeInstance() {
+	public static function getAll(): array {
-		self::$instance = null;
+		return self::ALL;
 	}
 
-	public static function getInstance() {
+	public static function getInstance(): BaseMappings {
 		self::$instance ??= new BaseMappings();
 		return self::$instance;
 	}
 
-	public static function getEntryFromList( $keylist, $key ) {
-		return self::ALL[$keylist][$key] ?? null;
-	}
-
 	public static function getOperatorByKey( $key ) {
 		if ( $key === '-' ) {
 			return MMLutil::uc2xNotation( '\u2212' ); // added this additionally for running all tests

tests/phpunit/unit/WikiTexVC/MMLmappings/BaseMappingsTest.php

@@ -0,0 +1,100 @@
+<?php
+
+namespace MediaWiki\Extension\Math\Tests\WikiTexVC\MMLmappings;
+
+use MediaWiki\Extension\Math\WikiTexVC\MMLmappings\BaseMappings;
+use MediaWiki\Extension\Math\WikiTexVC\MMLmappings\BaseParsing;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @covers \MediaWiki\Extension\Math\WikiTexVC\MMLmappings\BaseMappings
+ */
+class BaseMappingsTest extends TestCase {
+
+	public function testGetAll() {
+		$all = BaseMappings::getAll();
+		$this->assertIsArray( $all );
+		$this->assertNotEmpty( $all );
+	}
+
+	public function provideTestCases(): array {
+		// the second argument is an array of known problems which should be removed in the future
+		return [
+			'macros' => [ 'macros', [ 'SetSize', 'Overunderset', 'Root', 'MoveRoot', 'LeftRight', 'MoveLeftRight',
+				'rule', 'Rule', 'Nonscript', 'BuildRel', 'FBox', 'FrameBox', 'Strut', 'Cr', 'HFill', 'BeginEnd',
+				'HandleLabel', 'HandleRef', 'HandleNoTag', 'MmlToken' ]
+			],
+			'cancel' => [ 'cancel' ],
+			'mhchem' => [ 'mhchem' ],
+			'custom' => [ 'custom', [ 'Insert' ] ],
+			'special' => [ 'special', [ 'open', 'close', 'superscript', 'subscript', 'space', 'prime', 'comment',
+				'entry', 'hash' ] ], // only tilde exists
+		];
+	}
+
+	/**
+	 * @dataProvider provideTestCases
+	 */
+	public function testValidMethods( $setName, $knownProblems = [] ) {
+		foreach ( BaseMappings::getAll()[$setName] as $symbol => $payload ) {
+			$methodName = is_array( $payload ) ? $payload[0] : $payload;
+			if ( in_array( $methodName, $knownProblems ) ) {
+				continue;
+			}
+			$this->assertTrue( method_exists( BaseParsing::class, $methodName ),
+				'Method ' . $methodName . ' for symbol ' . $symbol . ' does not exist in BaseParsing' );
+
+		}
+	}
+
+	public function testGetCancelByKey() {
+		$this->assertEquals( 'updiagonalstrike', BaseMappings::getCancelByKey( '\\cancel' )[1] );
+	}
+
+	public function testGetOperatorByKey() {
+		$this->assertEquals( '&#x221A;', BaseMappings::getOperatorByKey( '\\surd' )[0] );
+		$this->assertEquals( '&#x2212;', BaseMappings::getOperatorByKey( '-' ) );
+	}
+
+	public function testGetCustomByKey() {
+		$this->assertEquals( '\u222E', BaseMappings::getCustomByKey( '\\oint' )[1] );
+	}
+
+	public function testGetDelimiterByKey() {
+		$this->assertEquals( '(', BaseMappings::getDelimiterByKey( '(' )[0] );
+	}
+
+	public function testGetMacroByKey() {
+		$this->assertEquals( 'D', BaseMappings::getMacroByKey( '\\displaystyle' )[1] );
+	}
+
+	public function testGetSpecialByKey() {
+		$this->assertEquals( 'tilde', BaseMappings::getSpecialByKey( '~' )[1] );
+		$this->assertNull( BaseMappings::getSpecialByKey( '_' ) );
+	}
+
+	public function testGetColorByKey() {
+		$this->assertEquals( '#ED1B23', BaseMappings::getColorByKey( 'red' )[0] );
+	}
+
+	public function testGetInstance() {
+		$this->assertInstanceOf( BaseMappings::class, BaseMappings::getInstance() );
+	}
+
+	public function testGetNullaryMacro() {
+		$this->assertEquals( '&#x00C5;', BaseMappings::getNullaryMacro( 'AA' )[0] );
+	}
+
+	public function testGetCharacterByKey() {
+		$this->assertEquals( '\u0393', BaseMappings::getCharacterByKey( '\\Gamma' ) );
+	}
+
+	public function testGetMhChemByKey() {
+		$this->assertEquals( 'ce', BaseMappings::getMhChemByKey( '\\ce' )[1] );
+	}
+
+	public function testGetIdentifierByKey() {
+		$this->assertEquals( '&#x03B1;', BaseMappings::getIdentifierByKey( '\\alpha' )[0] );
+	}
+
+}