The goal of these tests is to catch regressions with the phrase matching approach. We'd probably want an example from an RTL language, an LTR language, an example where a partial phrase match exists e.g. the link should be "book" but the second instance of it exists (the first instance is in a phrase like "book fair").
Description
Description
Details
Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | KStoller-WMF | T276517 [EPIC] Growth: "add a link" structured task 3.0 | |||
| Resolved | kostajh | T276860 Review test coverage for add link feature | |||
| Resolved | kostajh | T285651 Add Qunit tests for AddLinkArticleTarget.prototype.annotateSuggestions |
Event Timeline
Comment Actions
Change 722361 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):
[mediawiki/extensions/GrowthExperiments@master] QUnit: Add basic test for AddLinkArticleTarget.annotateSuggestions()
Comment Actions
Change 722361 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] QUnit: Add basic test for AddLinkArticleTarget.annotateSuggestions()
Comment Actions
I'm resolving, as there are some more tests added in T283985: Add Link: word substrings are turned into links.