Fix compiler warnings in net unit tests

[marcusklaas]

This closes issue #5709.

[highfive]

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @mbrubeck (or someone else) soon.

[hoppipolla-critic-bot]

Critic review: https://critic.hoppipolla.co.uk/r/4715

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

[jdm]

$ ./mach test-tidy
tests/unit/net/mime_classifier.rs:14: tab on line
tests/unit/net/mime_classifier.rs:15: tab on line
tests/unit/net/mime_classifier.rs:17: tab on line
tests/unit/net/mime_classifier.rs:19: tab on line
tests/unit/net/mime_classifier.rs:47: trailing whitespace

[marcusklaas]

Aaah, will take care of those things promptly.

[jdm]

Reviewed files:

• tests/unit/lib.rs @ r1
• tests/unit/net/image_cache_task.rs @ r1
• tests/unit/net/mime_classifier.rs @ r1

---

^{tests/unit/net/mime_classifier.rs, line 60 [r1] (raw file):}
If we use {:?} here, I think we can just use filename instead of filename_string.

---

---

Comments from the review on Reviewable.io

[marcusklaas]

I updated the pull request, but by amending my initial commit. Apparently reviewable doesn't like this. Should I just have added new commits?

[jdm]

My understanding is that reviewable is supposed to support that. @larsbergstrom?

[jdm]

Actually it looks like reviewable handled it just fine!

[jdm]

Reviewed files:

• tests/unit/net/mime_classifier.rs @ r2

---

Comments from the review on Reviewable.io

[jdm]

@bors-servo: r+

[marcusklaas]

Cool, I just got this by reading @hoppipolla-critic-bot's comment in this thread.

[bors-servo]

📌 Commit 0d482e3 has been approved by jdm

[bors-servo]

⌛ Testing commit 0d482e3 with merge 2b3acd1...

[bors-servo]

☀️ Test successful - android, gonk, linux1, linux2, mac1, mac2