Page MenuHomePhabricator

phabricator: remove tests and all recordings
ClosedPublic

Authored by Kwan on Oct 10 2019, 5:52 PM.

Details

Summary

The next commit is going to change the format of conduit API requests so none
of the VCR recordings will match and all the tests will fail.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Kwan created this revision.Oct 10 2019, 5:52 PM
This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.
dlax added a subscriber: dlax.Nov 22 2019, 5:09 AM

The next commit is going to change the format of conduit API requests so none of the VCR recordings will match and all the tests will fail.

@Kwan, it seems to me that there is no longer any recording data nor actual test from this changeset. Did you plan to add them back? (At the moment, it seems to me that test-phabricator.t tests nothing.)