Last modified: 2013-11-05 22:18:18 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T45899, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 43899 - DataValues test failures
DataValues test failures
Status: RESOLVED WORKSFORME
Product: MediaWiki extensions
Classification: Unclassified
DataValues (Other open bugs)
master
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on: 37209 44280 44307
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-12 14:48 UTC by Niklas Laxström
Modified: 2013-11-05 22:18 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Niklas Laxström 2013-01-12 14:48:33 UTC
Warning: Missing argument 1 for DataValues\BooleanValue::__construct() in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/BooleanValue.php on line 50
Notice: Undefined variable: value in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/BooleanValue.php on line 51

Warning: Missing argument 2 for DataValues\GeoCoordinateValue::__construct() in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/GeoCoordinateValue.php on line 82
Notice: Undefined variable: latitude in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/GeoCoordinateValue.php on line 83
Notice: Undefined variable: longitude in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/GeoCoordinateValue.php on line 87
Notice: Undefined variable: latitude in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/GeoCoordinateValue.php on line 95

Warning: Missing argument 1 for DataValues\IriValue::__construct() in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/IriValue.php on line 94
Warning: Missing argument 2 for DataValues\IriValue::__construct() in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/IriValue.php on line 94
Notice: Undefined variable: scheme in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/IriValue.php on line 101
Notice: Undefined variable: hierarchicalPart in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/IriValue.php on line 105
Notice: Undefined variable: scheme in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/IriValue.php on line 109
Notice: Undefined variable: hierarchicalPart in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/IriValue.php on line 110

Catchable fatal error: Argument 1 passed to DataValues\MediaWikiTitleValue::__construct() must be an instance of Title, none given in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/MediaWikiTitleValue.php on line 50
Comment 1 Jeroen De Dauw 2013-01-18 11:20:44 UTC
Steps to reproduce? I am not getting any errors when running the tests, and your report does not include the info of where the incorrect calls are happening.
Comment 2 Niklas Laxström 2013-01-18 15:11:54 UTC
Just running make safe for code unit tests. This is all the info unit tests give.
Comment 3 Daniel Kinzler 2013-01-20 11:34:00 UTC
Can't reproduce. Niklas, it would be very useful if you could provide stack traces for these issues.
Comment 4 Niklas Laxström 2013-01-20 11:38:44 UTC
It would be very helpful if PHP would provide stack traces for these issues.
Comment 5 Niklas Laxström 2013-01-20 12:47:22 UTC
Give me some time to provide more information. You do realize that the other option is to drop DataValues and everything that depends on it from translatewiki.net. I doubt that is the kind of publicity you want.
Comment 6 Jeroen De Dauw 2013-01-20 13:49:23 UTC
This is occurring on translatewiki? I was assuming on your local dev box since no other info is specified.

> It would be very helpful if PHP would provide stack traces for these issues.

Do you have xdebug installed and enabled?

Also, are you running the latest version of DataValues and the extensions using it?
Comment 7 Niklas Laxström 2013-01-20 14:27:08 UTC
No I don't have xdebug on production (the dev runs on the same server on the same environment).

This seems to be the instance of the same issue we had earlier: some other test messages up with PHP's error handling making these tests fail.
Comment 8 Jeroen De Dauw 2013-01-20 14:39:17 UTC
Can you link to the mentioned issue?
Comment 9 Niklas Laxström 2013-01-20 14:48:33 UTC
You should remember the array key exists issue bug earlier.

I've found at least two tests which cause this issue by selectively running tests:

(2175) WikitextContentTest::testGetSecondaryDataUpdates: E
(2176) WikitextContentTest::testGetSecondaryDataUpdates: E.....
(4900) DataValues\Test\IriValueTest::testMediaWikiTestCaseParentSetupCalled: .
(4901) DataValues\Test\MediaWikiTitleValueTest::testGetValue: .
(4902) DataValues\Test\MediaWikiTitleValueTest::testGetValue: .
(4903) DataValues\Test\MediaWikiTitleValueTest::testConstructor: 
Catchable fatal error: Argument 1 passed to DataValues\MediaWikiTitleValue::__construct() must be an instance of Title, none given in /www/dev.translatewiki.net/w/extensions/DataValues/DataValues/includes/values/MediaWikiTitleValue.php on line 50
Comment 10 Niklas Laxström 2013-01-20 14:50:42 UTC
Hilarious, if I run those two tests alone I get:

