Skip to content

Commit b5a8bd7

Browse files
committed
fix suspicious_open_options by removing manual options
1 parent a22e66a commit b5a8bd7

File tree

2 files changed

+5
-22
lines changed

2 files changed

+5
-22
lines changed

crypto/eth2_keystore/tests/tests.rs

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -54,25 +54,17 @@ fn file() {
5454
let dir = tempdir().unwrap();
5555
let path = dir.path().join("keystore.json");
5656

57-
let get_file = || {
58-
File::options()
59-
.write(true)
60-
.read(true)
61-
.create(true)
62-
.open(path.clone())
63-
.expect("should create file")
64-
};
65-
6657
let keystore = KeystoreBuilder::new(&keypair, GOOD_PASSWORD, "".into())
6758
.unwrap()
6859
.build()
6960
.unwrap();
7061

7162
keystore
72-
.to_json_writer(&mut get_file())
63+
.to_json_writer(File::create_new(&path).unwrap())
7364
.expect("should write to file");
7465

75-
let decoded = Keystore::from_json_reader(&mut get_file()).expect("should read from file");
66+
let decoded =
67+
Keystore::from_json_reader(File::open(&path).unwrap()).expect("should read from file");
7668

7769
assert_eq!(
7870
decoded.decrypt_keypair(BAD_PASSWORD).err().unwrap(),

crypto/eth2_wallet/tests/tests.rs

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -132,20 +132,11 @@ fn file_round_trip() {
132132
let dir = tempdir().unwrap();
133133
let path = dir.path().join("keystore.json");
134134

135-
let get_file = || {
136-
File::options()
137-
.write(true)
138-
.read(true)
139-
.create(true)
140-
.open(path.clone())
141-
.expect("should create file")
142-
};
143-
144135
wallet
145-
.to_json_writer(&mut get_file())
136+
.to_json_writer(File::create_new(&path).unwrap())
146137
.expect("should write to file");
147138

148-
let decoded = Wallet::from_json_reader(&mut get_file()).unwrap();
139+
let decoded = Wallet::from_json_reader(File::open(&path).unwrap()).unwrap();
149140

150141
assert_eq!(
151142
decoded.decrypt_seed(&[1, 2, 3]).err().unwrap(),

0 commit comments

Comments
 (0)