Page MenuHomePhabricator

Write unit tests for puppet rewrite.py changes
Open, Needs TriagePublic3 Estimated Story Points

Description

(Logging for visibility, and for potential further action i.r.t. documentation)

As mentioned in https://gerrit.wikimedia.org/r/c/operations/puppet/+/831955, we need to add unit tests to rewrite_integration_test.py:

  • "test that the mapping from public url to the internal one (i.e. /v1/AUTH_mw/<container>/<object path> is working as expected."

Documentation on rewrite.py (and it's associated tests) appears to be sparse

Event Timeline

@MusikAnimal: Removing task assignee as this open task has been assigned for more than two years - See the email sent to task assignee on October 11th.
Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be welcome! :)
If this task has been resolved in the meantime, or should not be worked on by anybody ("declined"), please update its task status via "Add Action… 🡒 Change Status".
Also see https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator. Thanks!