On 6/15/22 6:53 AM, Ævar Arnfjörð Bjarmason wrote: > Add tests that were missing from 6dcbdc0d661 (remote: create > fetch.credentialsInUrl config, 2022-06-06), we want to test how we > handle cases where the config comes from a file, and that we handle > "pushURL" correctly. The tests use your renamed transfer.credentialsInUrl, but you haven't renamed at this point. > Currently the "pushURL" case isn't handled at all, i.e. URLs aren't > warned about in "remote.*pushurl" , only for "remote.*.url". Good find. I suppose that the second test_expect_success will _fail_ at the point of this patch? Thanks, -Stolee