Page MenuHomePhabricator

In the Herald test console, don't consider transactions that Herald rules applied
ClosedPublic

Authored by epriestley on May 22 2019, 9:48 PM.
Tags
None
Referenced Files
F14080939: D20547.diff
Fri, Nov 22, 5:31 PM
Unknown Object (File)
Mon, Nov 18, 2:14 AM
Unknown Object (File)
Thu, Nov 14, 12:55 PM
Unknown Object (File)
Wed, Nov 13, 5:24 AM
Unknown Object (File)
Mon, Nov 11, 6:26 PM
Unknown Object (File)
Sun, Nov 10, 9:22 AM
Unknown Object (File)
Tue, Nov 5, 8:39 PM
Unknown Object (File)
Tue, Oct 29, 9:46 AM
Subscribers
None

Details

Summary

Depends on D20546. Ref T13283. Currently, if you do something (transactions "A", "B") and Herald does some things in response (transaction "C"), Herald acts only on the things you did ("A", "B") since the thing it did ("C") didn't exist yet, until it ran.

However, if you use the test console to test rules against the object we'll pick up all three transactions since they're all part of the same group. This isn't ideal.

To fix this, skip transactions which Herald applied, since it obviously didn't consider them when it was evaluating.

Test Plan
  • Created a revision, in the presence of a Herald rule that adds reviewers.
  • Then, ran the revision through the test console.
  • Before: saw the "Herald added reviewers: ..." transaction in the transaction group Herald evaluated.
  • After: saw only authentic human transactions.

Screen Shot 2019-05-22 at 2.44.45 PM.png (340×1 px, 69 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable