Page MenuHomePhabricator
Create Task
Maniphest T109670

ContentTranslation is not running PHPUnit structure tests
Closed, ResolvedPublic1 Estimated Story Points
Actions

Description

Would have prevented T109659: paraminfo for contenttranslationlist is broken on beta wikis from happening. UBN because not having this has caused UBNs.

Details

SubjectRepoBranchLines +/-
Add Wikidata as a dependency for ContentTranslationintegration/configmaster+2 -1
Add PHPUnit tests for ContentTranslationintegration/configmaster+2 -0
Customize query in gerrit

Related Objects

Event Timeline

Legoktm created this task.Aug 20 2015, 12:57 AM
Legoktm raised the priority of this task from to Unbreak Now!.
Legoktm updated the task description. (Show Details)
Legoktm added projects: ContentTranslation, Continuous-Integration-Config.
Legoktm subscribed.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 20 2015, 12:57 AM
Legoktm mentioned this in T109659: paraminfo for contenttranslationlist is broken on beta wikis.Aug 20 2015, 12:58 AM
jayvdb subscribed.Aug 20 2015, 1:00 AM
Ricordisamoa subscribed.Aug 20 2015, 5:16 AM
santhosh subscribed.Aug 20 2015, 7:07 AM
Arrbee added projects: LE-CX6-Sprint 2, ContentTranslation-Release6.Aug 20 2015, 7:10 AM
Arrbee set Security to None.
santhosh assigned this task to KartikMistry.Aug 20 2015, 7:23 AM
Amire80 moved this task from Backlog to Tech Debt on the ContentTranslation-Release6 board.Aug 20 2015, 7:40 AM
Amire80 moved this task from Needs Triage to CX6 on the ContentTranslation board.
Arrbee lowered the priority of this task from Unbreak Now! to High.Aug 20 2015, 8:23 AM
Arrbee raised the priority of this task from High to Unbreak Now!.
Arrbee moved this task from Backlog to In Progress on the LE-CX6-Sprint 2 board.
KartikMistry added a comment.EditedAug 20 2015, 2:29 PM

Any hints on what is missing in zuul/layout.yaml would be appreciated.

gerritbot subscribed.Aug 20 2015, 4:15 PM

Change 232749 had a related patch set uploaded (by KartikMistry):
Add extension-gate checks for ContentTranslation

https://gerrit.wikimedia.org/r/232749

gerritbot added a project: Patch-For-Review.Aug 20 2015, 4:15 PM
KartikMistry moved this task from In Progress to In Review on the LE-CX6-Sprint 2 board.Aug 20 2015, 4:28 PM
gerritbot added a comment.Aug 20 2015, 6:26 PM

Change 232749 merged by jenkins-bot:
Add PHPUnit tests for ContentTranslation

https://gerrit.wikimedia.org/r/232749

Legoktm mentioned this in rCICFe73123ad3221: Add PHPUnit tests for ContentTranslation.Aug 20 2015, 6:27 PM
Legoktm added a comment.Aug 20 2015, 6:33 PM

Tests enabled. Currently failing (https://integration.wikimedia.org/ci/job/mwext-testextension-zend/6817/console) due to:

18:28:33 1) ResourcesTest::testMissingDependencies
18:28:33 The module 'wikibase.api.RepoApi' required by 'ext.cx.wikibase.link' must exist
18:28:33 Failed asserting that an array contains 'wikibase.api.RepoApi'.

Is Wikibase a dependency? It's not listed on https://www.mediawiki.org/wiki/Extension:ContentTranslation#Extension_dependencies

I just copied the dependencies from the qunit job, which is EventLogging and ULS.

gerritbot added a comment.Aug 20 2015, 8:26 PM

Change 232837 had a related patch set uploaded (by Legoktm):
Add Wikidata as a dependency for ContentTranslation

https://gerrit.wikimedia.org/r/232837

gerritbot added a comment.Aug 20 2015, 8:30 PM

Change 232837 merged by jenkins-bot:
Add Wikidata as a dependency for ContentTranslation

https://gerrit.wikimedia.org/r/232837

Legoktm mentioned this in rCICF401abaa8ceb2: Add Wikidata as a dependency for ContentTranslation.Aug 20 2015, 8:30 PM
Legoktm closed this task as Resolved.Aug 20 2015, 8:58 PM

And rCICF313a86320764: ContentTranslation depends upon Echo & GuidedTour too.

KartikMistry moved this task from In Review to Done on the LE-CX6-Sprint 2 board.Aug 21 2015, 3:59 AM
Amire80 edited a custom field.Aug 25 2015, 5:04 PM
Amire80 mentioned this in T110518: ContentTranslation tests for mwext-testextension-zend tests fails in wmf19 and wmf20.Aug 27 2015, 1:20 PM
Arrbee added a project: Essential-Work.Sep 7 2015, 11:40 AM
• Petar.petkovic removed a project: Patch-For-Review.Sep 25 2018, 7:27 PM
• Petar.petkovic removed a subscriber: gerritbot.
Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptSep 25 2018, 7:27 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL