Page MenuHomePhabricator

When running unit tests, ignore any custom task fields
ClosedPublic

Authored by epriestley on Apr 2 2017, 3:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 19 2026, 9:05 AM
Unknown Object (File)
Mar 15 2026, 2:30 AM
Unknown Object (File)
Feb 19 2026, 9:27 PM
Unknown Object (File)
Nov 29 2025, 12:06 AM
Unknown Object (File)
Nov 25 2025, 5:10 PM
Unknown Object (File)
Oct 23 2025, 8:52 PM
Unknown Object (File)
Oct 23 2025, 1:01 AM
Unknown Object (File)
Oct 22 2025, 9:41 AM
Subscribers
None

Details

Summary

If you have maniphest.custom-field-definitions set to include "required" fields, a bunch of tests which create tasks can fail.

To avoid this, reset this config while running tests.

This mechanism should probably be more general (e.g., reset all config by default, only whitelist some config) but just fix this for now since it's a one-liner and doesn't make eventual cleanup any harder.

Test Plan

Ran arc unit, hitting tests that create tasks.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 2 2017, 4:13 PM
This revision was automatically updated to reflect the committed changes.