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
F19035481: D17595.id.diff
Tue, Nov 25, 5:10 PM
F18825293: D17595.id.diff
Oct 23 2025, 8:52 PM
F18821647: D17595.diff
Oct 23 2025, 1:01 AM
F18819796: D17595.diff
Oct 22 2025, 9:41 AM
F18508678: D17595.id.diff
Sep 5 2025, 2:57 AM
F18500784: D17595.diff
Sep 4 2025, 9:20 PM
F18229919: D17595.diff
Aug 20 2025, 11:44 AM
F18081468: D17595.id42330.diff
Aug 5 2025, 12:28 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.