Page MenuHomePhabricator

Add a test to verify that all routing maps are plausibly valid, and remove some dead routes
ClosedPublic

Authored by epriestley on Feb 14 2020, 10:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 28, 8:25 AM
Unknown Object (File)
Sat, Mar 23, 12:49 PM
Unknown Object (File)
Thu, Mar 21, 11:17 PM
Unknown Object (File)
Thu, Mar 14, 6:06 PM
Unknown Object (File)
Thu, Mar 14, 6:06 PM
Unknown Object (File)
Thu, Mar 7, 7:38 AM
Unknown Object (File)
Feb 19 2024, 8:52 PM
Unknown Object (File)
Feb 17 2024, 1:21 PM
Subscribers
Restricted Owners Package

Details

Summary

Previously, see D20999. See also https://discourse.phabricator-community.org/t/the-phutil-library-phutil-has-not-been-loaded/3543/.

There are a couple of dead "Config" routes after recent changes. Add test coverage to make sure routes all point somewhere valid, then remove all the dead routes that turned up.

Test Plan

Ran tests, saw failures. Removed dead routes, got clean tests.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Feb 14 2020, 10:39 PM

Specifically, all of the removed rules point at controllers which no longer exist. Most of these were Preview or Edit controllers which are now handled by EditEngine or some other piece of centralized infrastructure.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 15 2020, 2:06 AM
This revision was automatically updated to reflect the committed changes.