HomePhabricator

tests: fix 'if False' statements flagged by check-code

Authored by simpkins.

Description

tests: fix 'if False' statements flagged by check-code

Summary:
The check-code test in the upstream mercurial repository was recently updated
to start complaining about "if False" statements. Two locations in fb-hgext
had this issue.

These both look to be copy-and-pastes of the same function, unfortunately.
That should probably be cleaned up too, but I'll leave that for a separate diff
for now.

Test Plan: Confirmed the test-check-code test passes now.

Reviewers: #fbhgext, durham, quark

Reviewed By: #fbhgext, quark

Differential Revision: https://phab.mercurial-scm.org/D71

Details

Committed
simpkinsJul 13 2017, 12:29 AM
Reviewer
Restricted Project
Differential Revision
D71: tests: fix 'if False' statements flagged by check-code
Parents
rFBHGX444415a23367: conduit: cache and re-use conduit connections
Branches
Unknown
Tags
Unknown