add #[test] to functions in test modules

This commit is contained in:
Takayuki Maeda 2022-07-08 10:16:28 +09:00
parent fac8fa5672
commit 29dc72dd10
4 changed files with 103 additions and 192 deletions

View File

@ -8,75 +8,93 @@ fn a() {}
fn b() {} fn b() {}
mod test { mod test {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
mod tests { mod tests {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
mod test_a { mod test_a {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
mod a_test { mod a_test {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
mod tests_a { mod tests_a {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
mod a_tests { mod a_tests {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
mod fastest_search { mod fastest_search {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
#[cfg(test)] #[cfg(test)]
mod test_has_attr { mod test_has_attr {
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }
mod test_has_no_attr { mod test_has_no_attr {
#[cfg(test)] #[cfg(test)]
use super::a; //~ ERROR unused import: `super::a` use super::a;
#[test]
fn foo() { fn foo() {
a();
use crate::b; //~ ERROR unused import: `crate::b` use crate::b; //~ ERROR unused import: `crate::b`
} }
} }

View File

@ -10,113 +10,59 @@ note: the lint level is defined here
LL | #![deny(unused_imports)] LL | #![deny(unused_imports)]
| ^^^^^^^^^^^^^^ | ^^^^^^^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:11:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:14:13 --> $DIR/unused-imports-in-test-mode.rs:16:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:19:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:22:13 --> $DIR/unused-imports-in-test-mode.rs:26:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:27:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:30:13 --> $DIR/unused-imports-in-test-mode.rs:36:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:35:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:38:13
|
LL | use crate::b;
| ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:43:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:46:13 --> $DIR/unused-imports-in-test-mode.rs:46:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:51:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:54:13 --> $DIR/unused-imports-in-test-mode.rs:56:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:59:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:62:13 --> $DIR/unused-imports-in-test-mode.rs:66:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:68:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:71:13 --> $DIR/unused-imports-in-test-mode.rs:76:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-mode.rs:77:9
|
LL | use super::a;
| ^^^^^^^^
error: unused import: `crate::b` error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:80:13 --> $DIR/unused-imports-in-test-mode.rs:87:13
| |
LL | use crate::b; LL | use crate::b;
| ^^^^^^^^ | ^^^^^^^^
error: aborting due to 19 previous errors error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-mode.rs:98:13
|
LL | use crate::b;
| ^^^^^^^^
error: aborting due to 10 previous errors

View File

@ -8,56 +8,81 @@ fn b() {}
mod test { mod test {
use super::a; //~ ERROR unused import: `super::a` use super::a; //~ ERROR unused import: `super::a`
#[test]
fn foo() { fn foo() {
use crate::b; //~ ERROR unused import: `crate::b` a();
use crate::b;
} }
} }
mod tests { mod tests {
use super::a; //~ ERROR unused import: `super::a` use super::a; //~ ERROR unused import: `super::a`
#[test]
fn foo() { fn foo() {
use crate::b; //~ ERROR unused import: `crate::b` a();
use crate::b;
} }
} }
mod test_a { mod test_a {
use super::a; //~ ERROR unused import: `super::a` use super::a; //~ ERROR unused import: `super::a`
#[test]
fn foo() { fn foo() {
use crate::b; //~ ERROR unused import: `crate::b` a();
use crate::b;
} }
} }
mod a_test { mod a_test {
use super::a; //~ ERROR unused import: `super::a` use super::a; //~ ERROR unused import: `super::a`
#[test]
fn foo() { fn foo() {
use crate::b; //~ ERROR unused import: `crate::b` a();
use crate::b;
} }
} }
mod tests_a { mod tests_a {
use super::a; //~ ERROR unused import: `super::a` use super::a; //~ ERROR unused import: `super::a`
#[test]
fn foo() { fn foo() {
use crate::b; //~ ERROR unused import: `crate::b` a();
use crate::b;
} }
} }
mod a_tests { mod a_tests {
use super::a; //~ ERROR unused import: `super::a` use super::a; //~ ERROR unused import: `super::a`
#[test]
fn foo() { fn foo() {
use crate::b; //~ ERROR unused import: `crate::b` a();
use crate::b;
} }
} }
mod fastest_search { mod fastest_search {
use super::a; //~ ERROR unused import: `super::a` use super::a; //~ ERROR unused import: `super::a`
#[test]
fn foo() { fn foo() {
use crate::b; //~ ERROR unused import: `crate::b` a();
use crate::b;
}
}
#[cfg(test)]
mod test_has_attr {
use super::a;
#[test]
fn foo() {
a();
use crate::b;
} }
} }

View File

@ -22,114 +22,42 @@ help: consider adding a `#[cfg(test)]` to the containing module
LL | mod test { LL | mod test {
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-module.rs:12:13
|
LL | use crate::b;
| ^^^^^^^^
|
help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:8:1
|
LL | mod test {
| ^^^^^^^^
error: unused import: `super::a` error: unused import: `super::a`
--> $DIR/unused-imports-in-test-module.rs:17:9 --> $DIR/unused-imports-in-test-module.rs:19:9
| |
LL | use super::a; LL | use super::a;
| ^^^^^^^^ | ^^^^^^^^
| |
help: consider adding a `#[cfg(test)]` to the containing module help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:16:1 --> $DIR/unused-imports-in-test-module.rs:18:1
|
LL | mod tests {
| ^^^^^^^^^
error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-module.rs:20:13
|
LL | use crate::b;
| ^^^^^^^^
|
help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:16:1
| |
LL | mod tests { LL | mod tests {
| ^^^^^^^^^ | ^^^^^^^^^
error: unused import: `super::a` error: unused import: `super::a`
--> $DIR/unused-imports-in-test-module.rs:25:9 --> $DIR/unused-imports-in-test-module.rs:29:9
| |
LL | use super::a; LL | use super::a;
| ^^^^^^^^ | ^^^^^^^^
| |
help: consider adding a `#[cfg(test)]` to the containing module help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:24:1 --> $DIR/unused-imports-in-test-module.rs:28:1
|
LL | mod test_a {
| ^^^^^^^^^^
error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-module.rs:28:13
|
LL | use crate::b;
| ^^^^^^^^
|
help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:24:1
| |
LL | mod test_a { LL | mod test_a {
| ^^^^^^^^^^ | ^^^^^^^^^^
error: unused import: `super::a` error: unused import: `super::a`
--> $DIR/unused-imports-in-test-module.rs:33:9 --> $DIR/unused-imports-in-test-module.rs:39:9
| |
LL | use super::a; LL | use super::a;
| ^^^^^^^^ | ^^^^^^^^
| |
help: consider adding a `#[cfg(test)]` to the containing module help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:32:1 --> $DIR/unused-imports-in-test-module.rs:38:1
| |
LL | mod a_test { LL | mod a_test {
| ^^^^^^^^^^ | ^^^^^^^^^^
error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-module.rs:36:13
|
LL | use crate::b;
| ^^^^^^^^
|
help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:32:1
|
LL | mod a_test {
| ^^^^^^^^^^
error: unused import: `super::a`
--> $DIR/unused-imports-in-test-module.rs:41:9
|
LL | use super::a;
| ^^^^^^^^
|
help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:40:1
|
LL | mod tests_a {
| ^^^^^^^^^^^
error: unused import: `crate::b`
--> $DIR/unused-imports-in-test-module.rs:44:13
|
LL | use crate::b;
| ^^^^^^^^
|
help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:40:1
|
LL | mod tests_a {
| ^^^^^^^^^^^
error: unused import: `super::a` error: unused import: `super::a`
--> $DIR/unused-imports-in-test-module.rs:49:9 --> $DIR/unused-imports-in-test-module.rs:49:9
| |
@ -139,32 +67,26 @@ LL | use super::a;
help: consider adding a `#[cfg(test)]` to the containing module help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:48:1 --> $DIR/unused-imports-in-test-module.rs:48:1
| |
LL | mod a_tests { LL | mod tests_a {
| ^^^^^^^^^^^ | ^^^^^^^^^^^
error: unused import: `crate::b` error: unused import: `super::a`
--> $DIR/unused-imports-in-test-module.rs:52:13 --> $DIR/unused-imports-in-test-module.rs:59:9
| |
LL | use crate::b; LL | use super::a;
| ^^^^^^^^ | ^^^^^^^^
| |
help: consider adding a `#[cfg(test)]` to the containing module help: consider adding a `#[cfg(test)]` to the containing module
--> $DIR/unused-imports-in-test-module.rs:48:1 --> $DIR/unused-imports-in-test-module.rs:58:1
| |
LL | mod a_tests { LL | mod a_tests {
| ^^^^^^^^^^^ | ^^^^^^^^^^^
error: unused import: `super::a` error: unused import: `super::a`
--> $DIR/unused-imports-in-test-module.rs:57:9 --> $DIR/unused-imports-in-test-module.rs:69:9
| |
LL | use super::a; LL | use super::a;
| ^^^^^^^^ | ^^^^^^^^
error: unused import: `crate::b` error: aborting due to 8 previous errors
--> $DIR/unused-imports-in-test-module.rs:60:13
|
LL | use crate::b;
| ^^^^^^^^
error: aborting due to 15 previous errors