assertions for build

Author: syphar

.github/workflows/ci.yml

@@ -63,6 +63,9 @@ jobs:
       - name: Test
         run: cargo test --workspace --locked
 
+      - name: Test expensive tests
+        run: cargo test --workspace --locked -- --ignored
+
       - name: Clean up the database
         run: docker-compose down --volumes
 

src/docbuilder/rustwide_builder.rs

@@ -19,7 +19,7 @@ use rustwide::cmd::{Command, SandboxBuilder, SandboxImage};
 use rustwide::logging::{self, LogStorage};
 use rustwide::toolchain::ToolchainError;
 use rustwide::{Build, Crate, Toolchain, Workspace, WorkspaceBuilder};
-use serde_json::Value;
+use serde_json::{json, Value};
 use std::collections::{HashMap, HashSet};
 use std::path::Path;
 use std::sync::Arc;
@@ -760,12 +760,75 @@ mod tests {
     use crate::test::wrapper;
 
     #[test]
+    #[ignore]
     fn test_build_crate() {
         wrapper(|env| {
+            let crate_ = "log";
+            let version = "0.4.14";
+            let default_target = "x86_64-unknown-linux-gnu";
+            // env.fake_release()
+            //     .name(crate_)
+            //     .version(version)
+            //     .default_target(default_target)
+            //     .add_target("i686-pc-windows-msvc")
+            //     .add_target("i686-unknown-linux-gnu")
+            //     .add_target("x86_64-apple-darwin")
+            //     .add_target("x86_64-pc-windows-msvc")
+            //     .create()?;
+
             let mut builder = RustwideBuilder::init(env).unwrap();
             builder
-                .build_package("log", "0.4.14", PackageKind::CratesIo)
-                .map(|_| ())
+                .build_package(crate_, version, PackageKind::CratesIo)
+                .map(|_| ())?;
+
+            // check release record in the db (default and other targets)
+            let mut conn = env.db().conn();
+            let rows = conn
+                .query(
+                    "SELECT 
+                        r.rustdoc_status,
+                        r.default_target,
+                        r.doc_targets
+                    FROM 
+                        crates as c 
+                        INNER JOIN releases AS r ON c.id = r.crate_id
+                    WHERE 
+                        c.name = $1 AND 
+                        r.version = $2",
+                    &[&crate_, &version],
+                )
+                .unwrap();
+            let row = rows.get(0).unwrap();
+
+            assert_eq!(row.get::<_, bool>("rustdoc_status"), true);
+            assert_eq!(row.get::<_, String>("default_target"), default_target);
+            assert_eq!(
+                row.get::<_, Value>("doc_targets"),
+                json!([
+                    "x86_64-unknown-linux-gnu", // default target is first
+                    "i686-pc-windows-msvc",     // rest is alphabetically
+                    "i686-unknown-linux-gnu",
+                    "x86_64-apple-darwin",
+                    "x86_64-pc-windows-msvc",
+                ])
+            );
+
+            let storage = env.storage();
+            assert!(storage.exists("asdf")?);
+
+            let base = format!("rustdoc/{}/{}", crate_, version);
+
+            // default target was build
+            assert!(storage.exists(&format!("{}/index.html", base))?);
+
+            // other targets too
+            for target in DEFAULT_TARGETS {
+                if target != &default_target {
+                    assert!(storage.exists(&format!("{}/{}/index.html", base, target))?);
+                }
+            }
+
+            Ok(())
         })
     }
 }