feat(rust): add doctest check

Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
rvolosatovs · Oct 21, 2024 · ddfd1d8 · ddfd1d8
1 parent 0e0bc1c
commit ddfd1d8
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 5 deletions.
diff --git a/checks/default.nix b/checks/default.nix
@@ -74,6 +74,7 @@ in
       assert attrs.${system} ? "${name}-x86_64-unknown-linux-musl-oci";
       assert attrs.${system} ? ${name}; x;
   in
+    assert flakes.rust.complex.checks.${system} ? doctest;
     assert flakes.rust.complex.packages.${system} ? default;
     assert flakes.rust.hello-multibin.packages.${system} ? default;
     assert flakes.rust.hello-multibin.packages.${system} ? rust-hello-multibin-wasm32-wasip1;

diff --git a/examples/rust-complex/flake.nix b/examples/rust-complex/flake.nix
@@ -14,6 +14,7 @@
 
       build.workspace = true;
       clippy.workspace = true;
+      test.doc = true;
       test.workspace = true;
 
       buildOverrides = {

diff --git a/lib/rust/default.nix b/lib/rust/default.nix
@@ -128,6 +128,7 @@ in {
 
   defaultTestConfig.allFeatures = false;
   defaultTestConfig.allTargets = false;
+  defaultTestConfig.doc = false;
   defaultTestConfig.excludes = [];
   defaultTestConfig.features = [];
   defaultTestConfig.noDefaultFeatures = false;

diff --git a/lib/rust/mkAttrs.nix b/lib/rust/mkAttrs.nix
@@ -210,16 +210,24 @@ with self.lib.rust.targets;
         }
         callCraneWithDeps (mkHostArgs craneArgs);
 
+      callHostCraneCheckWithDeps = craneArgs:
+        trace' "callHostCraneCheckWithDeps" {
+          inherit craneArgs;
+        }
+        callHostCraneWithDeps (craneArgs
+          // {
+            doCheck = true; # without performing the actual testing, this check is useless
+          });
+
       checks =
         {
           clippy = callHostCraneWithDeps {} hostCraneLib.cargoClippy;
           doc = callHostCraneWithDeps {} hostCraneLib.cargoDoc;
           fmt = callHostCrane {} hostCraneLib.cargoFmt;
-          nextest =
-            callHostCraneWithDeps {
-              doCheck = true; # without performing the actual testing, this check is useless
-            }
-            hostCraneLib.cargoNextest;
+          nextest = callHostCraneCheckWithDeps {} hostCraneLib.cargoNextest;
+        }
+        // (optionalAttrs (test ? doc && test.doc)) {
+          doctest = callHostCraneCheckWithDeps {} hostCraneLib.cargoDocTest;
         }
         // (optionalAttrs (pathExists "${src}/Cargo.lock") {
           # TODO: Use `cargoLock` if `Cargo.lock` missing