forked from detekt/detekt
/
UseSumOfInsteadOfFlatMapSizeSpec.kt
161 lines (148 loc) · 5.21 KB
/
UseSumOfInsteadOfFlatMapSizeSpec.kt
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
package io.gitlab.arturbosch.detekt.rules.style
import io.gitlab.arturbosch.detekt.rules.KotlinCoreEnvironmentTest
import io.gitlab.arturbosch.detekt.test.compileAndLintWithContext
import org.assertj.core.api.Assertions.assertThat
import org.jetbrains.kotlin.cli.jvm.compiler.KotlinCoreEnvironment
import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.Test
@KotlinCoreEnvironmentTest
class UseSumOfInsteadOfFlatMapSizeSpec(val env: KotlinCoreEnvironment) {
val subject = UseSumOfInsteadOfFlatMapSize()
@Test
@DisplayName("Reports flatMap and size")
fun reportFlatMapAndSize() {
val code = """
fun test(list: List<Foo>) {
list.flatMap { it.foo }.size
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).hasSize(1)
assertThat(actual[0].message).isEqualTo("Use 'sumOf' instead of 'flatMap' and 'size'")
}
@Test
@DisplayName("Reports flatMap and count")
fun reportFlatMapAndCount() {
val code = """
fun test(list: List<Foo>) {
list.flatMap { it.foo }.count()
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).hasSize(1)
assertThat(actual[0].message).isEqualTo("Use 'sumOf' instead of 'flatMap' and 'count'")
}
@Test
@DisplayName("Reports flatMap and count with a argument")
fun reportFlatMapAndCountWithArgument() {
val code = """
fun test(list: List<Foo>) {
list.flatMap { it.foo }.count { it > 2 }
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).hasSize(1)
assertThat(actual[0].message).isEqualTo("Use 'sumOf' instead of 'flatMap' and 'count'")
}
@Test
@DisplayName("Reports flatten and size")
fun reportFlattenAndSize() {
val code = """
fun test(list: List<List<Int>>) {
list.flatten().size
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).hasSize(1)
assertThat(actual[0].message).isEqualTo("Use 'sumOf' instead of 'flatten' and 'size'")
}
@Test
@DisplayName("Reports flatMap and size on nullable list")
fun reportFlatMapAndSizeOnNullableList() {
val code = """
fun test(list: List<Foo>?) {
list?.flatMap { it.foo }?.size
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).hasSize(1)
}
@Test
@DisplayName("Reports flatMap and size on implicit list receiver")
fun reportFlatMapAndSizeOnImplicitListReceiver() {
val code = """
fun List<Foo>.test() {
flatMap { it.foo }.size
sumOf { it.foo.size }
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).hasSize(1)
}
@Test
@DisplayName("Reports flatMap and count on Set")
fun reportFlatMapAndCountOnSet() {
val code = """
fun test(set: Set<Bar>) {
set.flatMap { it.bar }.count()
}
class Bar(val bar: Set<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).hasSize(1)
}
@Test
@DisplayName("Does not report flatMap")
fun noReportFlatMap() {
val code = """
fun test(list: List<Foo>) {
list.flatMap { it.foo }
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).isEmpty()
}
@Test
@DisplayName("Does not report flatMap and first")
fun noReportFlatMapAndFirst() {
val code = """
fun test(list: List<Foo>) {
list.flatMap { it.foo }.first()
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).isEmpty()
}
@Test
@DisplayName("Does not report flatten")
fun noReportFlatten() {
val code = """
fun test(list: List<List<Int>>) {
list.flatten()
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).isEmpty()
}
@Test
@DisplayName("Does not report flatten and last")
fun noReportFlattenAndLast() {
val code = """
fun test(list: List<List<Int>>) {
list.flatten().last()
}
class Foo(val foo: List<Int>)
""".trimIndent()
val actual = subject.compileAndLintWithContext(env, code)
assertThat(actual).isEmpty()
}
}