Page MenuHomePhabricator

Make Herald test workflow modular and more clear
ClosedPublic

Authored by epriestley on Aug 2 2016, 3:31 PM.
Tags
None
Referenced Files
F19505888: D16360.id39365.diff
Fri, Jan 9, 2:54 PM
F19499130: D16360.id39350.diff
Thu, Jan 8, 1:36 PM
F19107173: D16360.diff
Dec 7 2025, 9:00 PM
F19072428: D16360.diff
Nov 30 2025, 10:31 PM
F19056737: D16360.diff
Nov 28 2025, 8:39 PM
F18845899: D16360.id39350.diff
Oct 29 2025, 2:59 PM
F18828127: D16360.diff
Oct 24 2025, 2:28 PM
F18823714: D16360.diff
Oct 23 2025, 1:02 PM
Subscribers
None

Details

Summary

Fixes T9719. Currently, the Herald "Test Console" has a big instanceof thing, so new adapters (like a Calendar adapter, or third-party adapters) aren't available automatically. Instead, do a standard modular thing: load the available adapters, ask which ones can test the object the user selected, then let the user pick which one they want to move forward with.

Additionally, it isn't very clear that you can't test "commit hook" rules because they rely on push state which we don't really have a good way to simulate. When the user picks a commit, we now show them the "Hook" events, but the options are disabled and explain why they can not be selected.

Test Plan
  • Ran test rules for revisions, commits, mocks, tasks, wiki documents, questions, and outbound mail.
  • Plugged in a commit, got a more-helpful choice screen explaining why you do a test run of hook rules.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable