Add unit tests for crate_package_default_visibility

Two tests in rendering.rs:
- render_crate_package_default_visibility: verifies custom visibility
  appears in the generated BUILD file instead of //visibility:public
- render_default_visibility_when_not_configured: verifies the default
  behavior (//visibility:public) when the option is not set

Update lockfile digest test expectations since the new config field
changes the serialized RenderConfig.

Author: Tamas Vajk

crate_universe/src/lockfile.rs

@@ -247,7 +247,7 @@ mod test {
         );
 
         assert_eq!(
-            Digest("edd73970897c01af3bb0e6c9d62f572203dd38a03c189dcca555d463990aa086".to_owned()),
+            Digest("3ae0c02accab9769e393d13aed5e78601a1d36e89db454ef8a249b879f1007e2".to_owned()),
             digest,
         );
     }
@@ -292,7 +292,7 @@ mod test {
         );
 
         assert_eq!(
-            Digest("8a4c1b3bb4c2d6c36e27565e71a13d54cff9490696a492c66a3a37bdd3893edf".to_owned()),
+            Digest("3a407984ca805d41b40f3fe1608371ec92288940d2786d4456bf8e0f66852828".to_owned()),
             digest,
         );
     }
@@ -323,7 +323,7 @@ mod test {
         );
 
         assert_eq!(
-            Digest("1e01331686ba1f26f707dc098cd9d21c39d6ccd8e46be03329bb2470d3833e15".to_owned()),
+            Digest("643a4ea83da473052a6a2965f9791bf85e14d55ead64edd661c3e915f19ee198".to_owned()),
             digest,
         );
     }
@@ -372,7 +372,7 @@ mod test {
         );
 
         assert_eq!(
-            Digest("45ccf7109db2d274420fac521f4736a1fb55450ec60e6df698e1be4dc2c89fad".to_owned()),
+            Digest("daf77d5674233fc3dcc8270b2e793f54c8428a1248dc25e90bb6ecc2315a7d0f".to_owned()),
             digest,
         );
     }

crate_universe/src/rendering.rs

@@ -2366,4 +2366,97 @@ mod test {
             "proc-macro lib must appear in proc_macro_deps:\n{binary_section}"
         );
     }
+
+    #[test]
+    fn render_crate_package_default_visibility() {
+        let mut context = Context::default();
+        let crate_id = CrateId::new("mock_crate".to_owned(), VERSION_ZERO_ONE_ZERO);
+        context.crates.insert(
+            crate_id.clone(),
+            CrateContext {
+                name: crate_id.name,
+                version: crate_id.version,
+                package_url: None,
+                repository: None,
+                targets: BTreeSet::from([Rule::Library(mock_target_attributes())]),
+                library_target_name: None,
+                common_attrs: CommonAttributes::default(),
+                build_script_attrs: None,
+                license: None,
+                license_ids: BTreeSet::default(),
+                license_file: None,
+                additive_build_file_content: None,
+                disable_pipelining: false,
+                extra_aliased_targets: BTreeMap::default(),
+                alias_rule: None,
+                override_targets: BTreeMap::default(),
+            },
+        );
+
+        let config = Arc::new(RenderConfig {
+            repository_name: "test_rendering".to_owned(),
+            regen_command: "cargo_bazel_regen_command".to_owned(),
+            crate_package_default_visibility: Some(vec![
+                "//third_party/rust:__subpackages__".to_owned()
+            ]),
+            ..RenderConfig::default()
+        });
+        let renderer = Renderer::new(config, mock_supported_platform_triples());
+        let output = renderer.render(&context, None).unwrap();
+
+        let build_file_content = output
+            .get(&PathBuf::from("BUILD.mock_crate-0.1.0.bazel"))
+            .unwrap();
+
+        assert!(
+            build_file_content.contains("//third_party/rust:__subpackages__"),
+            "Expected custom visibility in BUILD file:\n{}",
+            build_file_content
+        );
+        assert!(
+            !build_file_content.contains("//visibility:public"),
+            "Expected no public visibility in BUILD file:\n{}",
+            build_file_content
+        );
+    }
+
+    #[test]
+    fn render_default_visibility_when_not_configured() {
+        let mut context = Context::default();
+        let crate_id = CrateId::new("mock_crate".to_owned(), VERSION_ZERO_ONE_ZERO);
+        context.crates.insert(
+            crate_id.clone(),
+            CrateContext {
+                name: crate_id.name,
+                version: crate_id.version,
+                package_url: None,
+                repository: None,
+                targets: BTreeSet::from([Rule::Library(mock_target_attributes())]),
+                library_target_name: None,
+                common_attrs: CommonAttributes::default(),
+                build_script_attrs: None,
+                license: None,
+                license_ids: BTreeSet::default(),
+                license_file: None,
+                additive_build_file_content: None,
+                disable_pipelining: false,
+                extra_aliased_targets: BTreeMap::default(),
+                alias_rule: None,
+                override_targets: BTreeMap::default(),
+            },
+        );
+
+        let renderer = Renderer::new(mock_render_config(None), mock_supported_platform_triples());
+        let output = renderer.render(&context, None).unwrap();
+
+        let build_file_content = output
+            .get(&PathBuf::from("BUILD.mock_crate-0.1.0.bazel"))
+            .unwrap();
+
+        assert!(
+            build_file_content.contains("//visibility:public"),
+            "Expected public visibility in BUILD file:\n{}",
+            build_file_content
+        );
+    }
 }