about summary refs log tree commit diff
diff options
context:
space:
mode:
authorMichael Wright <mikerite@lavabit.com>2018-09-29 13:57:04 +0200
committerMichael Wright <mikerite@lavabit.com>2018-09-29 13:57:04 +0200
commitdb5c63b77ae022eae16d94fd63ff2d264e57c830 (patch)
tree41876d052b8e68b0a8f79411749859ddb3b023e6
parent4b4d758ce02591d2bb470a19926e3c315c18fef3 (diff)
downloadrust-db5c63b77ae022eae16d94fd63ff2d264e57c830.tar.gz
rust-db5c63b77ae022eae16d94fd63ff2d264e57c830.zip
Move tests into separate file
-rw-r--r--tests/ui/methods.rs14
-rw-r--r--tests/ui/methods.stderr32
-rw-r--r--tests/ui/unnecessary_filter_map.rs12
-rw-r--r--tests/ui/unnecessary_filter_map.stderr32
4 files changed, 45 insertions, 45 deletions
diff --git a/tests/ui/methods.rs b/tests/ui/methods.rs
index 111c6ce5a8c..7faa45b987d 100644
--- a/tests/ui/methods.rs
+++ b/tests/ui/methods.rs
@@ -443,17 +443,3 @@ fn main() {
     let opt = Some(0);
     let _ = opt.unwrap();
 }
-
-/// Checks implementation of `UNNECESSARY_FILTER_MAP` lint
-fn unnecessary_filter_map() {
-    let _ = (0..4).filter_map(|x| if x > 1 { Some(x) } else { None });
-    let _ = (0..4).filter_map(|x| { if x > 1 { return Some(x); }; None });
-    let _ = (0..4).filter_map(|x| match x {
-        0 | 1 => None,
-        _ => Some(x),
-    });
-
-    let _ = (0..4).filter_map(|x| Some(x + 1));
-
-    let _ = (0..4).filter_map(i32::checked_abs);
-}
diff --git a/tests/ui/methods.stderr b/tests/ui/methods.stderr
index b006813e26d..3189f375647 100644
--- a/tests/ui/methods.stderr
+++ b/tests/ui/methods.stderr
@@ -453,35 +453,5 @@ error: used unwrap() on an Option value. If you don't want to handle the None ca
     |
     = note: `-D clippy::option-unwrap-used` implied by `-D warnings`
 
-error: this `.filter_map` can be written more simply using `.filter`
-   --> $DIR/methods.rs:449:13
-    |
-449 |     let _ = (0..4).filter_map(|x| if x > 1 { Some(x) } else { None });
-    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-    |
-    = note: `-D clippy::unnecessary-filter-map` implied by `-D warnings`
-
-error: this `.filter_map` can be written more simply using `.filter`
-   --> $DIR/methods.rs:450:13
-    |
-450 |     let _ = (0..4).filter_map(|x| { if x > 1 { return Some(x); }; None });
-    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-
-error: this `.filter_map` can be written more simply using `.filter`
-   --> $DIR/methods.rs:451:13
-    |
-451 |       let _ = (0..4).filter_map(|x| match x {
-    |  _____________^
-452 | |         0 | 1 => None,
-453 | |         _ => Some(x),
-454 | |     });
-    | |______^
-
-error: this `.filter_map` can be written more simply using `.map`
-   --> $DIR/methods.rs:456:13
-    |
-456 |     let _ = (0..4).filter_map(|x| Some(x + 1));
-    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-
-error: aborting due to 60 previous errors
+error: aborting due to 56 previous errors
 
diff --git a/tests/ui/unnecessary_filter_map.rs b/tests/ui/unnecessary_filter_map.rs
new file mode 100644
index 00000000000..dd6cdc5d39d
--- /dev/null
+++ b/tests/ui/unnecessary_filter_map.rs
@@ -0,0 +1,12 @@
+fn main() {
+    let _ = (0..4).filter_map(|x| if x > 1 { Some(x) } else { None });
+    let _ = (0..4).filter_map(|x| { if x > 1 { return Some(x); }; None });
+    let _ = (0..4).filter_map(|x| match x {
+        0 | 1 => None,
+        _ => Some(x),
+    });
+
+    let _ = (0..4).filter_map(|x| Some(x + 1));
+
+    let _ = (0..4).filter_map(i32::checked_abs);
+}
diff --git a/tests/ui/unnecessary_filter_map.stderr b/tests/ui/unnecessary_filter_map.stderr
new file mode 100644
index 00000000000..045802047d2
--- /dev/null
+++ b/tests/ui/unnecessary_filter_map.stderr
@@ -0,0 +1,32 @@
+error: this `.filter_map` can be written more simply using `.filter`
+ --> $DIR/unnecessary_filter_map.rs:2:13
+  |
+2 |     let _ = (0..4).filter_map(|x| if x > 1 { Some(x) } else { None });
+  |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+  |
+  = note: `-D clippy::unnecessary-filter-map` implied by `-D warnings`
+
+error: this `.filter_map` can be written more simply using `.filter`
+ --> $DIR/unnecessary_filter_map.rs:3:13
+  |
+3 |     let _ = (0..4).filter_map(|x| { if x > 1 { return Some(x); }; None });
+  |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+error: this `.filter_map` can be written more simply using `.filter`
+ --> $DIR/unnecessary_filter_map.rs:4:13
+  |
+4 |       let _ = (0..4).filter_map(|x| match x {
+  |  _____________^
+5 | |         0 | 1 => None,
+6 | |         _ => Some(x),
+7 | |     });
+  | |______^
+
+error: this `.filter_map` can be written more simply using `.map`
+ --> $DIR/unnecessary_filter_map.rs:9:13
+  |
+9 |     let _ = (0..4).filter_map(|x| Some(x + 1));
+  |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+error: aborting due to 4 previous errors
+