Add test for when all members in a workspace are publish = false (#859

)

* Add snapshot testing with `insta`

* temporary dead code allows

* explicitly disable backtraces

* use `insta_cmd` for command snapshots

* move snapshot tests to lib tests

* add snapshot bin test harness

* Revert "move snapshot tests to lib tests"

This reverts commit 72f99bc.

* Add test for no lib targets in the workspace

* move snapshot outputs to lib tests

* move snapshots directory to lib tests

* Revert "move snapshots directory to lib tests"

This reverts commit b56a113.

* Revert "move snapshot outputs to lib tests"

This reverts commit 8b6494c.

* add test files

* Add test for all workspace members `publish = false`

* Add snapshot testing with `insta`

* temporary dead code allows

* explicitly disable backtraces

* use `insta_cmd` for command snapshots

* move snapshot tests to lib tests

* add snapshot bin test harness

* fix docs

* clippy fixes

* fix merge ;(

* ...

* remove old test file

* delete crates again

src/snapshot_tests.rs

@@ -195,3 +195,23 @@ fn workspace_no_lib_targets_error() {
         ],
     );
 }
+
+/// [#424](/~https://github.com/obi1kenobi/cargo-semver-checks/issues/424)
+///
+/// Running `cargo semver-checks --workspace` on a workspace whose members are all
+/// `publish = false`.
+#[test]
+fn workspace_all_publish_false() {
+    assert_integration_test(
+        "workspace_all_publish_false",
+        &[
+            "cargo",
+            "semver-checks",
+            "--manifest-path",
+            "test_crates/manifest_tests/workspace_all_publish_false/new",
+            "--baseline-root",
+            "test_crates/manifest_tests/workspace_all_publish_false/old",
+            "--workspace",
+        ],
+    );
+}

test_crates/manifest_tests/workspace_all_publish_false/new/Cargo.toml

@@ -0,0 +1,3 @@
+[workspace]
+resolver = "2"
+members = ["a", "b"]

test_crates/manifest_tests/workspace_all_publish_false/new/a/Cargo.toml

@@ -0,0 +1,7 @@
+[package]
+name = "a"
+version = "0.1.0"
+edition = "2021"
+publish = false
+
+[dependencies]

test_crates/manifest_tests/workspace_all_publish_false/new/a/src/lib.rs

@@ -0,0 +1,4 @@
+// Removed in new version, so would trigger `function_missing` lint,
+// but this crate is set to `publish = false`, so `cargo-semver-checks`
+// should not check it.
+// pub fn should_not_run() {}

test_crates/manifest_tests/workspace_all_publish_false/new/b/Cargo.toml

@@ -0,0 +1,7 @@
+[package]
+name = "b"
+version = "0.1.0"
+edition = "2021"
+publish = false
+
+[dependencies]

test_crates/manifest_tests/workspace_all_publish_false/new/b/src/lib.rs

@@ -0,0 +1,2 @@
+// An empty library target with `publish = false`, that should
+// not be semver-checked.

test_crates/manifest_tests/workspace_all_publish_false/old/Cargo.toml

@@ -0,0 +1,3 @@
+[workspace]
+resolver = "2"
+members = ["a", "b"]

test_crates/manifest_tests/workspace_all_publish_false/old/a/Cargo.toml

@@ -0,0 +1,7 @@
+[package]
+name = "a"
+version = "0.1.0"
+edition = "2021"
+publish = false
+
+[dependencies]

test_crates/manifest_tests/workspace_all_publish_false/old/a/src/lib.rs

@@ -0,0 +1,4 @@
+// Removed in new version, so would trigger `function_missing` lint,
+// but this crate is set to `publish = false`, so `cargo-semver-checks`
+// should not check it.
+pub fn should_not_run() {}

test_crates/manifest_tests/workspace_all_publish_false/old/b/Cargo.toml

@@ -0,0 +1,7 @@
+[package]
+name = "b"
+version = "0.1.0"
+edition = "2021"
+publish = false
+
+[dependencies]

test_crates/manifest_tests/workspace_all_publish_false/old/b/src/lib.rs

@@ -0,0 +1,2 @@
+// An empty library target with `publish = false`, that should
+// not be semver-checked.

test_outputs/snapshot_tests/cargo_semver_checks__snapshot_tests__workspace_all_publish_false-input.snap

@@ -0,0 +1,30 @@
+---
+source: src/snapshot_tests.rs
+expression: check
+---
+Check(
+  scope: Scope(
+    mode: DenyList(PackageSelection(
+      selection: Workspace,
+      excluded_packages: [],
+    )),
+  ),
+  current: Rustdoc(
+    source: Root("test_crates/manifest_tests/workspace_all_publish_false/new"),
+  ),
+  baseline: Rustdoc(
+    source: Root("test_crates/manifest_tests/workspace_all_publish_false/old"),
+  ),
+  release_type: None,
+  current_feature_config: FeatureConfig(
+    features_group: Heuristic,
+    extra_features: [],
+    is_baseline: false,
+  ),
+  baseline_feature_config: FeatureConfig(
+    features_group: Heuristic,
+    extra_features: [],
+    is_baseline: true,
+  ),
+  build_target: None,
+)

test_outputs/snapshot_tests/cargo_semver_checks__snapshot_tests__workspace_all_publish_false-output.snap

@@ -0,0 +1,8 @@
+---
+source: src/snapshot_tests.rs
+expression: result
+---
+success: true
+--- stdout ---
+
+--- stderr ---