mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-24 06:20:56 +01:00
Disable notifications while running unit tests
Summary: Currently, we'll try to publish notifications while running tests. This is at best unnecessary and at worst problematic (we don't stub out the server). For now, just never publish them. Test Plan: Ran unit tests with notifications enabled but the server down and didn't get a bunch of warnings. Reviewers: btrahan Reviewed By: btrahan Subscribers: epriestley Differential Revision: https://secure.phabricator.com/D10171
This commit is contained in:
parent
1bd714f8ac
commit
1e1eb6921b
2 changed files with 7 additions and 0 deletions
|
@ -5055,6 +5055,7 @@ phutil_register_library_map(array(
|
|||
'PhabricatorFlaggableInterface',
|
||||
'PhabricatorTokenReceiverInterface',
|
||||
'PhabricatorProjectInterface',
|
||||
'PhabricatorDestructibleInterface',
|
||||
),
|
||||
'PhabricatorSlowvotePollController' => 'PhabricatorSlowvoteController',
|
||||
'PhabricatorSlowvotePollPHIDType' => 'PhabricatorPHIDType',
|
||||
|
|
|
@ -110,6 +110,12 @@ abstract class PhabricatorTestCase extends ArcanistPhutilTestCase {
|
|||
'phabricator.application-settings',
|
||||
array());
|
||||
|
||||
// We can't stub this service right now, and it's not generally useful
|
||||
// to publish notifications about test execution.
|
||||
$this->env->overrideEnvConfig(
|
||||
'notification.enabled',
|
||||
false);
|
||||
|
||||
// TODO: Remove this when we remove "releeph.installed".
|
||||
$this->env->overrideEnvConfig('releeph.installed', true);
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue