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)
Sun, Apr 21, 11:50 PM
Unknown Object (File)
Wed, Apr 10, 8:15 AM
Unknown Object (File)
Sun, Apr 7, 10:36 AM
Unknown Object (File)
Feb 12 2024, 5:37 AM
Unknown Object (File)
Feb 9 2024, 10:38 PM
Unknown Object (File)
Jan 23 2024, 3:36 AM
Unknown Object (File)
Jan 21 2024, 10:47 PM
Unknown Object (File)
Nov 30 2023, 6:22 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.