Page MenuHomePhabricator

ReadingLists: Improve mocha test coverage
Closed, ResolvedPublic

Description

Enpoints:

  • GET /readinglists/v0/lists/pages: finding lists that contain a given page. Should be tested after creating two lists and putting page A in the first list, page B in the second, and C in both. Shoudl also check what happens when asking for page D.
  • GET /readinglists/v0/lists/{id}/entries: list entries in the given list. Test paging!
  • DELETE /readinglists/v0/lists/{id}/entries: remove entries from the list. Check be listing entries remaining in the list.
  • POST /readinglists/v0/lists/{id}/entries/batch create a batch of list entries

Event Timeline

FGoodwin changed the task status from Open to In Progress.Apr 25 2024, 2:27 PM

Change #1028869 had a related patch set uploaded (by Fgoodwin; author: Fgoodwin):

[mediawiki/extensions/ReadingLists@master] WIP: Mocha tests on ReadingLists

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

Change #1029277 had a related patch set uploaded (by Fgoodwin; author: Fgoodwin):

[mediawiki/extensions/ReadingLists@master] WIP: Add tests for lists/entries

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

Change #1031053 had a related patch set uploaded (by Fgoodwin; author: Fgoodwin):

[mediawiki/extensions/ReadingLists@master] WIP: Additional tests

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

FGoodwin moved this task from In Progress to Done on the MW-Interfaces-Team board.

Change #1031053 merged by jenkins-bot:

[mediawiki/extensions/ReadingLists@master] Additional tests

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

Change #1028869 abandoned by Fgoodwin:

[mediawiki/extensions/ReadingLists@master] WIP: Mocha tests on ReadingLists

Reason:

Child change merged

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

Change #1029277 abandoned by Fgoodwin:

[mediawiki/extensions/ReadingLists@master] WIP: Add tests for lists/entries

Reason:

Child change was merged

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