Page MenuHomePhabricator

[Wilds] Rename "formatters" to "sinks" and restore the console output sufficiently to see which tests are failing
ClosedPublic

Authored by epriestley on Sep 26 2018, 4:07 PM.
Tags
None
Referenced Files
F13043967: D19711.diff
Wed, Apr 17, 3:01 PM
Unknown Object (File)
Fri, Apr 12, 3:03 PM
Unknown Object (File)
Sat, Mar 30, 6:52 AM
Unknown Object (File)
Sat, Mar 30, 6:51 AM
Unknown Object (File)
Sat, Mar 30, 6:51 AM
Unknown Object (File)
Fri, Mar 29, 1:52 AM
Unknown Object (File)
Mar 14 2024, 5:03 PM
Unknown Object (File)
Mar 4 2024, 8:28 PM
Subscribers
None

Details

Summary

Ref T13098. Since I plan to implement "send the results to Harbormaster" as another type of formatter/output/sink, just rename the objects which receive unit test results and print/write/transmit them into "Sinks" (in the sense of Source/Sink).

Get the default console sink working well enough to see what's failing. As with all other changes in this series this is very rough, but the general idea is that I want to:

  • Let sinks stream both ongoing status information and final results.
  • For the console output, try to increase the signal-to-noise ratio of the output stream. Today, it's too easy to lose a failed test in the results. I want to improve this by outputting less frequently and summarizing passes ("93 tests passed.") so that the streaming output mostly shows failures and it's easier to make a decision to ^C and revise if you see something you don't like.
  • Also, add a summary mode at the end which makes sure failures show up on the console and aren't scrolled up 30 pages. For now, this is quite rough.
Test Plan
373 PASSED * 16 SKIPPED * 162 FAILED/BROKEN/UNSTABLE

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.
amckinley added inline comments.
src/unit/sink/ArcanistDefaultUnitSink.php
7

I slightly prefer SINK_KEY here. SINKKEY has kind of a nautical disaster flavor to it, don't you think?

This revision is now accepted and ready to land.Sep 26 2018, 9:35 PM
epriestley added inline comments.
src/unit/sink/ArcanistDefaultUnitSink.php
7

This is for consistentency with a lot of existing TYPECONST, EDGECONST, ENGINEKEY stuff, although I agree that SINKKEY reads pretty weirdly.

This revision was automatically updated to reflect the committed changes.