Skip to content

Commit f668c67

Browse files
committed
re-enable previously disabled tests
On Windows, `gix` will call the `git-credential-manager, but with `stderr` set to `inherit` it makes any errors visible to the user, just like `git` does. ``` 1 1 Updating git repository `https://foo.bar/foo/bar` 2 +warning: auto-detection of host provider took too long (>2000ms) 3 +warning: see https://aka.ms/gcm/autodetect for more information. 4 +fatal: A task was canceled. 5 +warning: auto-detection of host provider took too long (>2000ms) 6 +warning: see https://aka.ms/gcm/autodetect for more information.` ```` This, however, isn't what's desirable in tests sometimes, nor may it be desirable in Cargo. In the latest version of `gix`, the default behaviour changed permanently to not inheriting `stderr` of worktree filters or credential helpers, which fixes this issue.
1 parent 930d653 commit f668c67

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

tests/testsuite/git_auth.rs

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -105,11 +105,6 @@ fn setup_failed_auth_test() -> (SocketAddr, JoinHandle<()>, Arc<AtomicUsize>) {
105105
// Tests that HTTP auth is offered from `credential.helper`.
106106
#[cargo_test]
107107
fn http_auth_offered() {
108-
// TODO(Seb): remove this once possible.
109-
if cargo_uses_gitoxide() {
110-
// Without the fixes in https://github.com/Byron/gitoxide/releases/tag/gix-v0.41.0 this test is flaky.
111-
return;
112-
}
113108
let (addr, t, connections) = setup_failed_auth_test();
114109
let p = project()
115110
.file(
@@ -372,11 +367,6 @@ Caused by:
372367

373368
#[cargo_test]
374369
fn instead_of_url_printed() {
375-
// TODO(Seb): remove this once possible.
376-
if cargo_uses_gitoxide() {
377-
// Without the fixes in https://github.com/Byron/gitoxide/releases/tag/gix-v0.41.0 this test is flaky.
378-
return;
379-
}
380370
let (addr, t, _connections) = setup_failed_auth_test();
381371
let config = paths::home().join(".gitconfig");
382372
let mut config = git2::Config::open(&config).unwrap();

0 commit comments

Comments
 (0)