Skip to content

Commit cd1688b

Browse files
committed
env_vars: Wrap set/remove_var() calls with unsafe
1 parent 8b7b194 commit cd1688b

File tree

1 file changed

+19
-19
lines changed
  • crates/crates_io_env_vars/src

1 file changed

+19
-19
lines changed

crates/crates_io_env_vars/src/lib.rs

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -139,21 +139,21 @@ mod tests {
139139
fn test_var() {
140140
let _guard = MUTEX.lock().unwrap();
141141

142-
std::env::set_var(TEST_VAR, "test");
142+
unsafe { std::env::set_var(TEST_VAR, "test") };
143143
assert_some_eq!(assert_ok!(var(TEST_VAR)), "test");
144144

145-
std::env::remove_var(TEST_VAR);
145+
unsafe { std::env::remove_var(TEST_VAR) };
146146
assert_none!(assert_ok!(var(TEST_VAR)));
147147
}
148148

149149
#[test]
150150
fn test_required_var() {
151151
let _guard = MUTEX.lock().unwrap();
152152

153-
std::env::set_var(TEST_VAR, "test");
153+
unsafe { std::env::set_var(TEST_VAR, "test") };
154154
assert_ok_eq!(required_var(TEST_VAR), "test");
155155

156-
std::env::remove_var(TEST_VAR);
156+
unsafe { std::env::remove_var(TEST_VAR) };
157157
let error = assert_err!(required_var(TEST_VAR));
158158
assert_eq!(
159159
error.to_string(),
@@ -165,35 +165,35 @@ mod tests {
165165
fn test_var_parsed() {
166166
let _guard = MUTEX.lock().unwrap();
167167

168-
std::env::set_var(TEST_VAR, "42");
168+
unsafe { std::env::set_var(TEST_VAR, "42") };
169169
assert_some_eq!(assert_ok!(var_parsed::<i32>(TEST_VAR)), 42);
170170

171-
std::env::set_var(TEST_VAR, "test");
171+
unsafe { std::env::set_var(TEST_VAR, "test") };
172172
let error = assert_err!(var_parsed::<i32>(TEST_VAR));
173173
assert_eq!(
174174
error.to_string(),
175175
"Failed to parse CRATES_IO_ENV_VARS_TEST_VAR environment variable"
176176
);
177177

178-
std::env::remove_var(TEST_VAR);
178+
unsafe { std::env::remove_var(TEST_VAR) };
179179
assert_none!(assert_ok!(var_parsed::<i32>(TEST_VAR)));
180180
}
181181

182182
#[test]
183183
fn test_required_var_parsed() {
184184
let _guard = MUTEX.lock().unwrap();
185185

186-
std::env::set_var(TEST_VAR, "42");
186+
unsafe { std::env::set_var(TEST_VAR, "42") };
187187
assert_ok_eq!(required_var_parsed::<i32>(TEST_VAR), 42);
188188

189-
std::env::set_var(TEST_VAR, "test");
189+
unsafe { std::env::set_var(TEST_VAR, "test") };
190190
let error = assert_err!(required_var_parsed::<i32>(TEST_VAR));
191191
assert_eq!(
192192
error.to_string(),
193193
"Failed to parse CRATES_IO_ENV_VARS_TEST_VAR environment variable"
194194
);
195195

196-
std::env::remove_var(TEST_VAR);
196+
unsafe { std::env::remove_var(TEST_VAR) };
197197
let error = assert_err!(required_var_parsed::<i32>(TEST_VAR));
198198
assert_eq!(
199199
error.to_string(),
@@ -205,40 +205,40 @@ mod tests {
205205
fn test_list() {
206206
let _guard = MUTEX.lock().unwrap();
207207

208-
std::env::set_var(TEST_VAR, "test");
208+
unsafe { std::env::set_var(TEST_VAR, "test") };
209209
assert_ok_eq!(list(TEST_VAR), vec!["test"]);
210210

211-
std::env::set_var(TEST_VAR, "test, foo, bar ");
211+
unsafe { std::env::set_var(TEST_VAR, "test, foo, bar ") };
212212
assert_ok_eq!(list(TEST_VAR), vec!["test", "foo", "bar"]);
213213

214-
std::env::set_var(TEST_VAR, "");
214+
unsafe { std::env::set_var(TEST_VAR, "") };
215215
assert_ok_eq!(list(TEST_VAR), Vec::<String>::new());
216216

217-
std::env::remove_var(TEST_VAR);
217+
unsafe { std::env::remove_var(TEST_VAR) };
218218
assert_ok_eq!(list(TEST_VAR), Vec::<String>::new());
219219
}
220220

221221
#[test]
222222
fn test_list_parsed() {
223223
let _guard = MUTEX.lock().unwrap();
224224

225-
std::env::set_var(TEST_VAR, "42");
225+
unsafe { std::env::set_var(TEST_VAR, "42") };
226226
assert_ok_eq!(list_parsed(TEST_VAR, i32::from_str), vec![42]);
227227

228-
std::env::set_var(TEST_VAR, "42, 1, -53 ");
228+
unsafe { std::env::set_var(TEST_VAR, "42, 1, -53 ") };
229229
assert_ok_eq!(list_parsed(TEST_VAR, i32::from_str), vec![42, 1, -53]);
230230

231-
std::env::set_var(TEST_VAR, "42, what");
231+
unsafe { std::env::set_var(TEST_VAR, "42, what") };
232232
let error = assert_err!(list_parsed(TEST_VAR, i32::from_str));
233233
assert_eq!(
234234
error.to_string(),
235235
"Failed to parse value \"what\" of CRATES_IO_ENV_VARS_TEST_VAR environment variable"
236236
);
237237

238-
std::env::set_var(TEST_VAR, "");
238+
unsafe { std::env::set_var(TEST_VAR, "") };
239239
assert_ok_eq!(list_parsed(TEST_VAR, i32::from_str), Vec::<i32>::new());
240240

241-
std::env::remove_var(TEST_VAR);
241+
unsafe { std::env::remove_var(TEST_VAR) };
242242
assert_ok_eq!(list_parsed(TEST_VAR, i32::from_str), Vec::<i32>::new());
243243
}
244244
}

0 commit comments

Comments
 (0)