Merge pull request #1132 from jeckersb/kargs-no-env

cgwalters · web-flow · commit 00860d70061d · 2025-02-21T13:54:33.000-05:00
kargs: do not set ARCH envvar in tests
diff --git a/lib/src/kargs.rs b/lib/src/kargs.rs
@@ -229,7 +229,7 @@ match-architectures = ["x86_64", "aarch64"]
         .to_string();
         let parsed_kargs = parse_kargs_toml(&file_content, sys_arch).unwrap();
         assert_eq!(parsed_kargs, ["console=tty0", "nosmt"]);
-        std::env::set_var("ARCH", "aarch64");
+        let sys_arch = "aarch64";
         let parsed_kargs = parse_kargs_toml(&file_content, sys_arch).unwrap();
         assert_eq!(parsed_kargs, ["console=tty0", "nosmt"]);
     }

Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,7 @@ match-architectures = ["x86_64", "aarch64"]`
`229`	`229`	`.to_string();`
`230`	`230`	`let parsed_kargs = parse_kargs_toml(&file_content, sys_arch).unwrap();`
`231`	`231`	`assert_eq!(parsed_kargs, ["console=tty0", "nosmt"]);`
`232`		`- std::env::set_var("ARCH", "aarch64");`
	`232`	`+ let sys_arch = "aarch64";`
`233`	`233`	`let parsed_kargs = parse_kargs_toml(&file_content, sys_arch).unwrap();`
`234`	`234`	`assert_eq!(parsed_kargs, ["console=tty0", "nosmt"]);`
`235`	`235`	`}`