Ignore tests

[RazrFalcon]

Currently, the test code is also included, which is misleading.

For example, the float-cmp crate has only two unsafe expressions, but 32 are reported.

[anderejd]

Good find! Thank you.

[anderejd]

Question: Should there be a command line flag to include tests but keep it off by default?

[RazrFalcon]

Unsafe in tests is irrelative to the crate itself. They will not be included into the parent crate. So I don't see a point in including them even as an option.

[anderejd]

Sounds reasonable to me 👍

[anderejd]

A similar issue: https://github.com/anderejd/cargo-geiger/issues/12

[anderejd]

Related to: https://github.com/anderejd/cargo-geiger/issues/14

[anderejd]

Simple #[test] functions are now ignored. Fixed by: #26

Functions inside #[cfg(test)] modules that are not decorated with the #[test] attribute are still counted though, so that needs fixing.

This could be fixed as part of general tracking of code usage on a per function basis, see #22

[anderejd]

Since #26 the counts for float-cmp seems correct to me.

Since this commit:
anderejd@9fa3534
there's a new flag --include-tests. Without it float-cmp reports 8/8 unsafe expressions which seems correct, it has four transmutes, each with a dereference expression inside. With --include-tests the count is 20/20 unsafe expressions.

[anderejd]

Commit: anderejd@cdccce6 adds default ignore filter for #[cfg(test)] modules.

I think this issue can be closed now. Feel free to reopen if the solution is not good enough.