Disable notifications while running unit tests
ClosedPublic
Actions

Authored by epriestley on Aug 6 2014, 10:26 PM.

Tags

None

Referenced Files

	F19044726: D10171.id.diff
	Thu, Nov 27, 4:11 AM

	F19039712: D10171.diff
	Wed, Nov 26, 8:52 AM

	F18840810: D10171.id24466.diff
	Oct 28 2025, 4:03 AM

	F18840809: D10171.id24467.diff
	Oct 28 2025, 4:03 AM

	F18837338: D10171.id.diff
	Oct 27 2025, 4:57 AM

	F18834386: D10171.diff
	Oct 26 2025, 8:26 AM

	F18672661: D10171.id24466.diff
	Sep 25 2025, 8:02 AM

	F18652549: D10171.diff
	Sep 21 2025, 8:32 AM

View All Files

Subscribers

epriestley

Tokens

"Mountain of Wealth" token, awarded by joshuaspence.

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP1e1eb6921b1d: 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.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24466.Aug 6 2014, 10:26 PM

epriestley retitled this revision from to Disable notifications while running unit tests.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 6 2014, 10:26 PM

btrahan accepted this revision.Aug 6 2014, 10:27 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 6 2014, 10:27 PM

Harbormaster completed remote builds in B2071: Diff 24466.Aug 6 2014, 10:27 PM

Closed by commit rP1e1eb6921b1d (authored by @epriestley).

joshuaspence awarded a token.Aug 6 2014, 10:38 PM

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

1 line

infrastructure/

testing/

PhabricatorTestCase.php

6 lines

Diff 24467

View Options

src/__phutil_library_map__.php