1) WikitextContentTest::testGetSecondaryDataUpdates with data set #0 ('WikitextContentTest_testGetSecondaryDataUpdates_1', 'wikitext', 'hello \'\'world\'\'
', array(array(true, array())))
MWException: LinkCache doesn't currently know about this title: WikitextContentTest_testGetSecondaryDataUpdates_1

/www/dev.translatewiki.net/w/includes/Title.php:2994
/www/dev.translatewiki.net/w/extensions/SemanticMediaWiki/includes/SMW_ParseData.php:245
/www/dev.translatewiki.net/w/extensions/SemanticMediaWiki/includes/SMW_ParseData.php:518
/www/dev.translatewiki.net/w/includes/Hooks.php:255
/www/dev.translatewiki.net/w/includes/GlobalFunctions.php:3875
/www/dev.translatewiki.net/w/includes/LinksUpdate.php:108
/www/dev.translatewiki.net/w/includes/parser/ParserOutput.php:458
/www/dev.translatewiki.net/w/includes/content/AbstractContent.php:243
/www/dev.translatewiki.net/w/tests/phpunit/includes/content/WikitextContentTest.php:77
/www/dev.translatewiki.net/w/tests/phpunit/MediaWikiTestCase.php:131
/www/dev.translatewiki.net/w/tests/phpunit/MediaWikiPHPUnitCommand.php:61
/www/dev.translatewiki.net/w/tests/phpunit/MediaWikiPHPUnitCommand.php:47

2) WikitextContentTest::testGetSecondaryDataUpdates with data set #1 ('WikitextContentTest_testGetSecondaryDataUpdates_2', 'wikitext', 'hello [[world test 21344]]
', array(array(true, array(array(0)))))
MWException: LinkCache doesn't currently know about this title: WikitextContentTest_testGetSecondaryDataUpdates_2

/www/dev.translatewiki.net/w/includes/Title.php:2994
/www/dev.translatewiki.net/w/extensions/SemanticMediaWiki/includes/SMW_ParseData.php:245
/www/dev.translatewiki.net/w/extensions/SemanticMediaWiki/includes/SMW_ParseData.php:518
/www/dev.translatewiki.net/w/includes/Hooks.php:255
/www/dev.translatewiki.net/w/includes/GlobalFunctions.php:3875
/www/dev.translatewiki.net/w/includes/LinksUpdate.php:108
/www/dev.translatewiki.net/w/includes/parser/ParserOutput.php:458
/www/dev.translatewiki.net/w/includes/content/AbstractContent.php:243
/www/dev.translatewiki.net/w/tests/phpunit/includes/content/WikitextContentTest.php:77
/www/dev.translatewiki.net/w/tests/phpunit/MediaWikiTestCase.php:131
/www/dev.translatewiki.net/w/tests/phpunit/MediaWikiPHPUnitCommand.php:61
/www/dev.translatewiki.net/w/tests/phpunit/MediaWikiPHPUnitCommand.php:47
Comment 11 Niklas Laxström 2013-01-20 14:58:59 UTC
I don't quite understand why this causes the rest of the tests to fail. If I comment out the exception the tests run fine again.
Comment 12 Jeroen De Dauw 2013-01-21 09:52:50 UTC
That's an exception in core - not DataValues
Comment 13 Niklas Laxström 2013-01-24 08:23:14 UTC
Okay, we got some other tests fixed, but these are still failing for me, needs some more debugging.
Comment 14 Lydia Pintscher 2013-08-26 10:47:23 UTC
What's the status here? Is this going anywhere or should we close it?
Comment 15 Niklas Laxström 2013-08-26 12:53:03 UTC
Hard to say. I'm not being able to run the tests very far at the moment because of bug 52229.
Comment 16 Lydia Pintscher 2013-11-05 22:18:06 UTC
Ok thanks. Then let's close this. It doesn't seem actionable for anyone at this point so no use in keeping it open to clutter up the bugtracker.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links