Re: [PATCH 1/2] t5516/t5601: avoid using `localhost` for failing HTTPS requests

From: Jeff King <peff@peff.net>
Date: Nov 1, 2022, 02:25 UTC
Message-ID: <Y2CDrZNgNZ+flJBK@coredump.intra.peff.net>
In-Reply-To: <Y2B+jn4goN4OIcwL@coredump.intra.peff.net>

On Mon, Oct 31, 2022 at 10:03:58PM -0400, Jeff King wrote:

> I do think moving these tests to use a real http server is a better
> solution, though. I'll provide a patch in a moment.

Here that is. I hope this isn't co-opting your series; my goal was to do the work so that you wouldn't have to, but I admit it is assuming you agree with my approach. ;)

The first patch is the interesting one. The second one is just your 2/2 rebased onto the new location.

  [1/2]: t5516: move plaintext-password tests from t5601 and t5516
  [2/2]: t5516/t5601: be less strict about the number of credential warnings

 t/t5516-fetch-push.sh       | 31 ---------------
 t/t5551-http-fetch-smart.sh | 77 +++++++++++++++++++++++++++++++++++++
 t/t5601-clone.sh            | 23 -----------
 3 files changed, 77 insertions(+), 54 deletions(-)

-Peff