mirror of
https://github.com/ipetkov/crane.git
synced 2024-11-25 11:02:01 +03:00
4c6c77920b
Both `cargo nextest run` and `cargo llvm-cov` do not place any cargo options/flags right behind `cargo <HERE> ...`. All cargo related flags can be added to the `cargoNextestExtraArgs` attribute. [Issue #675] After altering `checkPhaseCargoCommand` to use one of two explicit command strings inside if/else blocks, it is no longer necessary to default `cmd` to an empty value when `withLlvmCov` is set to `true`. --------- Co-authored-by: Ivan Petkov <ivanppetkov@gmail.com>
107 lines
3.3 KiB
Nix
107 lines
3.3 KiB
Nix
{ cargo-llvm-cov
|
|
, cargo-nextest
|
|
, rustc
|
|
, lib
|
|
, stdenv
|
|
, mkCargoDerivation
|
|
, runCommand
|
|
}:
|
|
|
|
{ cargoArtifacts
|
|
, cargoExtraArgs ? ""
|
|
, cargoLlvmCovExtraArgs ? "--lcov --output-path $out/coverage"
|
|
, cargoNextestExtraArgs ? ""
|
|
, doInstallCargoArtifacts ? true
|
|
, partitions ? 1
|
|
, partitionType ? "count"
|
|
, withLlvmCov ? false
|
|
, ...
|
|
}@origArgs:
|
|
let
|
|
args = builtins.removeAttrs origArgs [
|
|
"cargoExtraArgs"
|
|
"cargoLlvmCovExtraArgs"
|
|
"cargoNextestExtraArgs"
|
|
"partitions"
|
|
"partitionType"
|
|
"withLlvmCov"
|
|
];
|
|
|
|
mkUpdatedArgs = { cmd ? "run", extraSuffix ? "", moreArgs ? "", withLlvmCov }: args // {
|
|
inherit cargoArtifacts;
|
|
pnameSuffix = "-nextest${extraSuffix}";
|
|
doCheck = args.doCheck or true;
|
|
|
|
buildPhaseCargoCommand = args.buildPhaseCargoCommand or ''
|
|
mkdir -p $out
|
|
${lib.optionalString withLlvmCov "cargo llvm-cov --version"}
|
|
cargo nextest --version
|
|
'';
|
|
|
|
# Work around Nextest bug: https://github.com/nextest-rs/nextest/issues/267
|
|
preCheck = (args.preCheck or "") + lib.optionalString stdenv.isDarwin ''
|
|
export DYLD_FALLBACK_LIBRARY_PATH=$(${rustc}/bin/rustc --print sysroot)/lib
|
|
'';
|
|
|
|
checkPhaseCargoCommand = ''
|
|
${if withLlvmCov
|
|
then "cargo llvm-cov nextest ${cargoLlvmCovExtraArgs}"
|
|
else "cargo nextest ${cmd}"} \
|
|
''${CARGO_PROFILE:+--cargo-profile $CARGO_PROFILE} \
|
|
${cargoExtraArgs} ${cargoNextestExtraArgs} ${moreArgs}
|
|
'';
|
|
|
|
nativeBuildInputs = (args.nativeBuildInputs or [ ])
|
|
++ [ cargo-nextest ]
|
|
++ lib.lists.optional withLlvmCov cargo-llvm-cov;
|
|
};
|
|
in
|
|
if partitions < 1 then
|
|
throw "paritions must be at least 1 or greater"
|
|
else if partitions == 1 then # Simple case do everything in one derivation
|
|
mkCargoDerivation (mkUpdatedArgs { inherit withLlvmCov; })
|
|
else # First build the tests in one derivation, then run each partition in another
|
|
let
|
|
mkArchiveArgs = root: "--archive-format tar-zst --archive-file ${root}/archive.tar.zst";
|
|
archive = mkCargoDerivation (mkUpdatedArgs {
|
|
cmd = "archive";
|
|
moreArgs = mkArchiveArgs "$out";
|
|
withLlvmCov = !(lib.asserts.assertMsg (!withLlvmCov) "withLLvmCov is not supported for partitioned runs");
|
|
});
|
|
mkPartition = nInt:
|
|
let
|
|
n = toString (nInt + 1);
|
|
in
|
|
mkCargoDerivation ((mkUpdatedArgs {
|
|
inherit withLlvmCov;
|
|
extraSuffix = "-p${toString n}";
|
|
moreArgs = builtins.concatStringsSep " " [
|
|
"${mkArchiveArgs archive}"
|
|
"--workspace-remap ."
|
|
"--partition ${partitionType}:${n}/${toString partitions}"
|
|
];
|
|
}) // {
|
|
# Everything we need is already in the archive
|
|
cargoArtifacts = null;
|
|
cargoVendorDir = null;
|
|
doInstallCargoArtifacts = false;
|
|
|
|
# Nextest does not like extra args when running with an archive
|
|
CARGO_PROFILE = "";
|
|
});
|
|
in
|
|
# Allow for retaining the artifacts from the `archive` derivation
|
|
# if callers want to chain other derivations after it. We provide
|
|
# the actual `partition*` derivations as inputs to ensure they are run.
|
|
runCommand "cargo-nextest-tests"
|
|
{
|
|
inherit doInstallCargoArtifacts;
|
|
buildInputs = lib.genList mkPartition partitions;
|
|
} ''
|
|
if [ "1" = "''${doInstallCargoArtifacts-}" ]; then
|
|
cp --recursive ${archive} $out/
|
|
else
|
|
mkdir -p $out
|
|
fi
|
|
''
|