Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix ci #103

Merged
merged 7 commits into from
Feb 25, 2025
Merged

Fix ci #103

merged 7 commits into from
Feb 25, 2025

Conversation

paladox
Copy link
Contributor

@paladox paladox commented Feb 23, 2025

No description provided.

@paladox paladox marked this pull request as draft February 23, 2025 20:12
@paladox
Copy link
Contributor Author

paladox commented Feb 23, 2025

I dunno how to fix the rest of ci.

it's failing with:

There were 4 failures:

1) SIL\Tests\Category\LanguageFilterCategoryViewerTest::testAddSubcategoryForNoInterlanguageLinksLookup
Failed asserting that a NULL is not empty.

/var/www/html/extensions/SemanticInterlanguageLinks/tests/phpunit/Unit/Category/LanguageFilterCategoryViewerTest.php:143

2) SIL\Tests\InterlanguageLinksLookupTest::testRedirectTargetFor
Failed asserting that two objects are equal.
--- Expected
+++ Actual
@@ @@
     'mTitleValue' => null
     'mIsBigDeletion' => null
     'mIsValid' => true
-    'mInstanceCacheKey' => 'SIL\Tests\InterlanguageLinksL...getFor'
+    'mInstanceCacheKey' => null
     'deprecatedPublicProperties' => Array (...)
     'dynamicPropertiesAccessDeprecated' => false
 )

/var/www/html/extensions/SemanticInterlanguageLinks/tests/phpunit/Unit/InterlanguageLinksLookupTest.php:67

3) SIL\Tests\InterlanguageListParserFunctionTest::testParseForValidLanguageTargetLinks with data set #2 (array(Title Object (...)), '{{FakeTemplate|#=0|target-lin...简体)‎}}')
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    0 => '{{FakeTemplate|#=0|target-link=分类:汉字|lang-code=zh-Hans|lang-name=中文(简体)‎}}'
+    0 => '{{FakeTemplate|#=0|target-link=分类:汉字|lang-code=zh-Hans|lang-name=中文(简体)}}'
     'noparse' => false
     'isHTML' => false
 )

/var/www/html/extensions/SemanticInterlanguageLinks/tests/phpunit/Unit/InterlanguageListParserFunctionTest.php:105

4) SIL\Tests\InterlanguageListParserFunctionTest::testParseForValidLanguageTargetLinks with data set #3 (array(Title Object (...)), '{{FakeTemplate|#=0|target-lin...简体)‎}}')
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    0 => '{{FakeTemplate|#=0|target-link=:Category:汉字|lang-code=zh-Hans|lang-name=中文(简体)‎}}'
+    0 => '{{FakeTemplate|#=0|target-link=:Category:汉字|lang-code=zh-Hans|lang-name=中文(简体)}}'
     'noparse' => false
     'isHTML' => false
 )

/var/www/html/extensions/SemanticInterlanguageLinks/tests/phpunit/Unit/InterlanguageListParserFunctionTest.php:105

I spent ages looking and couldn't figure it out.

Any ideas?

@paladox paladox marked this pull request as ready for review February 24, 2025 18:54
Copy link

codecov bot commented Feb 24, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@paladox
Copy link
Contributor Author

paladox commented Feb 24, 2025

Marked MW 1.43 support as experimental.

fails under MW 1.43. This I think is due to the behaviour change of
foreach ( $parserOutput->getLanguageLinks() as $languageLink ) {
in MW 1.43.

But I don't know how to fix that. So for now this pull is ok to merge as it adds support for MW 1.39-MW 1.42.

@jaideraf jaideraf merged commit b396c95 into master Feb 25, 2025
6 of 7 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants