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
F19057569: D20547.id49021.diff
Fri, Nov 28, 11:09 PM
F19054457: D20547.id.diff
Fri, Nov 28, 1:04 PM
F19049723: D20547.diff
Thu, Nov 27, 6:06 PM
F19049122: D20547.id.diff
Thu, Nov 27, 4:08 PM
F18991415: D20547.diff
Tue, Nov 18, 10:07 PM
F18835100: D20547.id.diff
Oct 26 2025, 1:21 PM
F18832005: D20547.diff
Oct 25 2025, 6:09 PM
F18813735: D20547.diff
Oct 20 2025, 6:48 PM
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