Physikerwelt has uploaded a new change for review.
https://gerrit.wikimedia.org/r/185675
Change subject: Remove workaround for missing dependence to Extension:Math
......................................................................
Remove workaround for missing dependence to Extension:Math
The workaround that creates a dummy class for the case that the
Math extsion has not been loaded by Jenkins is no longer required.
Change-Id: Ifd8a9b97d5c8982f99f3cd29d0b8cf161c0ead14
---
M tests/MathObjectTest.php
1 file changed, 0 insertions(+), 14 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MathSearch
refs/changes/75/185675/1
diff --git a/tests/MathObjectTest.php b/tests/MathObjectTest.php
index 46e5ece..1b8cda3 100644
--- a/tests/MathObjectTest.php
+++ b/tests/MathObjectTest.php
@@ -1,15 +1,4 @@
<?php
-//TODO: fix that ugly workaround
-/*
- * If the unit tests are run by jenkins, the Math extension is not present.
- * Thus a fake MathMathML class must be created in order to derive the
- * MathObject class that extends MathMathML
- */
-function createDummyMathMLClass(){
- class MathMathML{}
- return true;
-}
-
/**
* Test the mathObject script.
*
@@ -37,9 +26,6 @@
public function test() {
- if ( ! class_exists("MathMathML") ){
- createDummyMathMLClass();
- }
$comment = MathObject::extractMathTagsFromWikiText(
$this->HTMLComment );
$this->assertEquals( 0, sizeof( $comment ) );
$noWiki = MathObject::extractMathTagsFromWikiText(
$this->noWiki );
--
To view, visit https://gerrit.wikimedia.org/r/185675
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifd8a9b97d5c8982f99f3cd29d0b8cf161c0ead14
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MathSearch
Gerrit-Branch: master
Gerrit-Owner: Physikerwelt <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
