Skip to content

Commit b43460d

Browse files
sypharjyn514
authored andcommitted
fix new clippy warnings
1 parent 1702933 commit b43460d

File tree

5 files changed

+8
-8
lines changed

5 files changed

+8
-8
lines changed

src/cdn.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ mod tests {
179179
config.cloudfront_distribution_id_web = Some("distribution_id_web".into());
180180
config.cloudfront_distribution_id_static = Some("distribution_id_static".into());
181181
});
182-
invalidate_crate(&*env.config(), &*env.cdn(), "krate")?;
182+
invalidate_crate(&env.config(), &env.cdn(), "krate")?;
183183

184184
assert!(matches!(*env.cdn(), CdnBackend::Dummy(_)));
185185
if let CdnBackend::Dummy(ref invalidation_requests) = *env.cdn() {

src/test/fakes.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,7 @@ impl<'a> FakeRelease<'a> {
303303
fs::create_dir_all(&path)
304304
.with_context(|| format!("failed to create {}", path.display()))?;
305305
}
306-
let file = base_path.join(&path);
306+
let file = base_path.join(path);
307307
debug!("writing file {}", file.display());
308308
fs::write(file, data)?;
309309
}
@@ -422,7 +422,7 @@ impl<'a> FakeRelease<'a> {
422422
&self.registry_crate_data,
423423
)?;
424424
for build in &self.builds {
425-
build.create(&mut db.conn(), &*storage, release_id, default_target)?;
425+
build.create(&mut db.conn(), &storage, release_id, default_target)?;
426426
}
427427
if let Some(coverage) = self.doc_coverage {
428428
crate::db::add_doc_coverage(&mut db.conn(), release_id, coverage)?;

src/web/page/mod.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,6 @@ mod tera_tests {
3535
"fa_icon": "https://gph.is/1uOvmqR"
3636
});
3737

38-
assert_eq!(correct_json, serde_json::to_value(&alert).unwrap());
38+
assert_eq!(correct_json, serde_json::to_value(alert).unwrap());
3939
}
4040
}

src/web/sitemap.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -213,8 +213,8 @@ mod tests {
213213
assert!(response.status().is_success());
214214

215215
let content = response.text()?;
216-
assert!(content.contains(&"some_random_crate"));
217-
assert!(!(content.contains(&"some_random_crate_that_failed")));
216+
assert!(content.contains("some_random_crate"));
217+
assert!(!(content.contains("some_random_crate_that_failed")));
218218

219219
// and not in the others
220220
for letter in ('a'..='z').filter(|&c| c != 's') {
@@ -245,7 +245,7 @@ mod tests {
245245
assert!(response.status().is_success());
246246

247247
let content = response.text()?;
248-
assert!(content.contains(&"2022-08-28T00:00:00+00:00"));
248+
assert!(content.contains("2022-08-28T00:00:00+00:00"));
249249
Ok(())
250250
})
251251
}

src/web/statics.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ mod tests {
231231
assert_cache_control(&resp, CachePolicy::ForeverInCdnAndBrowser, &env.config());
232232
assert_eq!(
233233
resp.bytes()?,
234-
fs::read(&path).unwrap(),
234+
fs::read(path).unwrap(),
235235
"failed to fetch {:?}",
236236
url,
237237
);

0 commit comments

Comments
 (0)