fix(tests/bundle_env_overrides): make more robust (#1316)

Author: CraftSpider

Cargo.lock

@@ -59,7 +59,6 @@ members = [
     "crates/xdv",
     "crates/xetex_format",
     "crates/xetex_layout",
-    "tests/bundle_env_overrides_test",
 ]
 
 [workspace.lints.rust]

Cargo.toml

@@ -58,11 +58,18 @@ fn test_bundle_locked() {
 fn test_bundle_prefix() {
     let url_prefixed = run_test_program(
         PRE_FORMAT_VERSION,
-        &[("TECTONIC_BUNDLE_PREFIX", TEST_BUNDLE_PREFIX)],
+        &[
+            ("TECTONIC_BUNDLE_PREFIX", TEST_BUNDLE_PREFIX),
+            ("TECTONIC_BUNDLE_LOCKED", ""),
+            // ^ locked url should be ignored if empty
+        ],
     );
     let url_prefixed_versioned = run_test_program(
         TEST_FORMAT_VERSION,
-        &[("TECTONIC_BUNDLE_PREFIX", TEST_BUNDLE_PREFIX)],
+        &[
+            ("TECTONIC_BUNDLE_PREFIX", TEST_BUNDLE_PREFIX),
+            ("TECTONIC_BUNDLE_LOCKED", ""),
+        ],
     );
 
     assert_eq!(
@@ -86,18 +93,3 @@ fn test_precedence_locked_over_prefix() {
     );
     assert_eq!(url_both_env_set, TEST_BUNDLE_LOCKED);
 }
-
-#[test]
-fn test_empty_locked_bundle_ignored() {
-    let url_empty_locked = run_test_program(
-        TEST_FORMAT_VERSION,
-        &[
-            ("TECTONIC_BUNDLE_LOCKED", ""),
-            ("TECTONIC_BUNDLE_PREFIX", TEST_BUNDLE_PREFIX),
-        ],
-    );
-    assert_eq!(
-        url_empty_locked,
-        format!("{TEST_BUNDLE_PREFIX}/default_bundle_v{TEST_FORMAT_VERSION}.tar",)
-    );
-}

tests/bundle_env_overrides.rs

@@ -1 +1,2 @@
 target
+Cargo.lock

tests/bundle_env_overrides_test/.gitignore

@@ -5,7 +5,8 @@ version = "0.0.0-dev.0"
 edition = "2021"
 
 [dependencies]
-tectonic_bundles = { path = "../../crates/bundles", version = "0.0.0-dev.0" }
+tectonic_bundles = { path = "../../crates/bundles" }
 
-[package.metadata.internal_dep_versions]
-tectonic_bundles = "thiscommit:2025-07-26:IKt5CJV"
+# decouple this from the the top-level workspace
+[workspace]
+members = ["."]

tests/bundle_env_overrides_test/Cargo.toml

@@ -1,6 +1,6 @@
 # Bundle URL overrides Test
 
-This is a simple test program used by the integration tests in the `tectonic_bundles` crate to test the `get_fallback_bundle_url` function with different compile-time environment variable overrides.
+This is a simple test program used by the integration tests to test the [`tectonic_bundles::get_fallback_bundle_url`] function with different compile-time environment variable overrides.
 
 ## Purpose