From 9cb8feca8c38e306d89efa171e443f7b128ced13 Mon Sep 17 00:00:00 2001 From: Alex James Date: Sun, 21 Apr 2024 15:10:20 -0500 Subject: [PATCH] delta: run tests with 1 thread One of the tests (test_env_parsing_with_pager_set_to_bat) sets an environment variable, which can be flaky with multiple threads (see https://github.com/dandavison/delta/issues/1660). Work around this by running the tests with 1 thread until this is fixed upstream. --- pkgs/applications/version-management/delta/default.nix | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/pkgs/applications/version-management/delta/default.nix b/pkgs/applications/version-management/delta/default.nix index 3065d5672193..b343e473e3c8 100644 --- a/pkgs/applications/version-management/delta/default.nix +++ b/pkgs/applications/version-management/delta/default.nix @@ -44,6 +44,11 @@ rustPlatform.buildRustPackage rec { etc/completion/completion.{bash,fish,zsh} ''; + # test_env_parsing_with_pager_set_to_bat sets environment variables, + # which can be flaky with multiple threads: + # https://github.com/dandavison/delta/issues/1660 + dontUseCargoParallelTests = true; + checkFlags = lib.optionals stdenv.isDarwin [ "--skip=test_diff_same_non_empty_file" ];