Page MenuHomePhabricator

[VE WikifunctionsCall]: Add tests to VE integration
Closed, ResolvedPublic

Description

Description

Unit testing:

  • ext.wikilambda.app:
    • ext.wikilambda.app/mixins/api.js (with foreign api)
    • ext.wikilambda.app/store/stores/library.js (getDescription and lookupFunctions)
    • ext.wikilambda.app/store/stores/router.js (getWikilambdaConfig)
    • ext.wikilambda.app/store/stores/visualeditor.js
    • ext.wikilambda.app/components/visualeditor/FunctionCallSetup.vue
    • ext.wikilambda.app/components/visualeditor/FunctionSelect.vue
    • ext.wikilambda.app/components/visualeditor/FunctionInputSetup.vue
    • Remove ignore paths from jest.config.js
  • ext.wikilambda.visualeditor
    • Increase coverage threshold in jest.config.js

Integration testing:

  • ext.wikilambda.visualeditor

Selenium testing:

  • Visual Editor: add function call
  • Visual Editor: edit function call

Completion checklist

Event Timeline

Jdforrester-WMF renamed this task from [VE WikifunctionsCall]: Add tests to VE integraton to [VE WikifunctionsCall]: Add tests to VE integration.Mar 11 2025, 2:02 PM

Change #1129848 had a related patch set uploaded (by Daphne Smit; author: Daphne Smit):

[mediawiki/extensions/WikiLambda@master] VisualEditor: Add unit tests to VisualEditor integration

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

gengh changed the task status from Open to In Progress.Mar 25 2025, 3:33 PM

Change #1129848 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] VisualEditor: Add unit tests to VisualEditor integration

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

Change #1133805 had a related patch set uploaded (by Daphne Smit; author: Daphne Smit):

[mediawiki/extensions/WikiLambda@master] Add missing VisualEditor test for ExpandableDescription component

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

Change #1133805 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Add missing VisualEditor test for ExpandableDescription component

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

Jdforrester-WMF subscribed.

I think this is sufficiently done. Adding brower testing later is a nice-to-have, but doesn't block release.